ACM-14578: Adding supported TLS ciphers #178
Conversation
| @@ -25,6 +25,7 @@ local defaults = { | |||
| tls: {}, | |||
| rateLimiter: {}, | |||
| internal: {}, | |||
| securityContext: {}, | |||
There was a problem hiding this comment.
@jacobbaungard my update introduces this change (stolostron/observatorium#79) is this going to be a problem?
There was a problem hiding this comment.
Hmmm, good question. I don't believe it would be a problem.
We do have this in the running obs-api container
securityContext:
privileged: false
readOnlyRootFilesystem: true
It is added in the operator here:
As far as I understand we add those settings on top of any set by the securityContext here. Might be worth to just double check that the deployment though when testing this change.
|
|
/hold |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jacobbaungard, moadz The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/retest |
|
/test e2e-kind |
1 similar comment
|
/test e2e-kind |
Addresses https://issues.redhat.com/browse/ACM-14578