Skip to content

Commit

Permalink
NETOBSERV-972 check if cluster admin via namespaces
Browse files Browse the repository at this point in the history
Follow-up on netobserv#320, which relaxed the permission checks performed when
lokiAuth is DISABLED: after discussion, we roll back to a more strict
approach; however to mitigate the limitation of TokenReview (it doesn't
provide a reliable way to check for cluster admins right), we verify
that the user can list namespaces, assuming this is a cluster admin
capability.
  • Loading branch information
jotak committed May 9, 2023
1 parent e2b2a89 commit 175f3b3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion config/manager/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,6 @@ kind: Kustomization
images:
- name: controller
newName: quay.io/netobserv/network-observability-operator
newTag: 1.0.2
newTag: main
commonLabels:
app: netobserv-operator
6 changes: 3 additions & 3 deletions config/manager/manager.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,11 @@ spec:
- --downstream-deployment=$(DOWNSTREAM_DEPLOYMENT)
env:
- name: RELATED_IMAGE_EBPF_AGENT
value: quay.io/netobserv/netobserv-ebpf-agent:v0.3.0
value: quay.io/netobserv/netobserv-ebpf-agent:main
- name: RELATED_IMAGE_FLOWLOGS_PIPELINE
value: quay.io/netobserv/flowlogs-pipeline:v0.1.8
value: quay.io/netobserv/flowlogs-pipeline:main
- name: RELATED_IMAGE_CONSOLE_PLUGIN
value: quay.io/netobserv/network-observability-console-plugin:v0.1.9
value: quay.io/netobserv/network-observability-console-plugin:main
- name: DOWNSTREAM_DEPLOYMENT
value: "false"
image: controller:latest
Expand Down

0 comments on commit 175f3b3

Please sign in to comment.