-
Notifications
You must be signed in to change notification settings - Fork 418
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
Update e2e tests for Ginkgo v2 #2486
Update e2e tests for Ginkgo v2 #2486
Conversation
/assign @CecileRobertMichon @Jont828 |
Does this not need the CAPI version of these changes to merge first? |
No, these are just future-proofing changes that work with either v1 or v2 of Ginkgo. Also we are using |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CecileRobertMichon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Fixes CAPZ's misuse of Ginkgo container nodes and updates a
ParallelNodes
reference such that both will work when we move to Ginkgo v2.Nesting a
Context
inside anIt
container node is a warning in Ginkgo v1 but an error in v2. We can fix this now for correctness, to simplify the transition to v2, and to prevent more copy-and-pasting of this mistake.Which issue(s) this PR fixes:
Refs #2466
Special notes for your reviewer:
TODOs:
Release note: