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

Addons: AWS Pod Identity Webhook doesn't support sidecar containers. #16802

Closed
eliasscosta opened this issue Sep 3, 2024 · 0 comments · Fixed by #16803
Closed

Addons: AWS Pod Identity Webhook doesn't support sidecar containers. #16802

eliasscosta opened this issue Sep 3, 2024 · 0 comments · Fixed by #16803
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@eliasscosta
Copy link
Contributor

eliasscosta commented Sep 3, 2024

/kind bug

1. What kops version are you running? The command kops version, will display
this information.

Client version: 1.29.0 (git-v1.29.0)

2. What Kubernetes version are you running? kubectl version will print the
version if a cluster is running or provide the Kubernetes version specified as
a kops flag.

Server Version: v1.29.4

3. What cloud provider are you using?

AWS

4. What commands did you run? What is the simplest way to reproduce this issue?

When we create a pod with sidecar containers, the mutate webhook from amazon-eks-pod-identity-webhook removes the restartPolicy=Always from our initContainer.

Ref:

5. What happened after the commands executed?

Throw an error making it impossible to use the sidecar containers.

Pod "wp-01j6vwf91vw2rmhqmdrtyk76zv" is invalid: spec.initContainers[0].readinessProbe: Forbidden: may not be set for init containers without restartPolicy=Always

6. What did you expect to happen?

Expect the mutate don't remove the field from initContainers

7. Please provide your cluster manifest. Execute
kops get --name my.example.com -o yaml to display your cluster manifest.
You may want to remove your cluster name and other sensitive information.

Not applicable, the addon is a very old version.

9. Anything else do we need to know?

The fix release on version 0.5.3 of the Pod Webhook Identity

@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Sep 3, 2024
@eliasscosta eliasscosta changed the title Addons: AWS Pod Identity Webhook doesn't support native sidecar containers. Addons: AWS Pod Identity Webhook doesn't support sidecar containers. Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants