Skip to content

Conversation

@jatinsu
Copy link

@jatinsu jatinsu commented Dec 2, 2025

This PR is to create an enhancement for the OKD featureset.

For more context: openshift/api#2451

@jatinsu jatinsu marked this pull request as draft December 2, 2025 20:27
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 2, 2025
@openshift-ci openshift-ci bot requested review from ashcrow and hasbro17 December 2, 2025 20:27
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 2, 2025

[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 jeffdyoung 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

@jatinsu jatinsu force-pushed the OKD-featuregates branch 2 times, most recently from 075812e to a0218a1 Compare December 10, 2025 16:53
#### Scenario 1: Installing a new OKD cluster

1. OKD cluster administrator initiates a cluster installation using `openshift-install` built for OKD
2. The installer automatically sets the feature set to "OKD" in the cluster configuration
Copy link
Author

Choose a reason for hiding this comment

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

Should this be CVO?

- Resource consumption impact should be minimal as the feature set itself only controls which features are enabled, not the features themselves
- Individual features enabled by the OKD feature set may have their own resource implications

**MicroShift:**
Copy link
Author

Choose a reason for hiding this comment

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

OKD featureset shouldn't affect microshift right?

- For the initial implementation, no override is planned
- Can be added in a future enhancement if a compelling use case emerges

## Test Plan
Copy link
Author

Choose a reason for hiding this comment

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

Could you verify the Test Plan? We haven't really talked about creating tests. Should I create a JIRA ticket to create tests for the featureset?

- Feature set remains OKD after upgrade
- Feature gates are correctly maintained across upgrades

## Graduation Criteria
Copy link
Author

Choose a reason for hiding this comment

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

Could you verify this as well? To me, graduation criteria is really meant for features, not really the featureset. I was thinking of omitting this section, but I wanted to ask before I did so

### Upgrade Strategy

**OKD Clusters:**
- Existing OKD clusters without the OKD feature set: During the upgrade to the first version supporting the OKD feature set, the feature set should be automatically enabled if the cluster is detected as OKD. This should be handled by the cluster-version-operator or similar component.
Copy link
Author

Choose a reason for hiding this comment

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

I don't believe we've ever tested this. Would current OKD clusters have the featureset automatically applied once they upgrade? Or would the featureset only apply to new clusters?

### Downgrade Strategy

**Downgrading from a version with OKD feature set to a version without:**
- If an OKD cluster with the OKD feature set is downgraded to a version that does not recognize the OKD feature set:
Copy link
Author

Choose a reason for hiding this comment

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

Does OKD/OpenShift support downgrades?


**Recommendation:** Implement Option 1 (automatic migration) with clear logging and documentation. This provides the smoothest upgrade experience for OKD users.

## Version Skew Strategy
Copy link
Author

Choose a reason for hiding this comment

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

Could you verify the "Version Skew Strategy" section as well

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

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants