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

Install manifests use :master image even in tagged commits #4971

Closed
ikorolev93 opened this issue Jan 28, 2020 · 2 comments
Closed

Install manifests use :master image even in tagged commits #4971

ikorolev93 opened this issue Jan 28, 2020 · 2 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@ikorolev93
Copy link

NGINX Ingress controller version:
Starting from 0.27.0 and up

Kubernetes version (use kubectl version):

Environment:

  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:

What happened:
Tried to upgrade my existing kustomize-based installation just by replacing the git tag for remote base. Ended up having quay.io/kubernetes-ingress-controller/nginx-ingress-controller:master in the manifests (as a workaround I patched it in my kustomization.yaml).

Checking the official documentation, it also features :master images (docs suggest to run kubectl apply -f https://raw.githubusercontent.com/kubernetes/ingress-nginx/nginx-0.27.1/deploy/static/mandatory.yaml, which has :master tag).

What you expected to happen:
Having tagged commits reference an exact version, like it was in 0.26.2 and before.

What do you think went wrong?
#4887 was merged and it affected all tags after that

How to reproduce it:

curl -s https://raw.githubusercontent.com/kubernetes/ingress-nginx/nginx-0.27.1/deploy/static/mandatory.yaml | grep nginx-ingress-controller:master

Anything else we need to know:

/kind bug

@ikorolev93 ikorolev93 added the kind/bug Categorizes issue or PR as related to a bug. label Jan 28, 2020
@aledbf
Copy link
Member

aledbf commented Jan 28, 2020

Closing. Please update to 0.28.0

@aledbf aledbf closed this as completed Jan 28, 2020
@ikorolev93
Copy link
Author

Okay, static manifests have the right version now, but deploy/cloud-generic/kustomization.yaml still points to master:

images:
- name: quay.io/kubernetes-ingress-controller/nginx-ingress-controller
newTag: master

even though deployment.yaml has the right version:
image: quay.io/kubernetes-ingress-controller/nginx-ingress-controller:0.28.0

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

No branches or pull requests

2 participants