Skip to content

Commit 402eb85

Browse files
committed
WIP: remove control-plane label
Signed-off-by: Todd Short <[email protected]>
1 parent b72115c commit 402eb85

17 files changed

+55
-85
lines changed

docs/draft/api-reference/network-policies.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@ NetworkPolicy is implemented for both catalogd and operator-controller component
1919

2020
Each component has a dedicated NetworkPolicy that applies to its respective pod through label selectors:
2121

22-
* For catalogd: `control-plane=catalogd-controller-manager`
23-
* For operator-controller: `control-plane=operator-controller-controller-manager`
22+
* For catalogd: `app.kubernetes.io/name=catalogd`
23+
* For operator-controller: `app.kubernetes.io/name=operator-controller`
2424

2525
### Catalogd NetworkPolicy
2626

@@ -78,10 +78,10 @@ If you encounter network connectivity issues after deploying OLMv1, consider the
7878

7979
```bash
8080
# Verify catalogd pod labels
81-
kubectl get pods -n olmv1-system --selector=control-plane=catalogd-controller-manager
81+
kubectl get pods -n olmv1-system --selector=apps.kubernetes.io/name=catalogd
8282

8383
# Verify operator-controller pod labels
84-
kubectl get pods -n olmv1-system --selector=control-plane=operator-controller-controller-manager
84+
kubectl get pods -n olmv1-system --selector=apps.kubernetes.io/name=operator-controller
8585

8686
# Compare with actual pod names
8787
kubectl get pods -n olmv1-system | grep -E 'catalogd|operator-controller'

docs/draft/howto/consuming-metrics.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ apiVersion: monitoring.coreos.com/v1
226226
kind: ServiceMonitor
227227
metadata:
228228
labels:
229-
control-plane: operator-controller-controller-manager
229+
apps.kubernetes.io/name: operator-controller
230230
name: controller-manager-metrics-monitor
231231
namespace: olmv1-system
232232
spec:
@@ -251,7 +251,7 @@ spec:
251251
key: tls.key
252252
selector:
253253
matchLabels:
254-
control-plane: operator-controller-controller-manager
254+
apps.kubernetes.io/name: operator-controller
255255
EOF
256256
```
257257

@@ -268,7 +268,7 @@ apiVersion: monitoring.coreos.com/v1
268268
kind: ServiceMonitor
269269
metadata:
270270
labels:
271-
control-plane: catalogd-controller-manager
271+
apps.kubernetes.io/name: catalogd
272272
name: catalogd-metrics-monitor
273273
namespace: olmv1-system
274274
spec:
@@ -298,4 +298,4 @@ EOF
298298
```
299299

300300
[prometheus-operator]: https://github.com/prometheus-operator/kube-prometheus
301-
[rbac-k8s-docs]: https://kubernetes.io/docs/reference/access-authn-authz/rbac/
301+
[rbac-k8s-docs]: https://kubernetes.io/docs/reference/access-authn-authz/rbac/

docs/draft/howto/enable-helm-chart-support.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ To enable the Helm Chart support feature gate, you need to patch the `operator-c
2424
2. **Wait for the controller manager pods to be ready:**
2525

2626
```bash
27-
$ kubectl -n olmv1-system wait --for condition=ready pods -l control-plane=operator-controller-controller-manager
27+
$ kubectl -n olmv1-system wait --for condition=ready pods -l apps.kubernetes.io/name=operator-controller
2828
```
2929

3030
Once the above wait condition is met, the `HelmChartSupport` feature gate should be enabled in operator controller.

docs/draft/howto/profiling_with_pprof.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ The following steps are examples to demonstrate the required changes to enable P
2121
1. Run the following command to patch the Deployment and add the `--pprof-bind-address=:8082` flag:
2222

2323
```shell
24-
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l control-plane=operator-controller-controller-manager -o jsonpath='{.items[0].metadata.name}') \
24+
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l apps.kubernetes.io/name=operator-controller -o jsonpath='{.items[0].metadata.name}') \
2525
-n olmv1-system --type='json' -p='[
2626
{
2727
"op": "add",
@@ -127,7 +127,7 @@ go tool pprof -http=:8080 ./operator-controller-profile.pprof
127127
1. Run the following command to patch the Deployment and add the `--pprof-bind-address=:8083` flag:
128128

129129
```shell
130-
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l control-plane=catalogd-controller-manager -o jsonpath='{.items[0].metadata.name}') \
130+
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l apps.kubernetes.io/name=catalogd -o jsonpath='{.items[0].metadata.name}') \
131131
-n olmv1-system --type='json' -p='[
132132
{
133133
"op": "add",
@@ -235,7 +235,7 @@ go tool pprof -http=:8080 ./catalogd-profile.pprof
235235
1. Run the following command to bind to `--pprof-bind-address` the value `0` in order to disable the endpoint.
236236

237237
```shell
238-
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l control-plane=operator-controller-controller-manager -o jsonpath='{.items[0].metadata.name}') \
238+
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l apps.kubernetes.io/name=operator-controller -o jsonpath='{.items[0].metadata.name}') \
239239
-n olmv1-system --type='json' -p='[
240240
{
241241
"op": "replace",
@@ -266,7 +266,7 @@ kubectl delete pod curl-oper-con-pprof -n olmv1-system
266266

267267
1. Run the following command to bind to `--pprof-bind-address` the value `0` in order to disable the endpoint.
268268
```shell
269-
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l control-plane=catalogd-controller-manager -o jsonpath='{.items[0].metadata.name}') \
269+
kubectl patch deployment $(kubectl get deployments -n olmv1-system -l apps.kubernetes.io/name=catalogd -o jsonpath='{.items[0].metadata.name}') \
270270
-n olmv1-system --type='json' -p='[
271271
{
272272
"op": "replace",
@@ -294,4 +294,4 @@ re-start the deployment `kubectl rollout restart deployment -n olmv1-system cata
294294
kubectl delete pod curl-catalogd-pprof -n olmv1-system
295295
```
296296

297-
[pprof]: https://github.com/google/pprof/blob/main/doc/README.md
297+
[pprof]: https://github.com/google/pprof/blob/main/doc/README.md

helm/olmv1/templates/26-service-olmv1-system-catalogd-service.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ metadata:
88
{{- toYaml . | nindent 4 }}
99
{{- end }}
1010
labels:
11-
control-plane: catalogd-controller-manager
1211
app.kubernetes.io/name: catalogd
1312
{{- include "olmv1.labels" . | nindent 4 }}
1413
name: catalogd-service
@@ -28,5 +27,5 @@ spec:
2827
protocol: TCP
2928
targetPort: 7443
3029
selector:
31-
control-plane: catalogd-controller-manager
30+
app.kubernetes.io/name: catalogd
3231
{{- end }}

helm/olmv1/templates/27-service-olmv1-system-operator-controller-service.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ metadata:
88
{{- toYaml . | nindent 4 }}
99
{{- end }}
1010
labels:
11-
control-plane: operator-controller-controller-manager
1211
app.kubernetes.io/name: operator-controller
1312
{{- include "olmv1.labels" . | nindent 4 }}
1413
name: operator-controller-service
@@ -20,5 +19,5 @@ spec:
2019
protocol: TCP
2120
targetPort: 8443
2221
selector:
23-
control-plane: operator-controller-controller-manager
22+
app.kubernetes.io/name: operator-controller
2423
{{- end }}

helm/olmv1/templates/29-deployment-olmv1-system-catalogd-controller-manager.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ metadata:
66
kubectl.kubernetes.io/default-logs-container: manager
77
{{- include "olmv1.annotations" . | nindent 4 }}
88
labels:
9-
control-plane: catalogd-controller-manager
109
app.kubernetes.io/name: catalogd
1110
{{- include "olmv1.labels" . | nindent 4 }}
1211
name: catalogd-controller-manager
@@ -16,7 +15,7 @@ spec:
1615
replicas: 1
1716
selector:
1817
matchLabels:
19-
control-plane: catalogd-controller-manager
18+
app.kubernetes.io/name: catalogd
2019
template:
2120
metadata:
2221
annotations:
@@ -26,7 +25,6 @@ spec:
2625
{{- toYamlPretty . | nindent 8 }}
2726
{{- end }}
2827
labels:
29-
control-plane: catalogd-controller-manager
3028
app.kubernetes.io/name: catalogd
3129
{{- include "olmv1.labels" . | nindent 8 }}
3230
{{- with .Values.components.catalogd.deployment.podLabels }}

helm/olmv1/templates/30-deployment-olmv1-system-operator-controller-controller-manager.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ metadata:
66
kubectl.kubernetes.io/default-logs-container: manager
77
{{- include "olmv1.annotations" . | nindent 4 }}
88
labels:
9-
control-plane: operator-controller-controller-manager
109
app.kubernetes.io/name: operator-controller
1110
{{- include "olmv1.labels" . | nindent 4 }}
1211
name: operator-controller-controller-manager
@@ -15,7 +14,7 @@ spec:
1514
replicas: 1
1615
selector:
1716
matchLabels:
18-
control-plane: operator-controller-controller-manager
17+
app.kubernetes.io/name: operator-controller
1918
template:
2019
metadata:
2120
annotations:
@@ -25,7 +24,6 @@ spec:
2524
{{- toYamlPretty . | nindent 8 }}
2625
{{- end }}
2726
labels:
28-
control-plane: operator-controller-controller-manager
2927
app.kubernetes.io/name: operator-controller
3028
{{- include "olmv1.labels" . | nindent 8 }}
3129
{{- with .Values.components.operatorController.deployment.podLabels }}

helm/olmv1/templates/36-networkpolicy-olmv1-system-catalogd-controller-manager.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ spec:
2222
protocol: TCP
2323
podSelector:
2424
matchLabels:
25-
control-plane: catalogd-controller-manager
25+
app.kubernetes.io/name: catalogd
2626
policyTypes:
2727
- Ingress
2828
- Egress

helm/olmv1/templates/38-networkpolicy-olmv1-system-operator-controller-controller-manager.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ spec:
1818
protocol: TCP
1919
podSelector:
2020
matchLabels:
21-
control-plane: operator-controller-controller-manager
21+
app.kubernetes.io/name: operator-controller
2222
policyTypes:
2323
- Ingress
2424
- Egress

0 commit comments

Comments
 (0)