-
Notifications
You must be signed in to change notification settings - Fork 148
feat: add custom ingress label values #4063
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
Conversation
jessesimpson36
left a comment
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.
Implementation and tests look fine, but one minor point on including the labels within camundaPlatform.labels helper function rather than in the template.
|
I believe that we need this feature for all versions. |
I've discussed with @hisImminence, and we will start with 8.8 and later add support for other versions. |
aabouzaid
left a comment
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.
Great, LGTM 🙌
d1b27dc to
9628bf6
Compare
Which problem does the PR fix?
product-hub #2679
What's in this PR?
Adds a new labels value to each ingress in values.yaml
Checklist
Please make sure to follow our Contributing Guide.
Before opening the PR:
make go.update-golden-only.After opening the PR: