Fix roles / missing privileges

Makefile

@@ -234,7 +234,7 @@ release-chart:
 	cd ../helm-charts/; git add -A charts/packages; git fix; git push;
 	cd ../helm-charts/charts/${CHART_NAME}; cr upload --config ../../chart-releaser.yaml --skip-existing;
 	cd ../helm-charts/charts/${CHART_NAME}; rm -fr .cr-index; mkdir .cr-index; cr index --config ../../chart-releaser.yaml; cp .cr-index/index.yaml ../../index.yaml;
-	git fix; git push
+	../helm-charts; git fix; git push
 
 # go-install-tool will 'go install' any package with custom target and name of binary, if it doesn't exist
 # $1 - target path with name of binary

chart/Chart.yaml

@@ -10,9 +10,9 @@ description: |
 
 type: application
 
-version: 0.1.7
+version: 0.1.11
 
-appVersion: "0.1.7"
+appVersion: "0.1.11"
 
 home: https://github.com/lukaszraczylo/kubernetes-images-sync-operator
 

chart/templates/mr-raczylo-com-rbac.yaml

@@ -5,6 +5,18 @@ metadata:
   labels:
   {{- include "chart.labels" . | nindent 4 }}
 rules:
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - create
+  - delete
+  - get
+  - list
+  - patch
+  - update
+  - watch
 - apiGroups:
   - apps
   resources:
@@ -37,8 +49,7 @@ rules:
 - apiGroups:
   - raczylo.com
   resources:
-  - clusterimageexports
-  - clusterimages
+  - '*'
   verbs:
   - create
   - delete
@@ -50,13 +61,13 @@ rules:
 - apiGroups:
   - raczylo.com
   resources:
-  - clusterimageexports/finalizers
+  - '*/finalizers'
   verbs:
   - update
 - apiGroups:
   - raczylo.com
   resources:
-  - clusterimageexports/status
+  - '*/status'
   verbs:
   - get
   - patch

chart/templates/raczylo.com-clusterimage-editor-rbac.yaml

@@ -22,4 +22,7 @@ rules:
   resources:
   - clusterimages/status
   verbs:
-  - get
+  - get
+  - patch
+  - update
+  - watch

chart/templates/raczylo.com-clusterimage-viewer-rbac.yaml

@@ -18,4 +18,5 @@ rules:
   resources:
   - clusterimages/status
   verbs:
-  - get
+  - get
+  - watch

chart/templates/raczylo.com-clusterimageexport-editor-rbac.yaml

@@ -22,4 +22,7 @@ rules:
   resources:
   - clusterimageexports/status
   verbs:
-  - get
+  - get
+  - patch
+  - update
+  - watch

chart/templates/raczylo.com-clusterimageexport-viewer-rbac.yaml

@@ -9,13 +9,8 @@ rules:
   - raczylo.com
   resources:
   - clusterimageexports
+  - clusterimageexports/status
   verbs:
   - get
   - list
-  - watch
-- apiGroups:
-  - raczylo.com
-  resources:
-  - clusterimageexports/status
-  verbs:
-  - get
+  - watch

chart/values.yaml

@@ -11,7 +11,7 @@ cmRaczyloCom:
         - ALL
     image:
       repository: ghcr.io/lukaszraczylo/kubernetes-images-sync-operator
-      tag: 0.1.7
+      tag: 0.1.11
     resources:
       limits:
         cpu: 500m

config/rbac/raczylo.com_clusterimage_editor_role.yaml

@@ -25,3 +25,6 @@ rules:
       - clusterimages/status
     verbs:
       - get
+      - patch
+      - update
+      - watch

config/rbac/raczylo.com_clusterimage_viewer_role.yaml

@@ -21,3 +21,4 @@ rules:
       - clusterimages/status
     verbs:
       - get
+      - watch

config/rbac/raczylo.com_clusterimageexport_editor_role.yaml

@@ -25,3 +25,6 @@ rules:
       - clusterimageexports/status
     verbs:
       - get
+      - patch
+      - update
+      - watch

config/rbac/raczylo.com_clusterimageexport_viewer_role.yaml

@@ -11,13 +11,8 @@ rules:
       - raczylo.com
     resources:
       - clusterimageexports
+      - clusterimageexports/status
     verbs:
       - get
       - list
       - watch
-  - apiGroups:
-      - raczylo.com
-    resources:
-      - clusterimageexports/status
-    verbs:
-      - get

config/rbac/role.yaml

@@ -4,6 +4,18 @@ kind: ClusterRole
 metadata:
   name: mr-raczylo-com
 rules:
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - create
+  - delete
+  - get
+  - list
+  - patch
+  - update
+  - watch
 - apiGroups:
   - apps
   resources:
@@ -36,8 +48,7 @@ rules:
 - apiGroups:
   - raczylo.com
   resources:
-  - clusterimageexports
-  - clusterimages
+  - '*'
   verbs:
   - create
   - delete
@@ -49,13 +60,13 @@ rules:
 - apiGroups:
   - raczylo.com
   resources:
-  - clusterimageexports/finalizers
+  - '*/finalizers'
   verbs:
   - update
 - apiGroups:
   - raczylo.com
   resources:
-  - clusterimageexports/status
+  - '*/status'
   verbs:
   - get
   - patch

internal/controller/raczylo.com/clusterimage_controller.go

@@ -29,12 +29,11 @@ type ClusterImageReconciler struct {
 	ActiveJobs      int
 }
 
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimages,verbs=get;list;watch;create;update;patch;delete
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimages/status,verbs=get;update;patch
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimages/finalizers,verbs=update
+// +kubebuilder:rbac:groups=raczylo.com,resources=*,verbs=get;list;watch;create;update;patch;delete
+// +kubebuilder:rbac:groups=raczylo.com,resources=*/status,verbs=get;update;patch
+// +kubebuilder:rbac:groups=raczylo.com,resources=*/finalizers,verbs=update
 // # additional RBAC rules - create and manage jobs
 // +kubebuilder:rbac:groups=batch,resources=jobs,verbs=get;list;watch;create;update;patch;delete
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimageexports,verbs=get;list;watch;update;patch
 func (r *ClusterImageReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
 	l := log.FromContext(ctx)
 

internal/controller/raczylo.com/clusterimageexport_controller.go

@@ -30,15 +30,15 @@ type ClusterImageExportReconciler struct {
 	Scheme *runtime.Scheme
 }
 
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimageexports,verbs=get;list;watch;create;update;patch;delete
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimageexports/status,verbs=get;update;patch
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimageexports/finalizers,verbs=update
+// +kubebuilder:rbac:groups=raczylo.com,resources=*,verbs=get;list;watch;create;update;patch;delete
+// +kubebuilder:rbac:groups=raczylo.com,resources=*/status,verbs=get;update;patch
+// +kubebuilder:rbac:groups=raczylo.com,resources=*/finalizers,verbs=update
 // additional RBAC rules
-// +kubebuilder:rbac:groups=raczylo.com,resources=clusterimages,verbs=get;list;watch;create;update;patch;delete
 // +kubebuilder:rbac:groups=apps,resources=deployments,verbs=get;list;watch
 // +kubebuilder:rbac:groups=apps,resources=daemonsets,verbs=get;list;watch
 // +kubebuilder:rbac:groups=batch,resources=jobs,verbs=get;list;watch;create;update;patch;delete
 // +kubebuilder:rbac:groups=batch,resources=cronjobs,verbs=get;list;watch
+// +kubebuilder:rbac:groups="",resources=pods,verbs=get;list;watch;create;update;patch;delete
 
 const clusterImageExportFinalizer = "finalizer.clusterimageexport.raczylo.com"