-
Notifications
You must be signed in to change notification settings - Fork 2
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
Chart: Rename to ingress-nginx
.
#464
Conversation
Since fixing CI would require to rename the whole repository, I'm going to pause here and start into weekend... probably better. 😌 |
068fe29
to
a2c7e88
Compare
d49d592
to
290e179
Compare
8d935c0
to
45b2b50
Compare
Only thing left to do would be updating the test dependencies and updating the remaining occurrences of |
0bf68bc
to
282ab89
Compare
I just renamed all occurrences. |
525cda2
to
57ebdf8
Compare
Works as a fresh install in AWS & Azure, gonna explore & document a migration path for the "existing" part. |
15d171c
to
4e98c94
Compare
ingress-nginx
.ingress-nginx
.
ingress-nginx
.ingress-nginx
.
ingress-nginx
.nginx-ingress-controller-app
to ingress-nginx
.
nginx-ingress-controller-app
to ingress-nginx
.ingress-nginx
.
4e98c94
to
720ffa6
Compare
720ffa6
to
564ad00
Compare
lgtm! |
* Repository: Rename `nginx-ingress-controller-app` to `ingress-nginx`. * CircleCI: Do not push to app collections. * Tests: Fix & rework.
For changes in the chart, chart templates and container images, I executed the following tests, using the
hello-world
app, to verify them in live environments:Ingress
resources are reconciledIngress
resources are reconciled