Skip to content

Commit

Permalink
[jaeger-operator] Add ingressclasses to role.yaml so that the CRD has…
Browse files Browse the repository at this point in the history
… correct permissions when rbac.cluster.Role is set (#591)

* Update role.yaml to solve issue 581

Signed-off-by: shivendoodeshmukh <[email protected]>

* Bump Chart Version

Signed-off-by: shivendoodeshmukh <[email protected]>

---------

Signed-off-by: shivendoodeshmukh <[email protected]>
  • Loading branch information
shivendoodeshmukh authored Jul 30, 2024
1 parent de6fa20 commit 64baeb5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/jaeger-operator/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
description: jaeger-operator Helm chart for Kubernetes
name: jaeger-operator
version: 2.54.0
version: 2.55.0
appVersion: 1.57.0
home: https://www.jaegertracing.io/
icon: https://www.jaegertracing.io/img/jaeger-icon-reverse-color.svg
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger-operator/templates/role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,7 @@ rules:
- extensions
resources:
- ingresses
- ingressclasses
verbs:
- create
- delete
Expand Down

1 comment on commit 64baeb5

@surajnmandhare
Copy link

@surajnmandhare surajnmandhare commented on 64baeb5 Aug 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shivendoodeshmukh - Please could you also update ingressclasses permission for role on apigroup "networking.k8s.io"

  • apiGroups:
    • networking.k8s.io
      resources:
    • ingresses
    • ingressclasses
      verbs:
    • create
    • delete
    • get
    • list
    • patch
    • update
    • watch

Please sign in to comment.