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

Fix gateway failover test #1431

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

MaxBab
Copy link
Contributor

@MaxBab MaxBab commented Oct 19, 2023

The gateway failover test fails to locate gateway resource node in On-premise environments.
All environments created in a public cloud platforms sets the node in a Gateway resource without any suffix.
On-premise environment may set node name in a way, that suffix will be defined in Gateway resource.

Search for node in a Gateway resource in both ways - with and without a suffix.

Fixes: #1127

@submariner-bot
Copy link

🤖 Created branch: z_pr1431/MaxBab/fix_gateway_failover_test

@MaxBab MaxBab force-pushed the fix_gateway_failover_test branch from 6fad68c to 090f3fa Compare October 19, 2023 21:03
test/e2e/framework/gateways.go Outdated Show resolved Hide resolved
@MaxBab MaxBab force-pushed the fix_gateway_failover_test branch 3 times, most recently from 14a40a9 to cf60c33 Compare October 23, 2023 17:36
The gateway failover test fails to locate gateway resource node in
On-premise environments.
All environments created in a public cloud platforms sets the node in a
Gateway resource without any suffix.
On-premise environment may set node name in a way, that suffix will be
defined in Gateway resource.

Search for node in a Gateway resource in both ways - with and without a
suffix.

Signed-off-by: Maxim Babushkin <[email protected]>
@MaxBab MaxBab force-pushed the fix_gateway_failover_test branch from cf60c33 to de66f6b Compare October 23, 2023 17:37
@MaxBab MaxBab requested a review from tpantelis October 23, 2023 17:55
@skitt skitt merged commit 01a7cd7 into submariner-io:devel Oct 24, 2023
43 checks passed
@submariner-bot
Copy link

🤖 Closed branches: [z_pr1431/MaxBab/fix_gateway_failover_test]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug in gateway testing framework
4 participants