-
Notifications
You must be signed in to change notification settings - Fork 331
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
chore(*) remove old Ingress #3435
Conversation
Signed-off-by: Jakub Dyszkiewicz <[email protected]>
904250a
to
3896338
Compare
Codecov Report
@@ Coverage Diff @@
## master #3435 +/- ##
==========================================
+ Coverage 52.20% 52.29% +0.09%
==========================================
Files 939 939
Lines 54747 54395 -352
==========================================
- Hits 28581 28447 -134
+ Misses 23929 23728 -201
+ Partials 2237 2220 -17
Continue to review full report at Codecov.
|
Signed-off-by: Jakub Dyszkiewicz <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be released in Kuma 1.4.1? It is a breaking change but I hope everyone migrated a long time ago.
Is there a reason to remove it in 1.4.1?
IMHO we should not make breaking changes in point releases. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet. +1 on not backporting. We can ship this change on 1.5.0.
Signed-off-by: Jakub Dyszkiewicz <[email protected]>
…gress Signed-off-by: Jakub Dyszkiewicz <[email protected]>
Summary
Remove support for old Ingress embedded in Dataplane. In Kuma 1.2.0 we introduced the ZoneIngress, now it's time to finally get rid of the old ingress.
Issues resolved
Fix #3166
Documentation
Testing
Backwards compatibility
UPGRADE.md
with any steps users will need to takewhen upgrading.