Skip to content

Commit 0f86a22

Browse files
Merge pull request #1262 from deads2k/apply-03-ingress-route-status
update ingress component route SSA list markers
2 parents 613c415 + a9dae0f commit 0f86a22

File tree

4 files changed

+62
-3
lines changed

4 files changed

+62
-3
lines changed

config/v1/0000_10_config-operator_01_ingress.crd.yaml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,10 @@ spec:
7474
name:
7575
description: name is the metadata.name of the referenced secret
7676
type: string
77+
x-kubernetes-list-map-keys:
78+
- namespace
79+
- name
80+
x-kubernetes-list-type: map
7781
domain:
7882
description: "domain is used to generate a default host name for a route when the route's host name is empty. The generated host name will follow this pattern: \"<route-name>.<route-namespace>.<domain>\". \n It is also used as the default wildcard domain suffix for ingress. The default ingresscontroller domain will follow this pattern: \"*.<domain>\". \n Once set, changing domain is not currently supported."
7983
type: string
@@ -211,6 +215,9 @@ spec:
211215
type: string
212216
maxLength: 316
213217
pattern: ^([a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*/)?(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])$
218+
x-kubernetes-list-map-keys:
219+
- type
220+
x-kubernetes-list-type: map
214221
consumingUsers:
215222
description: consumingUsers is a slice of ServiceAccounts that need to have read permission on the servingCertKeyPairSecret secret.
216223
type: array
@@ -268,6 +275,10 @@ spec:
268275
resource:
269276
description: resource of the referent.
270277
type: string
278+
x-kubernetes-list-map-keys:
279+
- namespace
280+
- name
281+
x-kubernetes-list-type: map
271282
defaultPlacement:
272283
description: "defaultPlacement is set at installation time to control which nodes will host the ingress router pods by default. The options are control-plane nodes or worker nodes. \n This field works by dictating how the Cluster Ingress Operator will consider unset replicas and nodePlacement fields in IngressController resources when creating the corresponding Deployments. \n See the documentation for the IngressController replicas and nodePlacement fields for more information. \n When omitted, the default value is Workers"
273284
type: string

config/v1/types_ingress.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,9 @@ type IngressSpec struct {
5656
// .status.componentRoutes list, where participating operators write the status of
5757
// configurable routes.
5858
// +optional
59+
// +listType=map
60+
// +listMapKey=namespace
61+
// +listMapKey=name
5962
ComponentRoutes []ComponentRouteSpec `json:"componentRoutes,omitempty"`
6063

6164
// requiredHSTSPolicies specifies HSTS policies that are required to be set on newly created or updated routes
@@ -111,6 +114,9 @@ type IngressStatus struct {
111114
// componentRoutes is where participating operators place the current route status for routes whose
112115
// hostnames and serving certificates can be customized by the cluster-admin.
113116
// +optional
117+
// +listType=map
118+
// +listMapKey=namespace
119+
// +listMapKey=name
114120
ComponentRoutes []ComponentRouteStatus `json:"componentRoutes,omitempty"`
115121

116122
// defaultPlacement is set at installation time to control which
@@ -221,6 +227,8 @@ type ComponentRouteStatus struct {
221227
//
222228
// If Progressing is true, that means the component is taking some action related to the componentRoutes entry.
223229
// +optional
230+
// +listType=map
231+
// +listMapKey=type
224232
Conditions []metav1.Condition `json:"conditions,omitempty"`
225233

226234
// relatedObjects is a list of resources which are useful when debugging or inspecting how spec.componentRoutes is applied.

openapi/generated_openapi/zz_generated.openapi.go

Lines changed: 26 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

openapi/openapi.json

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5262,7 +5262,11 @@
52625262
"items": {
52635263
"default": {},
52645264
"$ref": "#/definitions/io.k8s.apimachinery.pkg.apis.meta.v1.Condition"
5265-
}
5265+
},
5266+
"x-kubernetes-list-map-keys": [
5267+
"type"
5268+
],
5269+
"x-kubernetes-list-type": "map"
52665270
},
52675271
"consumingUsers": {
52685272
"description": "consumingUsers is a slice of ServiceAccounts that need to have read permission on the servingCertKeyPairSecret secret.",
@@ -7022,7 +7026,12 @@
70227026
"items": {
70237027
"default": {},
70247028
"$ref": "#/definitions/com.github.openshift.api.config.v1.ComponentRouteSpec"
7025-
}
7029+
},
7030+
"x-kubernetes-list-map-keys": [
7031+
"namespace",
7032+
"name"
7033+
],
7034+
"x-kubernetes-list-type": "map"
70267035
},
70277036
"domain": {
70287037
"description": "domain is used to generate a default host name for a route when the route's host name is empty. The generated host name will follow this pattern: \"<route-name>.<route-namespace>.<domain>\".\n\nIt is also used as the default wildcard domain suffix for ingress. The default ingresscontroller domain will follow this pattern: \"*.<domain>\".\n\nOnce set, changing domain is not currently supported.",
@@ -7048,7 +7057,12 @@
70487057
"items": {
70497058
"default": {},
70507059
"$ref": "#/definitions/com.github.openshift.api.config.v1.ComponentRouteStatus"
7051-
}
7060+
},
7061+
"x-kubernetes-list-map-keys": [
7062+
"namespace",
7063+
"name"
7064+
],
7065+
"x-kubernetes-list-type": "map"
70527066
},
70537067
"defaultPlacement": {
70547068
"description": "defaultPlacement is set at installation time to control which nodes will host the ingress router pods by default. The options are control-plane nodes or worker nodes.\n\nThis field works by dictating how the Cluster Ingress Operator will consider unset replicas and nodePlacement fields in IngressController resources when creating the corresponding Deployments.\n\nSee the documentation for the IngressController replicas and nodePlacement fields for more information.\n\nWhen omitted, the default value is Workers",

0 commit comments

Comments
 (0)