Add status check jobs for local_e2e_tests matrix and linters (#5628)

Due to GH limitations, it is not possible to add a branch protection
rule pointing to all the jobs generated by GHA's matrix strategy. As a
workaround, we've introduced a new status check job that depends on the
job with the matrix, and whose status depends on the success of all the
jobs generated by that. In addition, we've taken advantage to add
another check-status job for all the linter jobs.

Signed-off-by: Jesús Miguel Benito Calzada <bjesus@vmware.com>

Author: Jesús Miguel Benito Calzada

.github/workflows/kubeapps-general.yaml

@@ -179,6 +179,15 @@ jobs:
     with:
       golang_version: ${{ needs.setup.outputs.golang_version }}
 
+  linters_result:
+    if: inputs.run_linters && always()
+    needs: linters
+    runs-on: ubuntu-latest
+    steps:
+      - name: "Check all linters result"
+        if: needs.linters.result != 'success'
+        run: exit 1
+
   test_go:
     needs:
       - setup
@@ -533,6 +542,20 @@ jobs:
           name: e2e_tests_${{ matrix.tests_group }}_reports
           path: integration/reports
 
+  # We need this job to aggregate the result of the local_e2e_tests matrix and summarize it. We don't need it because of
+  # GHA itself, but because in GH branch protection rules, we cannot check the status for a job matrix, and would have
+  # require the status for each job in the matrix (eg. local_e2e_tests(carvel)).
+  # See https://github.com/orgs/community/discussions/26822
+  local_e2e_tests_result:
+    needs:
+      - local_e2e_tests
+    runs-on: ubuntu-latest
+    if: always()
+    steps:
+      - name: "Check local_e2e_tests matrix status"
+        if: needs.local_e2e_tests.result != 'success'
+        run: exit 1
+
   push_images:
     if: needs.setup.outputs.running_on_main == 'true' || needs.setup.outputs.running_on_tag == 'true'
     runs-on: ubuntu-latest