OCPBUGS-74528: upkeep: HighlyAvailableArbiter has been GA for 2 releases#2759
OCPBUGS-74528: upkeep: HighlyAvailableArbiter has been GA for 2 releases#2759eggfoobar wants to merge 1 commit intoopenshift:masterfrom
Conversation
removing the HighlyAvailableArbiter featureGate, the feature has been GA for 2 releases Signed-off-by: ehila <ehila@redhat.com>
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
openshift-ci-robot
added
jira/valid-reference
|
Hello @eggfoobar! Some important instructions when contributing to openshift/api: |
|
@eggfoobar: This pull request references Jira Issue OCPBUGS-74528, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. 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. |
openshift-ci
bot
added
the
size/XXL
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (10)
📒 Files selected for processing (12)
💤 Files with no reviewable changes (10)
📝 WalkthroughWalkthroughThis pull request removes the HighlyAvailableArbiter feature gate across multiple components. Changes include removing the exported feature gate declaration, eliminating HighlyAvailableArbiter from all feature gate manifest files, updating the ControlPlaneTopology validation schema to allow HighlyAvailableArbiter without requiring the feature gate, removing documentation entries, and eliminating arbiter-specific logic from the test analyzer tool. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
ⓘ You are approaching your monthly quota for Qodo. Upgrade your plan Review Summary by QodoRemove HighlyAvailableArbiter feature gate - GA for 2 releases
WalkthroughsDescription• Remove HighlyAvailableArbiter feature gate after GA stabilization • Make HighlyAvailableArbiter topology mode always available • Update validation rules to allow arbiter mode unconditionally • Fix code formatting and indentation in feature gate definitions Diagramflowchart LR
A["HighlyAvailableArbiter<br/>Feature Gate"] -->|Remove| B["Always Available<br/>Topology Mode"]
C["Feature Gate<br/>Validation Rules"] -->|Simplify| D["Unconditional<br/>Enum Values"]
E["CRD Manifests"] -->|Update| F["Include Arbiter<br/>in All Profiles"]
File Changes1. features/features.go
|
Code Review by Qodo
1. controlPlaneTopology enum undocumented
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
| // +openshift:validation:FeatureGateAwareEnum:featureGate="",enum=HighlyAvailable;HighlyAvailableArbiter;SingleReplica;External | ||
| // +openshift:validation:FeatureGateAwareEnum:featureGate=DualReplica,enum=HighlyAvailable;HighlyAvailableArbiter;SingleReplica;DualReplica;External |
There was a problem hiding this comment.
1. controlplanetopology enum undocumented 📘 Rule violation ✓ Correctness
The controlPlaneTopology field comment does not document the HighlyAvailableArbiter and DualReplica enum values that are now allowed by validation markers. This leaves API consumers without an explanation of the behavior/meaning of these permitted values.
Agent Prompt
## Issue description
The `controlPlaneTopology` field's validation markers allow additional enum values (`HighlyAvailableArbiter` and `DualReplica`), but the field comment does not explain those values.
## Issue Context
Compliance requires that enum constraints implied by validation markers be documented in the field comment so users understand the API contract.
## Fix Focus Areas
- config/v1/types_infrastructure.go[99-109]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
|
@eggfoobar: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
Looks like the integration tests still reference the gate, that will need to be removed What payload E2E can we run to check that this PR doesn't break any existing jobs? Is there a way to run an arbiter job? |
3 participants
removing the HighlyAvailableArbiter featureGate, the feature has been GA for 2 releases