-
Notifications
You must be signed in to change notification settings - Fork 15
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
spec.template.spec.containers[0].image: Required value #38
Comments
Nevermind, I had a typo in the config. |
Hey , Having the same issue. What was the typo ? |
@huegelc can you help here? |
@Stolr I had a typo in my deployment yaml. This is a working example:
|
This is still an issue for me |
@echel0n @Stolr can you confirm that this is only for v0.0.23? Also plz share
|
@usamaahmadkhan : I only tried with v0.0.23 not with earlier version ( Earlier was not working #36 ) . K8S : 1.16.4 Configmap :
( Converted to a secret ) proxyinjector
And here is a deployment I'm trying to anotate
|
I am also having the same issue here. |
nvm i managed to get it working.
|
Is there any news about this ? |
@Stolr remove the proxyconfig: in the configmap like described by @kw-jk above. PRs are welcome for a permanent fix. :) |
Awesome @usamaahmadkhan thanks ! |
I am also getting same issue: Error: cannot patch "search-manager" with kind Deployment: Deployment.apps "search-manager" is invalid: spec.template.spec.containers[1].image: Required value Deployment.yaml apiVersion: apps/v1 Configmap.yaml kind: ConfigMap |
did you found a solution for that issue ? |
@n00bsi unfortunately I can't tell... It's been 5 years |
After upgrading to 0.0.23 I get following error when I want to inject a proxy container:
This is the deployment:
The text was updated successfully, but these errors were encountered: