Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ingress): remove secrets from clusterrole #361

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Conversation

jfroy
Copy link
Owner

@jfroy jfroy commented Sep 4, 2024

No description provided.

@jfroy jfroy force-pushed the nginx-rbac-no-secrets branch 2 times, most recently from 4c9a58a to df311a8 Compare September 4, 2024 17:12
Repository owner deleted a comment from github-actions bot Sep 4, 2024
Copy link

github-actions bot commented Sep 4, 2024

--- kubernetes/apps/network/ingress-nginx/internal Kustomization: flux-system/ingress-nginx-internal HelmRelease: network/ingress-nginx-internal

+++ kubernetes/apps/network/ingress-nginx/internal Kustomization: flux-system/ingress-nginx-internal HelmRelease: network/ingress-nginx-internal

@@ -20,12 +20,21 @@

   driftDetection:
     mode: enabled
   install:
     remediation:
       retries: 3
   interval: 30m
+  postRenderers:
+  - kustomize:
+      patches:
+      - patch: |
+          - op: remove
+            path: /rules/0/resources/4
+        target:
+          kind: ClusterRole
+          name: ingress-nginx-internal
   upgrade:
     cleanupOnFail: true
     remediation:
       retries: 3
   values:
     controller:
--- kubernetes/apps/network/ingress-nginx/external Kustomization: flux-system/ingress-nginx-external HelmRelease: network/ingress-nginx-external

+++ kubernetes/apps/network/ingress-nginx/external Kustomization: flux-system/ingress-nginx-external HelmRelease: network/ingress-nginx-external

@@ -23,12 +23,21 @@

   driftDetection:
     mode: enabled
   install:
     remediation:
       retries: 3
   interval: 30m
+  postRenderers:
+  - kustomize:
+      patches:
+      - patch: |
+          - op: remove
+            path: /rules/0/resources/4
+        target:
+          kind: ClusterRole
+          name: ingress-nginx-external
   upgrade:
     cleanupOnFail: true
     remediation:
       retries: 3
   values:
     controller:

@jfroy jfroy merged commit 9f52fa9 into main Sep 4, 2024
4 checks passed
@jfroy jfroy deleted the nginx-rbac-no-secrets branch September 4, 2024 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant