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(e2e): update drift suite #650

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

test(e2e): update drift suite #650

wants to merge 15 commits into from

Conversation

tallaxes
Copy link
Collaborator

@tallaxes tallaxes commented Jan 14, 2025

Description

  • Update E2E drift suite (among other things, adds budget and AKSNodeClass tests). Mostly aligned with AWS provider.
  • Fix AKSNodeClass hashing (add KubeletConfig) and add tests

Other E2E improvements:

  • Fix test AKSNodeClass overrides
  • Remove drift feature gate override (no longer needed)
  • Add check for leaked leases
  • Override coretest.DefaultImage (pause) with MCR version, once
  • Have make az-e2etests also run az-cleanenv

How was this change tested?

Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: #
  • No

Release Note


@tallaxes tallaxes added area/e2e-test-framework Issues or PRs related to refactoring the e2e test framework area/aksnodeclass Issues or PRs related to AKSNodeClass area/e2e-testing Issues or PRs related to e2e testing area/drift Issues or PRs related to Drift labels Jan 14, 2025
@tallaxes tallaxes self-assigned this Jan 14, 2025
@tallaxes tallaxes mentioned this pull request Jan 15, 2025
14 tasks
@tallaxes
Copy link
Collaborator Author

tallaxes commented Jan 16, 2025

Scenarios testing Drift failure have been failing occasionally, so added more By (f6d20e0) and attempted to deflake / make it more deterministic (deaa7ae). The last run passed, but may need to revisit if it fails again.

@tallaxes tallaxes requested a review from charliedmcb January 16, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/aksnodeclass Issues or PRs related to AKSNodeClass area/drift Issues or PRs related to Drift area/e2e-test-framework Issues or PRs related to refactoring the e2e test framework area/e2e-testing Issues or PRs related to e2e testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants