Skip to content

Commit

Permalink
MON-3287: Remove openshift-etcd related RBAC as they will be manager by
Browse files Browse the repository at this point in the history
cluster-etcd-operator now.

The resources may keep some of CMO related metadata, this is the trade-off to keep the
same names and avoid unnecessary API calls in CMO.
  • Loading branch information
machine424 committed Nov 23, 2023
1 parent 4bdf151 commit 9caffd0
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 84 deletions.
19 changes: 0 additions & 19 deletions assets/prometheus-k8s/role-binding-specific-namespaces.yaml
Expand Up @@ -57,25 +57,6 @@ items:
- kind: ServiceAccount
name: prometheus-k8s
namespace: openshift-monitoring
- apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
labels:
app.kubernetes.io/component: prometheus
app.kubernetes.io/instance: k8s
app.kubernetes.io/name: prometheus
app.kubernetes.io/part-of: openshift-monitoring
app.kubernetes.io/version: 2.48.0
name: prometheus-k8s
namespace: openshift-etcd
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: prometheus-k8s
subjects:
- kind: ServiceAccount
name: prometheus-k8s
namespace: openshift-monitoring
- apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
Expand Down
38 changes: 0 additions & 38 deletions assets/prometheus-k8s/role-specific-namespaces.yaml
Expand Up @@ -114,44 +114,6 @@ items:
- get
- list
- watch
- apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
labels:
app.kubernetes.io/component: prometheus
app.kubernetes.io/instance: k8s
app.kubernetes.io/name: prometheus
app.kubernetes.io/part-of: openshift-monitoring
app.kubernetes.io/version: 2.48.0
name: prometheus-k8s
namespace: openshift-etcd
rules:
- apiGroups:
- ""
resources:
- services
- endpoints
- pods
verbs:
- get
- list
- watch
- apiGroups:
- extensions
resources:
- ingresses
verbs:
- get
- list
- watch
- apiGroups:
- networking.k8s.io
resources:
- ingresses
verbs:
- get
- list
- watch
- apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
Expand Down
1 change: 0 additions & 1 deletion jsonnet/main.jsonnet
Expand Up @@ -206,7 +206,6 @@ local inCluster =
name: 'k8s',
alertmanagerName: $.values.alertmanager.name,
namespaces+: [
'openshift-etcd',
$.values.common.namespaceUserWorkload,
],
namespaceSelector: $.values.common.clusterMonitoringNamespaceSelector,
Expand Down
26 changes: 0 additions & 26 deletions manifests/0000_50_cluster-monitoring-operator_02-role.yaml
Expand Up @@ -566,32 +566,6 @@ rules:
- get
- list
- watch
- apiGroups:
- ""
resources:
- services
- endpoints
- pods
verbs:
- get
- list
- watch
- apiGroups:
- extensions
resources:
- ingresses
verbs:
- get
- list
- watch
- apiGroups:
- networking.k8s.io
resources:
- ingresses
verbs:
- get
- list
- watch
- apiGroups:
- ""
resources:
Expand Down

0 comments on commit 9caffd0

Please sign in to comment.