-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding config command #244
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linter & some wording fixes :D
I'd pre-approve but that would be dissmissed anyway
cmd/kubehound/config.go
Outdated
configCmd = &cobra.Command{ | ||
Use: "config", | ||
Short: "Show the current configuration", | ||
Long: `[devOnly] Spawn the kubehound dev stack for the system-tests (build from dockerfile)`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong comment
cmd/kubehound/config.go
Outdated
return nil | ||
} | ||
|
||
fmt.Println(" --- YAML ---") //nolint:forbidigo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't it confusing that you use ---
since it's also the "start of document" in yaml? But at the same time the rest of the line should fail since it's not valid yaml (and it also has a space prefixed, that might be a typo)?
"github.com/DataDog/KubeHound/pkg/cmd" | ||
"github.com/DataDog/KubeHound/pkg/telemetry/log" | ||
"github.com/spf13/cobra" | ||
"gopkg.in/yaml.v2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we have this dep already so it's fine, but otherwise I'd ask the reason of not being on v3.
Adding a
config
command to view the current config KubeHound (with a flag--path
to dump the content):