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

remove edge tests #10527

Merged
merged 6 commits into from
Jan 29, 2025
Merged

remove edge tests #10527

merged 6 commits into from
Jan 29, 2025

Conversation

jenshu
Copy link
Contributor

@jenshu jenshu commented Jan 28, 2025

Remove kubernetes e2e tests related to:

  • edge api
  • glooctl
  • validation webhook

Note: there may still be some references to the deleted tests/code; that should be ok since the tests don't get compiled right now. Any dangling references can be removed in follow-ups when we re-enable tests

Details

Features (i.e. folders under test/kubernetes/e2e/features) that were completely removed, because they only contained edge tests:

  • basicrouting
  • glooctl
  • gloomtls
  • validation

Features whose folders still remain, but with some/all of the edge-specific tests removed (these may be further pruned later):

  • client_tls
  • crd_categories
  • headless_svc
  • helm
  • istio
  • port_routing
  • server_tls
  • tracing
  • watch_namespace_selector

Any features not mentioned above, were not touched in this PR

@jenshu jenshu marked this pull request as ready for review January 28, 2025 18:47
@jenshu jenshu changed the title [WIP] remove edge tests remove edge tests Jan 28, 2025
@lgadban lgadban added this pull request to the merge queue Jan 29, 2025
Merged via the queue into kgateway-dev:main with commit ca9b260 Jan 29, 2025
8 checks passed
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.

2 participants