fix: change label selector for kubectl wait in post install job (#240)

* fix: change label selector for kubectl wait in post install job Signed-off-by: Vishal Choudhary <[email protected]> * fix: chainsaw tests Signed-off-by: Vishal Choudhary <[email protected]> * fix: tests Signed-off-by: Vishal Choudhary <[email protected]> * fix: regex Signed-off-by: Vishal Choudhary <[email protected]> --------- Signed-off-by: Vishal Choudhary <[email protected]>
kyverno · Dec 18, 2024 · 1096068 · 1096068
1 parent d589c42
commit 1096068
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 10 deletions.
diff --git a/.github/workflows/conformance-tests.yaml b/.github/workflows/conformance-tests.yaml
@@ -84,9 +84,8 @@ jobs:
           set -e
           set -e
           export HELM=${{ steps.helm.outputs.helm-path }}
-          helm repo add kyverno https://kyverno.github.io/kyverno/
-          kubectl create namespace kyverno
-          helm install kyverno --namespace kyverno kyverno/kyverno --set features.policyExceptions.enabled=true --set features.policyExceptions.namespace='*'
+          curl https://raw.githubusercontent.com/kyverno/kyverno/refs/heads/main/scripts/config/standard/kyverno.yaml > values.yaml
+          helm install kyverno -n kyverno --create-namespace ./kyverno/charts/kyverno --wait --values values.yaml
       - name: Wait for kyverno ready
         run: |
           set -e
@@ -102,7 +101,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           set -e
-          cd ./kyverno/test/conformance/chainsaw && chainsaw test --config ./../../../../.chainsaw.yaml --include-test-regex '^chainsaw$/${{ matrix.tests }}' --no-color=false
+          cd ./kyverno/test/conformance/chainsaw/reports && chainsaw test --config ./../../../../../.chainsaw.yaml --exclude-test-regex '^chainsaw$/^admission$/^label'
       - name: Debug failure
         if: failure()
         uses: ./.github/actions/failure-logs
diff --git a/.github/workflows/migration-tests.yaml b/.github/workflows/migration-tests.yaml
@@ -74,9 +74,8 @@ jobs:
         run: |
           set -e
           export HELM=${{ steps.helm.outputs.helm-path }}
-          helm repo add kyverno https://kyverno.github.io/kyverno/
-          kubectl create namespace kyverno
-          helm install kyverno --namespace kyverno kyverno/kyverno --set features.policyExceptions.enabled=true --set features.policyExceptions.namespace='*'
+          curl https://raw.githubusercontent.com/kyverno/kyverno/refs/heads/main/scripts/config/standard/kyverno.yaml > values.yaml
+          helm install kyverno -n kyverno --create-namespace ./kyverno/charts/kyverno --wait --values values.yaml
       - name: Wait for kyverno ready
         run: |
           set -e
@@ -116,7 +115,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           set -e
-          cd ./kyverno/test/conformance/chainsaw && chainsaw test --config ./../../../../.chainsaw.yaml --include-test-regex '^chainsaw$/${{ matrix.tests }}' --no-color=false
+          cd ./kyverno/test/conformance/chainsaw/reports && chainsaw test --config ./../../../../../.chainsaw.yaml --exclude-test-regex '^chainsaw$/^admission$/^label'
       - name: Debug failure
         if: failure()
         uses: ./.github/actions/failure-logs
diff --git a/charts/reports-server/templates/hooks/post-install-api-services.yaml b/charts/reports-server/templates/hooks/post-install-api-services.yaml
@@ -42,7 +42,7 @@ spec:
             - '-c'
             - |-
               set -euo pipefail
-              kubectl wait -n {{ $.Release.Namespace }} pod --for=condition=ready -l app.kubernetes.io/name={{ include "reports-server.fullname" . }} --timeout=120s
+              kubectl wait -n {{ $.Release.Namespace }} pod --for=condition=ready -l app.kubernetes.io/name={{ include "reports-server.name" . }} --timeout=120s
               kubectl apply -f - <<EOF
               {
                 "apiVersion": "apiregistration.k8s.io/v1",

diff --git a/charts/reports-server/templates/hooks/pre-delete-api-service-cleanup.yaml b/charts/reports-server/templates/hooks/pre-delete-api-service-cleanup.yaml
@@ -43,7 +43,7 @@ spec:
             - '-c'
             - |-
               set -euo pipefail
-              kubectl wait -n {{ $.Release.Namespace }} pod --for=condition=ready -l app.kubernetes.io/name={{ include "reports-server.fullname" . }} --timeout=120s
+              kubectl wait -n {{ $.Release.Namespace }} pod --for=condition=ready -l app.kubernetes.io/name={{ include "reports-server.name" . }} --timeout=120s
               kubectl delete apiservice v1alpha2.wgpolicyk8s.io v1.reports.kyverno.io --ignore-not-found=true
           {{- with .Values.apiServicesManagement.securityContext }}
           securityContext: