Skip to content

feat(chart): make retina-operator securityContext configurable - #2593

Open
Rémi BUISSON (darkweaver87) wants to merge 1 commit into
microsoft:mainfrom
darkweaver87:feat/configurable-operator-securitycontext
Open

feat(chart): make retina-operator securityContext configurable#2593
Rémi BUISSON (darkweaver87) wants to merge 1 commit into
microsoft:mainfrom
darkweaver87:feat/configurable-operator-securitycontext

Conversation

@darkweaver87

@darkweaver87 Rémi BUISSON (darkweaver87) commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Motivation

The retina-operator pod and container securityContext are hardcoded, so hardening fields (e.g. readOnlyRootFilesystem, seccompProfile) can't be set without patching the rendered manifest. This exposes operator.podSecurityContext and operator.securityContext, defaulted to the current values, so rendered output is unchanged.

@darkweaver87

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="TraefikLabs"

@github-actions

Copy link
Copy Markdown

This PR will be closed in 7 days due to inactivity.

@github-actions github-actions Bot added the meta/waiting-for-author Blocked and waiting on the author label Aug 28, 2026
@rbtr Evan Baker (rbtr) removed the meta/waiting-for-author Blocked and waiting on the author label Aug 28, 2026
@rbtr

Copy link
Copy Markdown
Collaborator

Hey Rémi BUISSON (@darkweaver87), thanks for the improvement here. Can you sign your commits? You've done sign-off (git commit -s) but we also need cryptographic signature (git commit -S). Thanks!

Signed-off-by: darkweaver87 <remi-buisson@orange.fr>
@darkweaver87
Rémi BUISSON (darkweaver87) force-pushed the feat/configurable-operator-securitycontext branch from 27a2460 to 1cb1504 Compare August 31, 2026 07:28
@darkweaver87

Copy link
Copy Markdown
Contributor Author

Thanks Evan Baker (@rbtr), done :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants