Skip to content

Ability to skip container mutation (for example for istio)#22

Merged
trublast merged 1 commit intomainfrom
feature/skip-mutation
Feb 18, 2025
Merged

Ability to skip container mutation (for example for istio)#22
trublast merged 1 commit intomainfrom
feature/skip-mutation

Conversation

@trublast
Copy link
Contributor

@trublast trublast commented Feb 17, 2025

Description

  • Ability to skip the mutation of certain containers via the secrets-store.deckhouse.io/skip-mutate-containers pod annotation or the skip_mutate_containers webhook environment variable.
  • CVE updates

Why do we need it, and what problem does it solve?

Istio initContainers do not have access to Vault when using the secrets-store.deckhouse.io/env-from-path annotation, so we want to skip such containers.

What is the expected result?

Pods with the secrets-store.deckhouse.io/env-from-path annotation and the sidecar.istio.io/inject: "true" label can start successfully.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@trublast trublast force-pushed the feature/skip-mutation branch from e336e44 to 3621f05 Compare February 17, 2025 19:17
@trublast trublast added the enhancement New feature or request label Feb 17, 2025
Signed-off-by: Maksim Kiselev <maksim.kiselev@flant.com>
@trublast trublast force-pushed the feature/skip-mutation branch from 8cb41ef to 18dbbdb Compare February 17, 2025 21:56
@trublast trublast merged commit 0958740 into main Feb 18, 2025
2 checks passed
@trublast trublast deleted the feature/skip-mutation branch February 21, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant