Skip to content

Commit

Permalink
fix: Apply workaround for jaeger rbac permissions
Browse files Browse the repository at this point in the history
Apply workaround for the jaeger ClusterRole, that seems to be missing
permissions to list ingressclasses. This makes the jaeger operator not
reconcile the expected ingress service. See
jaegertracing/helm-charts#549

Signed-off-by: Víctor Cuadrado Juan <[email protected]>
  • Loading branch information
viccuad committed Jul 29, 2024
1 parent d198b83 commit f5dcf72
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 0 deletions.
11 changes: 11 additions & 0 deletions resources/opentelemetry-jaeger-rbac-workaround.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
# This is a workaround for the jaeger ClusterRole, that seems to be missing permissions to list ingressclasses.
# This makes the jaeger operator not reconcile the expected ingress service.
# See https://github.com/jaegertracing/helm-charts/issues/549
rules:
- apiGroups:
- networking.k8s.io
resources:
- ingressclasses
verbs:
- get
- list
4 changes: 4 additions & 0 deletions tests/opentelemetry-tests.bats
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,10 @@ setup() {
helm upgrade -i --wait jaeger-operator jaegertracing/jaeger-operator \
-n jaeger --create-namespace \
--set rbac.clusterRole=true

# workaround for https://github.com/jaegertracing/helm-charts/issues/549
kubectl patch clusterrole jaeger-operator --patch "$(cat $RESOURCES_DIR/opentelemetry-jaeger-rbac-workaround.yaml)"

kubectl apply -f $RESOURCES_DIR/opentelemetry-jaeger.yaml
wait_pods -n jaeger

Expand Down

0 comments on commit f5dcf72

Please sign in to comment.