-
Notifications
You must be signed in to change notification settings - Fork 486
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: Support https for serviceAccounts and InternalTrafficPolicy for services. #6383
Conversation
relabelings: [] | ||
# - sourceLabels: [__meta_kubernetes_pod_node_name] | ||
# separator: ; | ||
# regex: ^(.*)$ | ||
# targetLabel: nodename | ||
# replacement: $1 | ||
# action: replace | ||
relabelings: [] |
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.
nit: can we move the example below the relabelings so that it doesn't get rendered weirdly on the README?
@@ -10,6 +10,14 @@ internal API changes are not present. | |||
Unreleased | |||
---------- | |||
|
|||
0.33.0 (2024-02-16) |
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.
nit: let's mention this on the commit message or split the version bump in a different commit so it's easier to keep track of when it came out?
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 % couple small nits you may want to check out.
PR Description
Which issue(s) this PR fixes
Notes to the Reviewer
PR Checklist