-
Notifications
You must be signed in to change notification settings - Fork 103
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
Helm: Fix extraArgs #2288
Helm: Fix extraArgs #2288
Conversation
Hi @kariya-mitsuru. Thanks for your PR! I am @adamjensenbot.
Make sure this PR appears in the liqo changelog, adding one of the following labels:
|
Hi @kariya-mitsuru, thanks for your contribution. In other components we use this syntax to add extraargs https://github.com/kariya-mitsuru/liqo/blob/a3748d493921bc07262461d7277a21fd5d9e633c/deployments/liqo/templates/liqo-crd-replicator-deployment.yaml#L39-L41. Can you adapt your commit to our syntax or modify all the other components? |
721e5b2
to
a5340da
Compare
LGTM! 👍🏻 @kariya-mitsuru |
ExtraArgs was not applied in the following two deployments and one cronjob, so fixed it so that it is applied correctly. - deployments/liqo/templates/liqo-metric-agent-deployment.yaml - deployments/liqo/templates/liqo-proxy-deployment.yaml - deployments/liqo/templates/liqo-telemetry-cronjob.yaml
29aca63
to
e95a162
Compare
/rebase test=true |
/merge |
Description
ExtraArgs was not applied in the following two deployments and one cronjob, so fixed it so that it is applied correctly.