Skip to content

Address seccomp profile todo:#28384

Merged
Honny1 merged 1 commit intocontainers:mainfrom
EDuToit:seccomp
Mar 27, 2026
Merged

Address seccomp profile todo:#28384
Honny1 merged 1 commit intocontainers:mainfrom
EDuToit:seccomp

Conversation

@EDuToit
Copy link
Copy Markdown
Contributor

@EDuToit EDuToit commented Mar 26, 2026

  • resolve seccomp profile from config default
  • assign profile if default path is not defined

Fixes: #28379

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • Tests have been added/updated (or no tests are needed)
  • Documentation has been updated (or no documentation changes are needed)
  • All commits pass make validatepr (format/lint checks)
  • Release note entered in the section below (or None if no user-facing changes)

Does this PR introduce a user-facing change?

None

@github-actions github-actions Bot added the kind/api-change Change to remote API; merits scrutiny label Mar 26, 2026
- resolve seccomp profile from config default
- assign profile if default path is not defined

Signed-off-by: EDuToit <eben.dutoit07@gmail.com>
@packit-as-a-service
Copy link
Copy Markdown

[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore.

@mheon mheon added the No New Tests Allow PR to proceed without adding regression tests label Mar 26, 2026
@mheon
Copy link
Copy Markdown
Member

mheon commented Mar 26, 2026

Sure
LGTM

@EDuToit
Copy link
Copy Markdown
Contributor Author

EDuToit commented Mar 26, 2026

#28379

Copy link
Copy Markdown
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@Honny1 Honny1 merged commit d0e0571 into containers:main Mar 27, 2026
81 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/api-change Change to remote API; merits scrutiny No New Tests Allow PR to proceed without adding regression tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Properly set Seccomp profile path if non-default in Compat Info endpoint

3 participants