-
Notifications
You must be signed in to change notification settings - Fork 262
Adding securityContext options at the pod and container level. #494
Conversation
The tests seem to be failing. |
|
containerSecurityContext: {}
helmOperator: {}
tiller: {} needs to be change to containerSecurityContext:
helmOperator: {}
tiller: {} |
Thanks for contributing! Just a few pointers to get these tests passing |
@hiddeco can we rerun tests? |
Rebasing and force-pushing will trigger a new build while also getting rid of the |
chart/helm-operator/Chart.yaml
Outdated
@@ -1,6 +1,6 @@ | |||
apiVersion: v1 | |||
appVersion: "1.2.0" | |||
version: 1.1.0 | |||
version: 1.1.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.
FYI: we do not bump versions on changes, but bump it on release.
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.
Thanks @swade1987 🌷
Allowing the configuration of
SecurityContext
at the pod and individual container levels.