-
Notifications
You must be signed in to change notification settings - Fork 37
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
fix: extra pod specs for cra #116
Conversation
1bda138
to
5b05e0e
Compare
@soniqua, could you add an helm unittest to verify usage of |
3190452
to
205b825
Compare
205b825
to
d0dc981
Compare
path: spec.template.spec | ||
template: broker_deployment.yaml |
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.
@soniqua, nice usage instead of giga-snapshot :)
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.
LGTM 👍
Not a huge fan of the naming, this allows for extra fields in the PodSpec, but the name could be interpreted as extra distinct PodSpecs. However, appreciate it's existing and would be a breaking change to modify.
Allows
extraPodSpecs
to be specified for Container Registry Agent