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: add extraArgs to clustermesh-apiserver #25693
helm: add extraArgs to clustermesh-apiserver #25693
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.
Thanks for the PR @rcanderson23, Helm changes LGTM, bringing in @cilium/sig-clustermesh to the review.
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.
Thanks @rcanderson23! Changes look good to me.
You should run the following commands to update the auto-generated files:
make -C install/kubernetes
make -C Documentation update-helm-values
It seems that the linter is not yet happy. @rcanderson23 Could you please rerun again |
f913af3
to
89e5978
Compare
/test Job 'Cilium-PR-K8s-1.26-kernel-net-next' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.26-kernel-net-next/292/ If it is a flake and a GitHub issue doesn't already exist to track it, comment Then please upload the Jenkins artifacts to that issue. |
Cilium L4LB XDP hit #25663. Rerunning |
/test-1.26-net-next Hit #24514 Job 'Cilium-PR-K8s-1.26-kernel-net-next' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.26-kernel-net-next/355/ If it is a flake and a GitHub issue doesn't already exist to track it, comment Then please upload the Jenkins artifacts to that issue. |
Cilium Runtime is failing because the PR is not up-to-date with main, but these tests are not required at the moment. |
/test-1.26-net-next
Job 'Cilium-PR-K8s-1.26-kernel-net-next' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.26-kernel-net-next/375/ If it is a flake and a GitHub issue doesn't already exist to track it, comment Then please upload the Jenkins artifacts to that issue. |
@rcanderson23 Could you please rebase onto main to pick the latest changes and make the CI happy? |
89e5978
to
cde84cf
Compare
This allows the ability supply extra arguments to clustermesh-apiserver. It is currently not possible to supply other kvstore-opts via env variables due higher precedence given to arguments. Signed-off-by: Carson Anderson <carson.anderson@goteleport.com>
cde84cf
to
4387b2f
Compare
/test |
/test-runtime Hit known flake: #25291 |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.
This adds
extraArgs
to clustermesh-apiserver in the helm chart. This will allow users to increasekvstore-opt
such asetcd.qps
which can't be overriden via env variables since args take precedence.