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

chore(*) remove old Ingress #3435

Merged
merged 4 commits into from
Dec 8, 2021
Merged

Conversation

jakubdyszkiewicz
Copy link
Contributor

@jakubdyszkiewicz jakubdyszkiewicz commented Dec 6, 2021

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

  • Docs were replaced long time ago.

Testing

  • Unit tests
  • E2E tests
  • Manual testing on Universal
  • Manual testing on Kubernetes

Backwards compatibility

  • Update UPGRADE.md with any steps users will need to take
    when upgrading.
  • Should this be released in Kuma 1.4.1? It is a breaking change but I hope everyone migrated a long time ago.

Signed-off-by: Jakub Dyszkiewicz <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #3435 (3896338) into master (1642741) will increase coverage by 0.09%.
The diff coverage is 37.50%.

Impacted file tree graph

@@            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     
Impacted Files Coverage Δ
api/mesh/v1alpha1/dataplane_helpers.go 85.58% <ø> (+3.84%) ⬆️
pkg/api-server/dataplane_overview_endpoints.go 66.66% <ø> (ø)
...kg/core/resources/apis/mesh/dataplane_validator.go 97.79% <ø> (+3.90%) ⬆️
...kg/core/resources/apis/mesh/generated_resources.go 81.02% <ø> (ø)
...g/core/resources/apis/mesh/zone_ingress_helpers.go 11.42% <ø> (-22.45%) ⬇️
pkg/dns/vips_allocator.go 73.75% <ø> (-0.76%) ⬇️
pkg/kds/context/context.go 52.63% <0.00%> (ø)
pkg/kds/zone/components.go 16.39% <ø> (-0.53%) ⬇️
...ugins/bootstrap/k8s/xds/hooks/api_server_bypass.go 0.00% <0.00%> (ø)
...ns/runtime/k8s/controllers/configmap_controller.go 8.16% <ø> (-5.73%) ⬇️
... and 21 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1642741...3896338. Read the comment docs.

Signed-off-by: Jakub Dyszkiewicz <[email protected]>
@jakubdyszkiewicz jakubdyszkiewicz marked this pull request as ready for review December 6, 2021 17:53
@jakubdyszkiewicz jakubdyszkiewicz requested a review from a team as a code owner December 6, 2021 17:53
Copy link
Contributor

@michaelbeaumont michaelbeaumont left a 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?

@jpeach
Copy link
Contributor

jpeach commented Dec 7, 2021

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.

Copy link
Contributor

@lahabana lahabana left a 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.

@jakubdyszkiewicz jakubdyszkiewicz enabled auto-merge (squash) December 8, 2021 10:14
@jakubdyszkiewicz jakubdyszkiewicz merged commit 1d984c8 into master Dec 8, 2021
@jakubdyszkiewicz jakubdyszkiewicz deleted the chore/remove-old-ingress branch December 8, 2021 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate ingress
5 participants