SPLAT-2410: Promote ClusterAPIMachineManagementVSphere to TechPreview#2724
SPLAT-2410: Promote ClusterAPIMachineManagementVSphere to TechPreview#2724AnnaZivkovic wants to merge 1 commit intoopenshift:masterfrom
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Hello @AnnaZivkovic! Some important instructions when contributing to openshift/api: |
|
@AnnaZivkovic: This pull request references SPLAT-2410 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Cache: Disabled due to data retention organization setting Knowledge base: Disabled due to data retention organization setting 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe PR updates ClusterAPIMachineManagementVSphere feature-gate usage: features.go now initializes the gate for both TechPreviewNoUpgrade and DevPreviewNoUpgrade (previously only DevPreviewNoUpgrade). Two payload manifests (Hypershift-TechPreviewNoUpgrade and SelfManagedHA-TechPreviewNoUpgrade) move ClusterAPIMachineManagementVSphere from the disabled list to the enabled list. features.md shows an edit where the ClusterAPIMachineManagementVSphere row was removed and then re-added with no other logic changes. No public APIs were altered. 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.5.0)Error: build linters: unable to load custom analyzer "kubeapilinter": tools/_output/bin/kube-api-linter.so, plugin: not implemented Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Comment |
|
/test verify |
ⓘ Your monthly quota for Qodo has expired. Upgrade your plan ⓘ Paying users. Check that your Qodo account is linked with this Git user account |
|
/lgtm |
|
Scheduling tests matching the |
|
/test e2e-aws-serial-2of2 |
ⓘ Your monthly quota for Qodo has expired. Upgrade your plan ⓘ Paying users. Check that your Qodo account is linked with this Git user account |
|
/approve |
|
/retest |
1 similar comment
|
/retest |
|
/override ci/prow/e2e-aws-serial-techpreview-1of2 Confident that this isn't a related failure |
|
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/e2e-aws-serial-techpreview-1of2 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/verified by CI |
|
@JoelSpeed: This PR has been marked as verified by DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/test verify |
ⓘ Your monthly quota for Qodo has expired. Upgrade your plan ⓘ Paying users. Check that your Qodo account is linked with this Git user account |
|
@AnnaZivkovic You'll need to rebase your code on latest master and run the generation again to fix verify |
Enable ClusterAPIMachineManagementVSphere feature gate in TechPreviewNoUpgrade feature set in addition to DevPreviewNoUpgrade.
b68ca1a to
9b8f43c
Compare
|
/lgtm |
|
Scheduling tests matching the |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcpowermac, JoelSpeed The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Enable ClusterAPIMachineManagementVSphere feature gate in TechPreviewNoUpgrade feature set in addition to DevPreviewNoUpgrade.