Skip to content

Commit 3379c2c

Browse files
committed
install/0000_00_cluster-version-operator_02_networkpolicy: Add inclusion annotations
Somehow we'd missed these in 07c6b5c (OTA-1521: Add a default-deny network policy for CVO namespace, 2025-05-27, #1198), so the policy had not been getting applied to any clusters [1]. While I'm at it, I've also shifted the YAML comments into a kubernetes.io/description annotation, as we do for most of our manifests, so the purpose and implementation notes are available to curious cluster-admins the same as they're available to CVO devs. [1]: https://issues.redhat.com/browse/OCPBUGS-77762
1 parent 54dc4ef commit 3379c2c

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

install/0000_00_cluster-version-operator_02_networkpolicy.yaml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,16 @@
11
apiVersion: networking.k8s.io/v1
22
kind: NetworkPolicy
33
metadata:
4-
# This NetworkPolicy is used to deny all ingress and egress traffic by default in this namespace,
5-
# serving as a baseline. At the moment no other Network Policy should be needed:
6-
# - CVO is a host-networked Pod, so it is not affected by network policies
7-
# - Bare `version` Pods spawned by CVO do not require any network communication
84
name: default-deny
95
namespace: openshift-cluster-version
6+
annotations:
7+
kubernetes.io/description: |
8+
This NetworkPolicy is used to deny all ingress and egress traffic by default in this namespace, matching all Pods, and serving as a baseline. At the moment no other Network Policy should be needed:
9+
- The cluster-version operator (CVO) is a host-networked Pod, so it is not affected by network policies.
10+
- Bare `version-...` Pods spawned by CVO do not require any network communication.
11+
exclude.release.openshift.io/internal-openshift-hosted: "true"
12+
include.release.openshift.io/self-managed-high-availability: "true"
1013
spec:
11-
# Match all pods in the namespace
1214
podSelector: {}
1315
policyTypes:
1416
- Ingress

0 commit comments

Comments
 (0)