Skip to content

set securityContext's from values#291

Open
dmaes wants to merge 2 commits intocrowdsecurity:mainfrom
dmaes:main
Open

set securityContext's from values#291
dmaes wants to merge 2 commits intocrowdsecurity:mainfrom
dmaes:main

Conversation

@dmaes
Copy link

@dmaes dmaes commented Jul 31, 2025

I actually needed #90 , but since it hasn't had any activity since over a year, and is making multiple changes (that IMHO should be split into different PR's) at once, I've created this simple change that gives users the flexibility to overwrite the securityContext definitions, while at the same time not breaking any existing deployments.

Replaces hardcoded securityContext stanza's with
values that can be overwritten by users if needed
@github-actions github-actions bot added the needs/kind Kind label required label Jul 31, 2025
@github-actions
Copy link

@dmaes: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.

  • /kind feature
  • /kind enhancement
  • /kind fix
  • /kind chore
  • /kind dependencies
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the forked project rr404/oss-governance-bot repository.

@github-actions
Copy link

@dmaes: There are no area labels on this PR. You can add as many areas as you see fit.

  • /area agent
  • /area local-api
  • /area cscli
  • /area security
  • /area configuration
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the forked project rr404/oss-governance-bot repository.

@dmaes
Copy link
Author

dmaes commented Jul 31, 2025

/kind enhancement
/area configuration

@github-actions github-actions bot added kind/enhancement New feature or request area/configuration and removed needs/kind Kind label required needs/area labels Jul 31, 2025
@fwmarcel
Copy link

I am also interested in this PR.

securityContext:
allowPrivilegeEscalation: false
privileged: false
{{- toYaml .Values.appsec.wai_for_lapi.securityContext | nindent 10 }}

Choose a reason for hiding this comment

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

Small typo here, should be wait_for_lapi

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants