-
Notifications
You must be signed in to change notification settings - Fork 327
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(kuma-cp) support other service name in DP injector #1491
Conversation
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.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.
Looks perfectly fine to me! LGTM
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
@@ -687,4 +687,4 @@ spec: | |||
secret: | |||
secretName: loki | |||
- name: storage | |||
emptyDir: {} | |||
emptyDir: {} |
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.
Does it mean a new update mode for golden files isn't adding an empty line at the end?
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.
maybe. It's just overriding file with the output content. If content has no empty line at the end, there won't be empty line in the golden file. Is that a problem?
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.
No, no problem, GitHub so much highlights the fact there is no empty line at the end, so I asked :)
pkg/test/matchers/golden_yaml.go
Outdated
} | ||
|
||
func MatchGoldenJSON(goldenFilePath string) types.GomegaMatcher { | ||
return MatchGolden(gomega.MatchYAML, goldenFilePath) |
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 it should be MatchJSON
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com>
Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com> (cherry picked from commit 4d2d01c) # Conflicts: # app/kumactl/pkg/install/k8s/control-plane/helmtemplates_vfsdata.go # mk/e2e.new.mk
…1494) * chore(kuma-cp) support other service name in DP injector (#1491) Signed-off-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com> (cherry picked from commit 4d2d01c) # Conflicts: # app/kumactl/pkg/install/k8s/control-plane/helmtemplates_vfsdata.go # mk/e2e.new.mk * fix(*) conflicts Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com> Co-authored-by: Jakub Dyszkiewicz <jakub.dyszkiewicz@gmail.com> Co-authored-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Summary
Full changelog
kuma_runtime_kubernetes_control_plane_service_name
so Kuma the service name change can be applied and injector will inject Kuma DP with proper URLkuma-cni
service account name where as the service account was created with this patternserviceAccountName: {{ include "kuma.name" . }}-cni
templates/pre-"
,templates/post-
, not only prefix. This way when kuma is a sub chart, it will work.Documentation