Skip to content

Commit cdacd52

Browse files
Network Segmentation: Sync up rbacs for helm
Sync up changes performed in the following PRs in order to make user defined networks work when deploying cluster using helm. - ovn-kubernetes/ovn-kubernetes#4486 - ovn-kubernetes/ovn-kubernetes#4612 Signed-off-by: Flavio Fernandes <[email protected]>
1 parent 71481d1 commit cdacd52

File tree

8 files changed

+28
-1
lines changed

8 files changed

+28
-1
lines changed

helm/ovn-kubernetes/charts/ovnkube-control-plane/templates/ovnkube-control-plane.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,8 @@ spec:
124124
value: {{ default "" .Values.global.enableEgressQos | quote }}
125125
- name: OVN_MULTI_NETWORK_ENABLE
126126
value: {{ hasKey .Values.global "enableMultiNetwork" | ternary .Values.global.enableMultiNetwork false | quote }}
127+
- name: OVN_NETWORK_SEGMENTATION_ENABLE
128+
value: {{ default "" .Values.global.enableNetworkSegmentation | quote }}
127129
- name: OVN_HYBRID_OVERLAY_NET_CIDR
128130
value: {{ default "" .Values.global.hybridOverlayNetCidr | quote }}
129131
- name: OVN_DISABLE_SNAT_MULTIPLE_GWS

helm/ovn-kubernetes/charts/ovnkube-control-plane/templates/rbac-ovnkube-cluster-manager.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,13 +74,17 @@ rules:
7474
- egressfirewalls
7575
- egressqoses
7676
- userdefinednetworks
77+
- clusteruserdefinednetworks
7778
verbs: [ "get", "list", "watch" ]
7879
- apiGroups: ["k8s.ovn.org"]
7980
resources:
8081
- egressips
8182
- egressservices/status
8283
- userdefinednetworks
8384
- userdefinednetworks/status
85+
- clusteruserdefinednetworks
86+
- clusteruserdefinednetworks/status
87+
- clusteruserdefinednetworks/finalizers
8488
verbs: [ "patch", "update" ]
8589
- apiGroups: [""]
8690
resources:

helm/ovn-kubernetes/charts/ovnkube-master/templates/deployment-ovnkube-master.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -234,6 +234,8 @@ spec:
234234
value: {{ default "" .Values.global.enableEgressQos | quote }}
235235
- name: OVN_MULTI_NETWORK_ENABLE
236236
value: {{ hasKey .Values.global "enableMultiNetwork" | ternary .Values.global.enableMultiNetwork false | quote }}
237+
- name: OVN_NETWORK_SEGMENTATION_ENABLE
238+
value: {{ default "" .Values.global.enableNetworkSegmentation | quote }}
237239
- name: OVN_EGRESSSERVICE_ENABLE
238240
value: {{ default "" .Values.global.enableEgressService | quote }}
239241
- name: OVN_HYBRID_OVERLAY_NET_CIDR

helm/ovn-kubernetes/charts/ovnkube-master/templates/rbac-ovnkube-master.yaml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,8 @@ rules:
8383
- egressqoses
8484
- egressservices
8585
- adminpolicybasedexternalroutes
86+
- userdefinednetworks
87+
- clusteruserdefinednetworks
8688
verbs: [ "get", "list", "watch" ]
8789
- apiGroups: ["k8s.cni.cncf.io"]
8890
resources:
@@ -92,7 +94,11 @@ rules:
9294
- apiGroups: ["k8s.cni.cncf.io"]
9395
resources:
9496
- network-attachment-definitions
95-
verbs: ["patch"]
97+
verbs: [ "patch", "update" ]
98+
- apiGroups: [ "k8s.cni.cncf.io" ]
99+
resources:
100+
- network-attachment-definitions
101+
verbs: [ "create", "delete" ]
96102
- apiGroups: ["policy.networking.k8s.io"]
97103
resources:
98104
- adminnetworkpolicies/status
@@ -106,6 +112,11 @@ rules:
106112
- egressservices/status
107113
- adminpolicybasedexternalroutes/status
108114
- egressqoses/status
115+
- userdefinednetworks
116+
- userdefinednetworks/status
117+
- clusteruserdefinednetworks
118+
- clusteruserdefinednetworks/status
119+
- clusteruserdefinednetworks/finalizers
109120
verbs: [ "patch", "update" ]
110121
- apiGroups: [""]
111122
resources:

helm/ovn-kubernetes/charts/ovnkube-node/templates/ovnkube-node.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,8 @@ spec:
227227
value: {{ default "" .Values.global.lFlowCacheLimitKb | quote }}
228228
- name: OVN_MULTI_NETWORK_ENABLE
229229
value: {{ hasKey .Values.global "enableMultiNetwork" | ternary .Values.global.enableMultiNetwork false | quote }}
230+
- name: OVN_NETWORK_SEGMENTATION_ENABLE
231+
value: {{ default "" .Values.global.enableNetworkSegmentation | quote }}
230232
- name: OVN_ENABLE_INTERCONNECT
231233
value: {{ hasKey .Values.global "enableInterconnect" | ternary .Values.global.enableInterconnect false | quote }}
232234
- name: OVN_ENABLE_MULTI_EXTERNAL_GATEWAY

helm/ovn-kubernetes/charts/ovnkube-single-node-zone/templates/ovnkube-single-node-zone.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -412,6 +412,8 @@ spec:
412412
value: {{ default "" .Values.global.lFlowCacheLimitKb | quote }}
413413
- name: OVN_MULTI_NETWORK_ENABLE
414414
value: {{ hasKey .Values.global "enableMultiNetwork" | ternary .Values.global.enableMultiNetwork false | quote }}
415+
- name: OVN_NETWORK_SEGMENTATION_ENABLE
416+
value: {{ default "" .Values.global.enableNetworkSegmentation | quote }}
415417
- name: OVNKUBE_NODE_MGMT_PORT_NETDEV
416418
value: {{ default "" .Values.global.nodeMgmtPortNetdev | quote }}
417419
- name: OVN_EMPTY_LB_EVENTS

helm/ovn-kubernetes/charts/ovnkube-zone-controller/templates/ovnkube-zone-controller.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -311,6 +311,8 @@ spec:
311311
value: {{ default "" .Values.global.enableEgressQos | quote }}
312312
- name: OVN_MULTI_NETWORK_ENABLE
313313
value: {{ hasKey .Values.global "enableMultiNetwork" | ternary .Values.global.enableMultiNetwork false | quote }}
314+
- name: OVN_NETWORK_SEGMENTATION_ENABLE
315+
value: {{ default "" .Values.global.enableNetworkSegmentation | quote }}
314316
- name: OVN_HYBRID_OVERLAY_NET_CIDR
315317
value: {{ default "" .Values.global.hybridOverlayNetCidr | quote }}
316318
- name: OVN_DISABLE_SNAT_MULTIPLE_GWS

helm/ovn-kubernetes/templates/rbac-ovnkube-node.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,8 @@ rules:
181181
- egressqoses
182182
- egressservices
183183
- adminpolicybasedexternalroutes
184+
- userdefinednetworks
185+
- clusteruserdefinednetworks
184186
verbs: [ "get", "list", "watch" ]
185187
{{- if eq (hasKey .Values.global "enableOvnKubeIdentity" | ternary .Values.global.enableOvnKubeIdentity true) true }}
186188
- apiGroups: ["certificates.k8s.io"]

0 commit comments

Comments
 (0)