Skip to content

[charts/sn-platform, sn-platform-slim] Address security findings - #1284

Merged
tuteng merged 7 commits into
masterfrom
fix/ticket-4974-operator-security-values
Sep 30, 2026
Merged

tuteng merged 7 commits into
masterfrom
fix/ticket-4974-operator-security-values

Conversation

@tuteng

@tuteng tuteng commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Changes

  • Expose Secret volume modes and projected volume configuration for operator-managed workloads in both sn-platform and sn-platform-slim.
  • Add chart controls for Detector probes, Console init container and Job security contexts, and initialization Job TTLs; harden the Toolset and JWT initialization containers.
  • Enable proxy.securityContext.readOnlyRootFilesystem: true by default in both charts. The existing proxy.securityContext template passes this policy to PulsarProxy.spec.pod.securityContext, including the generated check-broker init container with the updated operator.
  • Keep optional numeric values such as Secret volume modes and Job TTLs compatible with the Helm version used by chart CI, including explicit 0 values.

Validation

  • helm lint passes for both charts with default and all chart CI values files.
  • helm template confirms both Proxy CRs contain spec.pod.securityContext.readOnlyRootFilesystem: true by default, and optional numeric fields preserve 0.
  • Earlier test releases of both charts reached Ready in the EKS pulsar namespace with a read-only Proxy main container. Live validation of the new operator-generated check-broker context is pending access to the upgraded cluster.

Documentation

  • doc — updated both chart READMEs with the operator dependency and Proxy security context behavior.

@tuteng
tuteng requested review from a team as code owners September 29, 2026 02:47
Copilot AI balanced review requested due to automatic review settings September 29, 2026 02:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

@tuteng:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions Bot added the doc-info-missing This pr needs to mark a document option in description label Sep 29, 2026
@tuteng tuteng changed the title feat(sn-platform-slim): expose operator Secret volume modes feat: expose operator Secret volume modes Sep 29, 2026
@tuteng tuteng changed the title feat: expose operator Secret volume modes [charts/sn-platform, sn-platform-slim] Address ticket 4974 security findings Sep 29, 2026
@github-actions github-actions Bot removed the doc-info-missing This pr needs to mark a document option in description label Sep 29, 2026
@github-actions

Copy link
Copy Markdown

@tuteng:Thanks for providing doc info!

@github-actions github-actions Bot added the doc This pr contains a document label Sep 29, 2026
@tuteng tuteng changed the title [charts/sn-platform, sn-platform-slim] Address ticket 4974 security findings [charts/sn-platform, sn-platform-slim] Address security findings Sep 29, 2026
@tuteng
tuteng merged commit a82b776 into master Sep 30, 2026
5 checks passed
@tuteng
tuteng deleted the fix/ticket-4974-operator-security-values branch September 30, 2026 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc This pr contains a document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants