-
Notifications
You must be signed in to change notification settings - Fork 77
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
revise kubelinter config / Add securityContext section to Helm chart #833
Conversation
[CHATOPS:HELP] ChatOps commands.
|
961d7b7
to
aee65d4
Compare
This PR may conflict with #826 (maybe charts/vald/values.yaml).so it is better to wait for it to merge into master. |
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
I think it's okay to merge before merge #826, I'll add same configuration into gateway/{lb, backup, meta} |
82a2c46
to
e124223
Compare
5ba0765
to
d3395f2
Compare
7c91c4b
to
1b8b8a3
Compare
Signed-off-by: Rintaro Okamura <[email protected]> :pencil2: Fix typo Signed-off-by: Rintaro Okamura <[email protected]> :wrench: Add securityContext section to newly added componentes Signed-off-by: Rintaro Okamura <[email protected]>
Signed-off-by: Rintaro Okamura <[email protected]>
1b8b8a3
to
6676151
Compare
Signed-off-by: Rintaro Okamura <[email protected]>
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
Signed-off-by: Rintaro Okamura [email protected]
Description:
Related Issue:
nothing
How Has This Been Tested?:
nothing
Environment:
Types of changes:
Changes to Core Features:
Checklist: