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

Add gateway listeners scaling test #529

Merged
merged 1 commit into from
Sep 18, 2024
Merged

Conversation

averevki
Copy link
Contributor

Changes

  • Overwrite wait_for_ready for DNSPolicy with timeout increased to 5 min instead of 1 min, large amount of DNS records take some time to create
  • Create 2 gateways with max listeners and see if both dns policies are enforced and some random listeners are reachable

Closes #528

@averevki averevki added Test case New test case DNS Issues for DNSOperator labels Sep 10, 2024
@averevki averevki self-assigned this Sep 10, 2024
trepel
trepel previously approved these changes Sep 11, 2024
Copy link
Contributor

@trepel trepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am approving this one. However, it failed for me twice out of ~8 times so the 300s timelimit might need to be increased. We can adjust based on flakiness we observe in nightly pipelines.

testsuite/kuadrant/policy/dns.py Outdated Show resolved Hide resolved
@averevki
Copy link
Contributor Author

averevki commented Sep 11, 2024

I am approving this one. However, it failed for me twice out of ~8 times so the 300s timelimit might need to be increased. We can adjust based on flakiness we observe in nightly pipelines.

I expect some problems with this test due to a large amount of DNS records getting created/deleted. I too am getting timeouts sometimes when I'm trying to run this test again right after the execution. I hope nightlies will handle a single run of this test in parallel with other ones

azgabur
azgabur previously approved these changes Sep 13, 2024
@averevki
Copy link
Contributor Author

rebase^

trepel
trepel previously approved these changes Sep 17, 2024
@trepel
Copy link
Contributor

trepel commented Sep 17, 2024

  • Overwrite wait_for_ready for DNSPolicy with timeout increased to 5 min instead of 1 min, large amount of DNS records take some time to create

LGTM, I just can't see any wait_for_ready being overridden so I am a bit confused. You probably mean that wait_for_full_enforced were overridden (which is used in wait_for_ready)

trepel
trepel previously approved these changes Sep 17, 2024
@jsmolar jsmolar merged commit 3359909 into Kuadrant:main Sep 18, 2024
3 checks passed
@averevki averevki deleted the dns-scale-test branch September 18, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNS Issues for DNSOperator Test case New test case
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DNS Scale test (gateway with 64 listeners)
4 participants