-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
chore(argo-workflows): update workflows version to most 3.0.7 #811
chore(argo-workflows): update workflows version to most 3.0.7 #811
Conversation
Signed-off-by: Alex Sears <me@alexsears.com>
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.
Hello 👋 ,
Thanks for you contribution. I think we should also sync these changes here:
- feat(controller): Add liveness probe argo-workflows#5875
- fix(controller): Increase readiness timeout from 1s to 30s argo-workflows#6007
These changes were introduced in 3.0.4+
Very cool. I'll tackle this later tonight. |
Signed-off-by: Alex Sears <me@alexsears.com>
@@ -77,12 +77,16 @@ spec: | |||
ports: | |||
- name: metrics | |||
containerPort: {{ .Values.controller.metricsConfig.port }} | |||
- containerPort: 6060 | |||
livenessProbe: |
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.
I think this should be configurable. In the current configuration the controller may be down up to four minutes until Kubernetes will detect that, and many people will want a faster reaction. The entire section may be put into the config, with the value moved into in values.yaml
:
livenessProbe: {{ .Values.controller.livenessProbe | toYaml | nindent 12 }}
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.
good call. this has been updated
Signed-off-by: Alex Sears <me@alexsears.com>
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
…oj#811) * chore(argo-workflows): update workflows version to most 3.0.7 Signed-off-by: Alex Sears <me@alexsears.com> * chore(argo-worflows): update controller liveness probe Signed-off-by: Alex Sears <me@alexsears.com> * chore(argo-workflows): templatize controller liveness probe Signed-off-by: Alex Sears <me@alexsears.com> Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
Chart.yaml
following Semantic Versioning.Changes are automatically published when merged to
master
. They are not published on branches.