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

Use ingress for e2e tests #160

Merged
merged 10 commits into from
May 12, 2023
Merged

Use ingress for e2e tests #160

merged 10 commits into from
May 12, 2023

Conversation

pablochacin
Copy link
Collaborator

Description

Set up an ingress controller for e2e test clusters and expose services using ingresses instead of node ports.
This will allow multiple test cases of the same test to run concurrently on the same cluster without collision.

It is a step towards solving #147 but different tests still need to use different host ports to avoid conflicts.
This limitation will be addressed in a follow-up PR.

Fixes #

Checklist:

  • My code follows the coding style of this project
  • I have performed a self-review of my code
  • I have commented on my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works.
  • Any dependent changes have been merged and published in downstream modules

Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
@pablochacin pablochacin merged commit ae8fb5f into main May 12, 2023
6 checks passed
@pablochacin pablochacin deleted the e2e-test-ingress branch May 12, 2023 15:35
@pablochacin pablochacin mentioned this pull request Aug 9, 2023
8 tasks
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.

None yet

1 participant