Release 0.22.0 #3658
Release 0.22.0 #3658
Conversation
aledbf
added
the
do-not-merge/work-in-progress
k8s-ci-robot
added
size/L
aledbf
force-pushed
the
release-0.22
branch
2 times, most recently
from
9d4ffd3 to
faea265
Compare
Changelog.md
Outdated
| - Adds support for HTTP2 Push Preload annotation | ||
| - Allow to disable NGINX prometheus metrics | ||
| - Add --disable-catch-all option to disable catch-all server | ||
| - Add flag --metrics-per-hos to make per-host metrics optional |
There was a problem hiding this comment.
--metrics-per-hos -> --metrics-per-host
Changelog.md
Outdated
| - New balancer implementation: consistent hash subset | ||
| - Adds support for HTTP2 Push Preload annotation | ||
| - Allow to disable NGINX prometheus metrics | ||
| - Add --disable-catch-all option to disable catch-all server |
There was a problem hiding this comment.
What about:
New --disable-catch-all flag to ignore catch-all ingresses
| - [X] [#3648](https://github.com/kubernetes/ingress-nginx/pull/3648) Remove stickyness cookie domain from Lua balancer to match old behavior | ||
| - [X] [#3649](https://github.com/kubernetes/ingress-nginx/pull/3649) Empty access_by_lua_block breaks satisfy any | ||
| - [X] [#3655](https://github.com/kubernetes/ingress-nginx/pull/3655) Remove flag sort-backends | ||
| - [X] [#3656](https://github.com/kubernetes/ingress-nginx/pull/3656) Change default value of flag for ssl chain completion |
There was a problem hiding this comment.
Can this be a breaking change potentially? For users that's relying on ingress controller to complete the chain.
| - [X] [#3569](https://github.com/kubernetes/ingress-nginx/pull/3569) fix status updated: make sure ingress.status is copied | ||
| - [X] [#3573](https://github.com/kubernetes/ingress-nginx/pull/3573) Update Certificate Generation Docs to not use MD5 | ||
| - [X] [#3581](https://github.com/kubernetes/ingress-nginx/pull/3581) lua randomseed per worker | ||
| - [X] [#3582](https://github.com/kubernetes/ingress-nginx/pull/3582) Sort ingresses by creation timestamp |
There was a problem hiding this comment.
This is also a significant change of behaviour - until now the most recently updated ingress annotation would take precedence but with this change updates won't matter anymore - only creation times matter.
There was a problem hiding this comment.
until now the most recently updated ingress annotation would take precedence but with this change updates won't matter anymore - only creation times matter.
No. This only matters when there is an overlap in the path definition. This change ensures we always use the first created Ingress.
aledbf
added
the
do-not-merge/work-in-progress
|
/lgtm |
k8s-ci-robot
added
the
lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aledbf, ElvinEfendi The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
aledbf
removed
the
do-not-merge/work-in-progress
Test image:
quay.io/kubernetes-ingress-controller/nginx-ingress-controller:dev