-
Notifications
You must be signed in to change notification settings - Fork 25
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
Updating helm chart #130
base: main
Are you sure you want to change the base?
Updating helm chart #130
Conversation
Merge when 0.4.o is released. |
@@ -25,9 +31,12 @@ spec: | |||
- JSONPath: .spec.maxReplicas | |||
name: max replicas | |||
type: integer | |||
- JSONPath: .spec.dryRun | |||
- JSONPath: .status.conditions[?(@.type=="DryRun")].status |
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 what I was talking about during the standup.
- JSONPath: .status.conditions[0].type | ||
name: condition | ||
type: string | ||
- JSONPath: .status.conditions[0].status | ||
name: condition status | ||
type: string |
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.
how can we be sure it is always the first item in the conditions slice?
ddProfiling: | ||
enabled: false | ||
|
||
workers: 1 |
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.
IMO to be backward compatible the workers
value should be empty. because here you will always add the new args that doesn't exist on previous version.
OR, you add an if
in the template file. Something like
{{- if no eq (int .Values.workers) 1 }}
- -workers={{ .Values.workers }}
{{- end }}
What does this PR do?
Upon merging releasing 0.4.0, we update the chart.
Motivation
seamless upgrade experience for users.
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
Write there any instructions and details you may have to test your PR.