fix: helm tests, seem like they need more time in this env

Author: lockwobr

.github/workflows/operator-ci.yaml

@@ -54,8 +54,21 @@ env:
 
 # There is a single job in this workflow. It's configured to run on the latest available version of Ubuntu.
 jobs:
+  unit-test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Setup Go 1.23
+        uses: actions/setup-go@v5
+        with:
+          go-version: 1.23
+      - name: Unit tests
+        run: |
+          cd operator
+          make unit-tests
   k8s-tests:
     runs-on: ubuntu-latest
+    needs: [unit-test] # Don't run the k8s tests if the unit tests fail
     steps:
       - uses: actions/checkout@v4
         with:
@@ -81,7 +94,7 @@ jobs:
         run: |
           cd operator
           GITHUB_TOKEN=${{ secrets.github_token }} make create-kind-cluster
-          make test
+          make e2e-tests
   build-and-push-operator:
     runs-on: ubuntu-latest
     needs: [k8s-tests] # Don't run the build and push if the k8s tests fail
@@ -112,6 +125,8 @@ jobs:
 
       - name: Build the operator container image
         id: build
+        env:
+          platforms: ${{ env.PLATFORMS }}
         run: |
           apt-get update && apt-get install -y make git jq
           cd operator

k8s-tests/chainsaw/helm/helm-chart-test/chainsaw-test.yaml

@@ -18,24 +18,18 @@
 # LICENSE END
 # 
 
-
-
-
-
-
-
 # yaml-language-server: $schema=https://raw.githubusercontent.com/kyverno/chainsaw/main/.schemas/json/test-chainsaw-v1alpha1.json
 apiVersion: chainsaw.kyverno.io/v1alpha1
 kind: Test
 metadata:
   name: helm-chart
 spec:
-  description: This test asserts that the helm chart is working as expected. Specifcally it asserts that the helm chart works when given a different
+  description: This test asserts that the helm chart is working as expected. Specifically it asserts that the helm chart works when given a different
     deployment name than skyhook-operator and that the tolerations that are given to the chart through a values file work as expected.
   concurrent: false
   timeouts:
-    assert: 120s
-    exec: 120s
+    assert: 240s
+    exec: 240s
   steps:
   - try:
     - script:

k8s-tests/chainsaw/helm/helm-scale-test/chainsaw-test.yaml

@@ -18,12 +18,6 @@
 # LICENSE END
 # 
 
-
-
-
-
-
-
 # yaml-language-server: $schema=https://raw.githubusercontent.com/kyverno/chainsaw/main/.schemas/json/test-chainsaw-v1alpha1.json
 apiVersion: chainsaw.kyverno.io/v1alpha1
 kind: Test
@@ -34,8 +28,8 @@ spec:
     deployment name than skyhook-operator and that the tolerations that are given to the chart through a values file work as expected.
   concurrent: false
   timeouts:
-    assert: 120s
-    exec: 120s
+    assert: 240s
+    exec: 240s
   steps:
   - try:
     - script:

operator/Makefile

@@ -223,7 +223,7 @@ merge-coverage:
 	echo "mode: set" > $(REPORTING)/cover.out
 	## skip first line with +2
 	tail -n +2 $(REPORTING)/temp-cover.out | sed  '/mode: set/d' >> $(REPORTING)/cover.out
-	$(sedrp) 's/^\/.*(\/skyhook\/operator.*)/github\.com\/NVIDIA\1/g' $(REPORTING)/cover.out ## TODO: need to test more
+	$(sedrp) 's|^/.*skyhook/operator/(.*)$$|github\.com/NVIDIA/skyhook/\1|g' $(REPORTING)/cover.out
 
 .PHONY: lint
 lint: golangci-lint license-check ## Run golangci-lint linter & yamllint