Skip to content

[WIP] chore: Update curl container securityContext settings#101

Closed
coreydaley wants to merge 1 commit intoredhat-developer:mainfrom
coreydaley:2025-02-12-curl-pod-security-context-fix
Closed

[WIP] chore: Update curl container securityContext settings#101
coreydaley wants to merge 1 commit intoredhat-developer:mainfrom
coreydaley:2025-02-12-curl-pod-security-context-fix

Conversation

@coreydaley
Copy link
Copy Markdown

@coreydaley coreydaley commented Feb 13, 2025

Description of the change

Set secure defaults for the securityContext on the curl container.

Existing or Associated Issue(s)

https://issues.redhat.com/browse/RHIDP-6005

Additional Information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the values.yaml and added to the README.md. The pre-commit utility can be used to generate the necessary content. Use pre-commit run -a to apply changes.
  • JSON Schema template updated and re-generated the raw schema via pre-commit hook.
  • List tests pass for Chart using the Chart Testing tool and the ct lint command.

@openshift-ci openshift-ci Bot requested review from gazarenkov and tumido February 13, 2025 02:20
@coreydaley coreydaley changed the title chore: Update curl container securityContext settings [WIP] chore: Update curl container securityContext settings Feb 13, 2025
@coreydaley
Copy link
Copy Markdown
Author

I am working on finding a solution for the non-numeric user id, whether that is getting CURL to fix their image, finding another one that works, or building our own

@coreydaley coreydaley closed this Feb 13, 2025
@coreydaley coreydaley reopened this Feb 13, 2025
@sonarqubecloud
Copy link
Copy Markdown

Comment thread charts/backstage/ci/with-custom-image-for-test-pod-values.yaml Outdated
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2025

@coreydaley coreydaley closed this May 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants