Skip to content

[CWS] Add declared hardening infos - #519

Open
mftoure wants to merge 4 commits into
masterfrom
momar/hardening-declared
Open

[CWS] Add declared hardening infos#519
mftoure wants to merge 4 commits into
masterfrom
momar/hardening-declared

Conversation

@mftoure

@mftoure mftoure commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

Reviewers: please see the review guidelines.

@mftoure mftoure changed the title Momar/hardening declared [CWS] Add declared hardening infos Sep 10, 2026
@mftoure
mftoure marked this pull request as ready for review September 10, 2026 11:55
@mftoure
mftoure requested a review from a team as a code owner September 10, 2026 11:55
Comment thread proto/cws/dumpsv1/activity_dump.proto Outdated
optional string localhost_profile = 2;
}

message HardeningDeclared {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry for the bikeshedding but should this be something like SecurityContext ?

Comment thread proto/cws/dumpsv1/activity_dump.proto Outdated
optional string localhost_profile = 2;
}

message HardeningDeclared {

@spikat spikat Sep 10, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we rename this to ContainerSecurityContext instead? IMHO it better represents the data inside and stick to the workloadmeta. WDYT?

@mftoure
mftoure force-pushed the momar/hardening-declared branch from 12158e2 to 44bb6d8 Compare September 10, 2026 15:23
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.

3 participants