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

test(policy): update e2e_egress_network for reliability #13412

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

olix0r
Copy link
Member

@olix0r olix0r commented Dec 2, 2024

The e2e_egress_network test is failing for two reasons:

  1. httpbin.org is currently down and returning 503 errors.
  2. The default tests change the default policies and relaunch a container after the policy is changed. In under-resourced environments, this update may be slow or unreliable.

This change fixes this by:

  1. Using postman-echo.com instead of httpbin.org.
  2. Splitting the tests into distinct allow and deny tests to avoid mutating the default policy.

@olix0r olix0r requested a review from a team as a code owner December 2, 2024 20:56
The e2e_egress_network test is failing for two reasons:

1. httpbin.org is currently down and returning 503 errors.
2. The default tests change the default policies and relaunch a container after
   the policy is changed. In under-resourced environments, this update may be
   slow or unreliable.

This change fixes this by:

1. Using postman-echo.com instead of httpbin.org.
2. Splitting the tests into distinct allow and deny tests to avoid mutating the
   default policy.
@olix0r olix0r force-pushed the ver/e2e-egress-postman branch from 87d47c9 to 233597f Compare December 2, 2024 20:59
@olix0r olix0r enabled auto-merge (squash) December 2, 2024 21:17
@olix0r olix0r merged commit a2e1883 into main Dec 2, 2024
43 checks passed
@olix0r olix0r deleted the ver/e2e-egress-postman branch December 2, 2024 21:39
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