-
Notifications
You must be signed in to change notification settings - Fork 45
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
charts,salt: Bump NGINX Ingress chart to v4.10.3 #4395
Conversation
Hello m4nch0t,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command:
Alternatively, the |
Bump the NGINX Ingress chart to v4.10.3 and also bump the Ingress Controller image to v1.10.3 Upgrade NGINX Ingress chart using: ``` rm -rf charts/ingress-nginx helm repo add ingress-nginx https://kubernetes.github.io/ingress-nginx helm repo update helm fetch -d charts --untar ingress-nginx/ingress-nginx --version 4.10.3 ``` Re-render NGINX Ingress salt states using: ``` ./doit.sh codegen:chart_ingress-nginx ``` Also update the dashboards with ``` curl https://raw.githubusercontent.com/kubernetes/ingress-nginx/helm-chart-4.10.3/deploy/grafana/dashboards/request-handling-performance.json \ -Lo salt/metalk8s/addons/nginx-ingress/deployed/files/ingress-nginx-performance.json curl https://raw.githubusercontent.com/kubernetes/ingress-nginx/helm-chart-4.10.3/deploy/grafana/dashboards/request-handling-performance.json \ -Lo salt/metalk8s/addons/nginx-ingress/deployed/files/ingress-nginx-performance.json ```
6fc0162
to
c174d82
Compare
/create_integration_branches |
ConflictA conflict has been raised during the creation of I have not created the integration branch. Here are the steps to resolve this conflict: $ git fetch
$ git checkout -B w/128.0/improvement/bump-ingress origin/development/128.0
$ git merge origin/improvement/bump-ingress
$ # <intense conflict resolution>
$ git commit
$ git push -u origin w/128.0/improvement/bump-ingress The following options are set: create_integration_branches |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: The following options are set: create_integration_branches |
/approve |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue None. Goodbye m4nch0t. The following options are set: approve, create_integration_branches |
Bump the NGINX Ingress chart to v4.10.3 and also bump the Ingress Controller image to v1.10.3
Also update the dashboards