Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not mutate pod spec security context #592

Conversation

tspearconquest
Copy link
Contributor

@tspearconquest tspearconquest commented Aug 19, 2023

Fixes #591

@tspearconquest tspearconquest force-pushed the do_not_mutate_pod_spec_security_context branch from 40ebbe5 to 7141e5f Compare August 19, 2023 11:12
…mutating the pod spec (which affects containers other than its own and overwrites what a user, administrator, or another webhook may have set)

Signed-off-by: Thomas Spear <[email protected]>
Copy link
Collaborator

@181192 181192 left a comment

Choose a reason for hiding this comment

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

Great, thanks @tspearconquest 😌

@181192 181192 merged commit 6a63704 into SparebankenVest:master Aug 28, 2023
@HammerNL89
Copy link

HammerNL89 commented Aug 28, 2023

Great, thanks @tspearconquest 😌

Do you have an eta for new (patch) release?

@tspearconquest
Copy link
Contributor Author

Hi @181192 we need this turned into a 1.5.1 release please.

@181192
Copy link
Collaborator

181192 commented Aug 28, 2023

@tspearconquest @HammerNL89 I have added a pre-release 1.5.1-beta.1

@HammerNL89
Copy link

I have added a pre-release 1.5.1-beta.1

Thanks, do you have an eta for a non beta version (1.5.1) release? We can't use beta versions. This would help us decide whether or not we revert to a previous version of akv2k8s or wait for the new release.

@Zombro
Copy link

Zombro commented Oct 18, 2023

the over-writing of pod security context is fairly problematic
I'm in the same position as @HammerNL89. cannot use a beta version. ETA on 1.5.1 release ?

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.

[BUG] Pod spec security contexts are being mutated
4 participants