-
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
introduce the component
package
#488
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.
This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, refactoring, documentation, tooling
4add6f5
to
2bb46e9
Compare
component
package
VolumeMounts: volumeMounts, | ||
Command: nil, | ||
Args: nil, | ||
SecurityContext: &corev1.SecurityContext{ |
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.
should we make this configurable as well?
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.
this is the default PodTemplateSpec, so it is not configurable.
The Idea it to create later another function that will use DatadogAgent.Spec.Override
to override the PodTemplate if needed. Or it will be configurable in a Feature
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.
sorry commented too quickly, after reviewing more i understand
Co-authored-by: Celene <celene@datadoghq.com>
What does this PR do?
Introduce the
component
package with theNewDefaultClusterAgentPodTemplateSpec
functionthat aims to create a default cluster-agent
PodTemplateSpec
Motivation
Refactor code for the feature factory
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
it is code refactoring, unit-tests and test should be green.