Skip to content

Commit

Permalink
config/v1/types_cluster_version: Add v4.17 capability set
Browse files Browse the repository at this point in the history
This is a synonym for v4.16, because there have not been
capability-recommendation changes since that release.  4.17 is about
to go Generally Available, which means v4.17 will always be a synonym
for v4.16.  I think defining synonym capability sets is busywork that
takes developer time without adding customer value, but [1] points out
that so far every 4.y release has had a v4.y capability set string
associated with it, and folks might find adjustments to that pattern
surprising.  I'm declaring the v4.17 set to continue the existing
set-for-each-4.y pattern (and breaking the
set-for-each-4.y-with-a-different-default pattern) to unblock 4.17's
GA, but we may revisit this approach in the future with more lead
time.  This commit backports (without v4.18) [2] to release-4.17.

[1]: https://issues.redhat.com/browse/OCPBUGS-41111
[2]: #2022
  • Loading branch information
wking committed Sep 10, 2024
1 parent 7f4e8b9 commit da9cfc5
Show file tree
Hide file tree
Showing 7 changed files with 31 additions and 1 deletion.
26 changes: 25 additions & 1 deletion config/v1/types_cluster_version.go
Original file line number Diff line number Diff line change
Expand Up @@ -428,7 +428,7 @@ var KnownClusterVersionCapabilities = []ClusterVersionCapability{
}

// ClusterVersionCapabilitySet defines sets of cluster version capabilities.
// +kubebuilder:validation:Enum=None;v4.11;v4.12;v4.13;v4.14;v4.15;v4.16;vCurrent
// +kubebuilder:validation:Enum=None;v4.11;v4.12;v4.13;v4.14;v4.15;v4.16;v4.17;vCurrent
type ClusterVersionCapabilitySet string

const (
Expand Down Expand Up @@ -472,6 +472,12 @@ const (
// version of OpenShift is installed.
ClusterVersionCapabilitySet4_16 ClusterVersionCapabilitySet = "v4.16"

// ClusterVersionCapabilitySet4_17 is the recommended set of
// optional capabilities to enable for the 4.17 version of
// OpenShift. This list will remain the same no matter which
// version of OpenShift is installed.
ClusterVersionCapabilitySet4_17 ClusterVersionCapabilitySet = "v4.17"

// ClusterVersionCapabilitySetCurrent is the recommended set
// of optional capabilities to enable for the cluster's
// current version of OpenShift.
Expand Down Expand Up @@ -556,6 +562,24 @@ var ClusterVersionCapabilitySets = map[ClusterVersionCapabilitySet][]ClusterVers
ClusterVersionCapabilityIngress,
ClusterVersionCapabilityCloudControllerManager,
},
ClusterVersionCapabilitySet4_17: {
ClusterVersionCapabilityBaremetal,
ClusterVersionCapabilityConsole,
ClusterVersionCapabilityInsights,
ClusterVersionCapabilityMarketplace,
ClusterVersionCapabilityStorage,
ClusterVersionCapabilityOpenShiftSamples,
ClusterVersionCapabilityCSISnapshot,
ClusterVersionCapabilityNodeTuning,
ClusterVersionCapabilityMachineAPI,
ClusterVersionCapabilityBuild,
ClusterVersionCapabilityDeploymentConfig,
ClusterVersionCapabilityImageRegistry,
ClusterVersionCapabilityOperatorLifecycleManager,
ClusterVersionCapabilityCloudCredential,
ClusterVersionCapabilityIngress,
ClusterVersionCapabilityCloudControllerManager,
},
ClusterVersionCapabilitySetCurrent: {
ClusterVersionCapabilityBaremetal,
ClusterVersionCapabilityConsole,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ spec:
- v4.14
- v4.15
- v4.16
- v4.17
- vCurrent
type: string
type: object
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ spec:
- v4.14
- v4.15
- v4.16
- v4.17
- vCurrent
type: string
type: object
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ spec:
- v4.14
- v4.15
- v4.16
- v4.17
- vCurrent
type: string
type: object
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ spec:
- v4.14
- v4.15
- v4.16
- v4.17
- vCurrent
type: string
type: object
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,7 @@ spec:
- v4.14
- v4.15
- v4.16
- v4.17
- vCurrent
type: string
type: object
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,7 @@ spec:
- v4.14
- v4.15
- v4.16
- v4.17
- vCurrent
type: string
type: object
Expand Down

0 comments on commit da9cfc5

Please sign in to comment.