Skip to content

Conversation

@fabriziopandini
Copy link
Member

What this PR does / why we need it:
Add new lifecycle hooks / change existing hooks as defined by the lifecycle hooks proposal.

Which issue(s) this PR fixes:
Rif #12720

@fabriziopandini fabriziopandini added the area/runtime-sdk Issues or PRs related to Runtime SDK label Oct 20, 2025
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Oct 20, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign chrischdi for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Oct 20, 2025
@fabriziopandini fabriziopandini force-pushed the add-chained-upgrade-lifecyclehooks branch 2 times, most recently from e8af4c6 to 44888d3 Compare October 22, 2025 13:21
Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure why diff is so bad 😓
what I did is

  • minor adjustments to TestComputeControlPlaneVersion
  • drop TestComputeControlPlaneVersion_callAfterControlPlaneUpgrade and TestComputeControlPlaneVersion_callBeforeClusterUpgrade_trackIntentOfCallingAfterClusterUpgrade because they are replaced by the new test in lifecycle_hooks_test.go

@fabriziopandini fabriziopandini force-pushed the add-chained-upgrade-lifecyclehooks branch from 44888d3 to cae6d82 Compare October 22, 2025 13:27
@fabriziopandini
Copy link
Member Author

/test pull-cluster-api-e2e-main

@fabriziopandini fabriziopandini force-pushed the add-chained-upgrade-lifecyclehooks branch from cae6d82 to 21b0d44 Compare October 23, 2025 19:52
@fabriziopandini
Copy link
Member Author

/test pull-cluster-api-e2e-main

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

Labels

area/runtime-sdk Issues or PRs related to Runtime SDK cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants