Skip to content

Stablize E2E Tests#287

Merged
gflarity merged 19 commits into
ai-dynamo:mainfrom
gflarity:bump_e2e_runner
Dec 11, 2025
Merged

Stablize E2E Tests#287
gflarity merged 19 commits into
ai-dynamo:mainfrom
gflarity:bump_e2e_runner

Conversation

@gflarity

@gflarity gflarity commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

What type of PR is this?

BUG

What this PR does / why we need it:

Currently our E2E tests fail intermittently at different spots. I found two three issues:

  1. The standard/free github runners only have 4cpu and 16G RAM. This sometimes causes non-deterministic timeouts.
  2. There's no check that the validating webhook is running before we start the tests. This only seems to be an issue when there's not enough CPU and things are slower.
  3. 30 seconds just isn't a long enough for the default time out, bumped it to 60 to avoid needing to re-run tests due to random timeouts.

This PR fixes both of these issues.

Does this PR introduce a API change?

NONE

Additional documentation e.g., enhancement proposals, usage docs, etc.:


@gflarity gflarity changed the title bump the e2e runner Stablize E2E Tests Dec 10, 2025
Comment thread operator/e2e/setup/k8s_clusters.go Outdated
Comment thread .github/workflows/e2e-test.yaml Outdated
Comment thread .github/workflows/e2e-test.yaml
Comment thread .github/workflows/e2e-test.yaml Outdated
Comment thread operator/e2e/setup/k8s_clusters.go
Comment thread operator/e2e/setup/k8s_clusters.go Outdated
@gflarity gflarity merged commit 0f07586 into ai-dynamo:main Dec 11, 2025
4 of 6 checks passed
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.

4 participants