Skip to content

Fix E2E tests#8139

Merged
TiberiuGC merged 1 commit intomainfrom
fix-integration-tests
Jan 13, 2025
Merged

Fix E2E tests#8139
TiberiuGC merged 1 commit intomainfrom
fix-integration-tests

Conversation

@TiberiuGC
Copy link
Copy Markdown
Contributor

Description

This PR addresses multiple E2E test failures at once, by making the following changes:

  • regenerate JSON schema
  • update test case to account for new default addon (metrics-server)
  • ensure some IPv6 related validation are only run on cluster creation (and not on other commands e.g. deletion)
  • ensure custom_ami test runs on at least EKS 1.31 as it's the minimum required for the recently added Ubuntu 24.04 support

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the userdocs directory)
  • Manually tested
  • Made sure the title of the PR is a good description that can go into the release notes
  • (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

  • Backfilled missing tests for code in same general area 🎉
  • Refactored something and made the world a better place 🌟

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-release-notes Causes PR not to show in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants