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

UDN: Add external -> services e2e #4699

Open
dceara opened this issue Sep 4, 2024 · 2 comments · May be fixed by #4702
Open

UDN: Add external -> services e2e #4699

dceara opened this issue Sep 4, 2024 · 2 comments · May be fixed by #4702
Labels
area/e2e-testing feature/user-defined-network-segmentation All PRs related to User defined network segmentation lifecycle/stale All issues (> 60 days) and PRs (>90 days) with no activity.

Comments

@dceara
Copy link
Contributor

dceara commented Sep 4, 2024

What would you like to see added/updated/deleted in our end-to-end tests?

We currently are missing tests for the "external to nodePort" case for user defined networks - we do test from a pod to a nodeport on a different node but we should probably use a "real" external client:

#4648 (comment)

Tests should be added for the load balancer case too.

Why is this needed?

Ensure better functionality testing of services in user defined networks.

@tssurya
Copy link
Contributor

tssurya commented Sep 5, 2024

/assign @qinqon

Copy link

github-actions bot commented Nov 8, 2024

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the lifecycle/stale All issues (> 60 days) and PRs (>90 days) with no activity. label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/e2e-testing feature/user-defined-network-segmentation All PRs related to User defined network segmentation lifecycle/stale All issues (> 60 days) and PRs (>90 days) with no activity.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants