-
Notifications
You must be signed in to change notification settings - Fork 98
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
[kubectl plugin] Add validate and find commands #65
Conversation
1eb1438
to
4a882f7
Compare
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.
nice PR.
I have just a proposition about how to structure the different commands to have it maybe more generic.
WDYT if we put both auto-discovery command like this
kubectl datadog validate ad pod <pod-name>
kubectl datadog validate ad service <service-name>
like this if we want to support after
kubectl datadog validate ad deployment|daemonset|statefulset <pod-name>
or others resources it will be simpler ?
Later, we can think also to implement:
kubectl datadog validate config
cmd/kubectl-datadog/find/find.go
Outdated
func New(streams genericclioptions.IOStreams) *cobra.Command { | ||
o := newOptions(streams) | ||
cmd := &cobra.Command{ | ||
Use: "find [pod name] [flags]", |
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.
What do you think if we move this command as a agent
subcommand ?
example: kubectl datadog agent find <myapp-pod>
Makes perfect sense thanks! I will update the PR accordingly 👍 |
4a882f7
to
474c334
Compare
@clamoriniere PR updated |
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.
💯
could you please update the PR description according to your change.
What does this PR do?
validate ad pod
command for pod annotationsvalidate ad service
command for service/endpoints annotationsagent find
command to get the agent monitoring a given podMotivation
make operations easier