Skip to content

STAC-25601: correct securityContext and required-privileges claims - #389

Open
viliakov wants to merge 2 commits into
rancher:stagingfrom
viliakov:STAC-25601-restricted-securitycontext
Open

STAC-25601: correct securityContext and required-privileges claims#389
viliakov wants to merge 2 commits into
rancher:stagingfrom
viliakov:STAC-25601-restricted-securitycontext

Conversation

@viliakov

Copy link
Copy Markdown
Contributor

The page claimed every chart container sets the restricted securityContext. It did not — the accompanying chart change in helm-charts-internal makes it true, and this documents what is left over.

  • Attributes the host privileges to the node agent DaemonSet (and privileged to its process-agent container) instead of "the agent"; the cluster agent, checks agent, RBAC agent, header injector and OTel components need none of them.
  • Documents the logs agent, which was missing entirely.
  • Covers CIS-hardened RKE2 as well as the Rancher-restricted PSA template, since both default to restricted.
  • Lists the three exempt containers in a table with the action for each.

Reviewer notes:

  • The "starting from version v2.3.8" provenance is dropped. Complete coverage only holds from the chart release that carries the helm-charts-internal change, and I do not know that version number yet — please add it if you do.
  • == Rancher-Restricted PSA Template became == Restrictive Pod Security policies. An explicit [#_rancher_restricted_psa_template] anchor keeps the old URL fragment working; #_required_privileges (linked from k8s-suse-rancher-prime-agent-air-gapped.adoc) is unchanged.
  • Validated with asciidoctor: renders without warnings, all anchors and xrefs resolve. No local Antora build (no node in this checkout).

Tracking: https://stackstate.atlassian.net/browse/STAC-25601

The page claimed every chart container sets the restricted securityContext,
and attributed the agent's host privileges to "the agent" rather than the node
agent specifically. Names the node agent and logs agent as the only agent
exceptions, adds CIS-hardened RKE2 alongside Rancher-restricted PSA, and lists
the three exempt containers in a table.

The "starting from version v2.3.8" provenance is dropped: complete coverage
only holds from the chart change in helm-charts-internal that accompanies this,
and the release version is not known yet.
@viliakov
viliakov requested a review from a team as a code owner August 20, 2026 14:20
@netlify

netlify Bot commented Aug 20, 2026

Copy link
Copy Markdown

Deploy Preview for suse-obs ready!

Name Link
🔨 Latest commit 7acc485
🔍 Latest deploy log https://app.netlify.com/projects/suse-obs/deploys/6a88474cd21c5d00097c9ab9
😎 Deploy Preview https://deploy-preview-389--suse-obs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

A gate-by-gate sweep of the chart turned up four more init containers that
chown data volumes as root and so cannot satisfy the restricted profile. All
are disabled by default, so they are listed as a follow-on note rather than in
the exception table.
[NOTE]
====
All SUSE Observability Helm chart containers are configured with the following `securityContext` settings starting from version `v2.3.8` and onwards:
Every container in the {stackstate-product-name} and {stackstate-product-name} Agent Helm charts sets the following `securityContext` values, so that no namespace exemption is needed for them:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could we describe these as effective settings and qualify the exceptions first? Some rendered containers inherit runAsNonRoot and seccompProfile from the pod, so not every container sets all four fields itself. The table also describes three workload cases but four containers, and the platform needs no exemption only while the optional volume-permission init containers remain disabled.

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