Fix release workflows

- Check for dirty repository in release
- Fix helm release workflow name
- Rename dist dir to manifests as dist is used by goreleaser

Author: mohamed-essam

.github/workflows/docker.yml

@@ -39,6 +39,11 @@ jobs:
         with:
           fetch-depth: 0 # It is required for GoReleaser to work properly
 
+      - name: Install modules
+        run: go mod tidy
+
+      - name: check git status
+        run: git --no-pager diff --exit-code
       # Set up BuildKit Docker container builder to be able to build
       # multi-platform images and export cache
       # https://github.com/docker/setup-buildx-action

.github/workflows/helm.yml

@@ -1,4 +1,4 @@
-name: Release Helm Chart and Carvel package
+name: Release Helm Chart
 
 on:
   push:

Makefile

@@ -125,8 +125,8 @@ docker-buildx: ## Build and push docker image for the manager for cross-platform
 
 .PHONY: build-installer
 build-installer: manifests ## Generate a consolidated YAML with CRDs and deployment.
-	mkdir -p dist
-	$(HELM) template --include-crds netbird-operator helm/netbird-operator > dist/install.yaml
+	mkdir -p manifests
+	$(HELM) template --include-crds netbird-operator helm/netbird-operator > manifests/install.yaml
 
 ##@ Deployment
 

README.md

@@ -19,7 +19,7 @@ This operator enables easily provisioning NetBird access on kubernetes clusters,
 
 ```sh
 kubectl create namespace netbird
-kubectl apply -n netbird -f https://github.com/netbirdio/kubernetes-operator/releases/latest/dist/install.yaml
+kubectl apply -n netbird -f https://github.com/netbirdio/kubernetes-operator/releases/latest/manifests/install.yaml
 ```
 
 **Using the Helm Chart**
@@ -38,7 +38,7 @@ helm show values netbirdio/netbird-operator
 **Using install.yaml**
 
 ```sh
-kubectl delete -n netbird -f https://github.com/netbirdio/kubernetes-operator/releases/latest/dist/install.yaml
+kubectl delete -n netbird -f https://github.com/netbirdio/kubernetes-operator/releases/latest/manifests/install.yaml
 kubectl delete namespace netbird
 ```
 

dist/install.yaml renamed to manifests/install.yaml

@@ -75,8 +75,41 @@ spec:
           status:
             description: NBSetupKeyStatus defines the observed state of NBSetupKey.
             properties:
-              ready:
-                type: boolean
+              conditions:
+                items:
+                  description: NBSetupKeyCondition defines a condition in NBSetupKey
+                    status.
+                  properties:
+                    lastProbeTime:
+                      description: Last time we probed the condition.
+                      format: date-time
+                      type: string
+                    lastTransitionTime:
+                      description: Last time the condition transitioned from one status
+                        to another.
+                      format: date-time
+                      type: string
+                    message:
+                      description: Human-readable message indicating details about
+                        last transition.
+                      type: string
+                    reason:
+                      description: Unique, one-word, CamelCase reason for the condition's
+                        last transition.
+                      type: string
+                    status:
+                      description: |-
+                        Status is the status of the condition.
+                        Can be True, False, Unknown.
+                      type: string
+                    type:
+                      description: Type is the type of the condition.
+                      type: string
+                  required:
+                  - status
+                  - type
+                  type: object
+                type: array
             type: object
         type: object
     served: true
@@ -132,6 +165,14 @@ rules:
   - get
   - patch
   - update
+- apiGroups:
+  - ""
+  resources:
+  - pods
+  verbs:
+  - get
+  - list
+  - watch
 - apiGroups:
   - ""
   resources: