Group CI tests by version (#1410)

* ci(test) split workflows for legacy and railgun

Split test workflows into common, legacy/1.x-specific, and
railgun/2.x-specific tests. Common tests are linting, code generation
verification, manifest generation, and unit tests. Specific tests are
integration tests.

Rename workflows and jobs for clarity.

Use consistent conditions for tests and adjust to run when appropriate:

- All tests run on branch pushes.
- All tests run on PRs to main and next.
- Common and 1.x tests run on 1.x tags.
- Common and 2.x tests run on 2.x tags.

* ci(test) rename workflow files

* Apply suggestions from code review

Co-authored-by: Michał Flendrich <michal@flendrich.pro>

Co-authored-by: Michał Flendrich <michal@flendrich.pro>

.github/workflows/build-test.yaml → .github/workflows/1x-integration-test.yaml

@@ -1,8 +1,38 @@
-name: Test
+name: 1.x Integration Tests
 
-on: [push]
+on:
+  pull_request:
+    branches:
+      - 'main'
+      - 'next'
+  push:
+    branches:
+      - '*'
+    tags:
+      - '1.*'
+      - 'v1.*'
 
 jobs:
+  # Golang Integration tests, using the legacy manager/1.x CLI
+  integration-test:
+    runs-on: ubuntu-latest
+    steps:
+    - name: setup golang
+      uses: actions/setup-go@v2
+      with:
+        go-version: '^1.16'
+    - name: cache go modules
+      uses: actions/cache@v2.1.6
+      with:
+        path: ~/go/pkg/mod
+        key: ${{ runner.os }}-build-codegen-${{ hashFiles('**/go.sum') }}
+        restore-keys: |
+          ${{ runner.os }}-go
+    - name: checkout repository
+      uses: actions/checkout@v2
+    - name: run railgun integration tests on legacy KIC
+      run: make test.integration.legacy
+      working-directory: ./railgun
   build:
     runs-on: ubuntu-latest
     steps:
@@ -32,8 +62,8 @@ jobs:
       with:
         name: image
         path: image.tar
-
-  test-k8s:
+  # Bash-based integration tests
+  integration-test-legacy:
     needs: build
     runs-on: ubuntu-latest
     env:

.github/workflows/integration_test.yaml → .github/workflows/2x-integration-test.yaml

@@ -1,4 +1,4 @@
-name: Integration Tests
+name: 2.x Integration Tests
 
 on:
   pull_request:
@@ -7,10 +7,10 @@ on:
       - 'next'
   push:
     branches:
-      - 'main'
-      - 'next'
+      - '*'
     tags:
-      - 'v*.*.*'
+      - '2.*'
+      - 'v2.*'
 
 jobs:
   integration-test-dbless:
@@ -51,22 +51,3 @@ jobs:
     - name: run railgun integration tests
       run: make test.integration.postgres
       working-directory: ./railgun
-  integration-test-legacy:
-    runs-on: ubuntu-latest
-    steps:
-    - name: setup golang
-      uses: actions/setup-go@v2
-      with:
-        go-version: '^1.16'
-    - name: cache go modules
-      uses: actions/cache@v2.1.6
-      with:
-        path: ~/go/pkg/mod
-        key: ${{ runner.os }}-build-codegen-${{ hashFiles('**/go.sum') }}
-        restore-keys: |
-          ${{ runner.os }}-go
-    - name: checkout repository
-      uses: actions/checkout@v2
-    - name: run railgun integration tests on legacy KIC
-      run: make test.integration.legacy
-      working-directory: ./railgun

.github/workflows/test.yaml → .github/workflows/common-test.yaml

@@ -1,9 +1,19 @@
-name: Test
+# Linting, unit tests, and generated resource verification
+name: Common Tests
 
-on: [push, pull_request]
+on:
+  pull_request:
+    branches:
+      - 'main'
+      - 'next'
+  push:
+    branches:
+      - '*'
+    tags:
+      - '*'
 
 jobs:
-  lint:
+  lint-and-verify-generated:
     runs-on: ubuntu-latest
     steps:
     - name: Setup go
@@ -34,7 +44,7 @@ jobs:
     - name: Verify manifest consistency
       run: |
         make verify-manifests
-  test:
+  unit-test:
     runs-on: ubuntu-latest
     steps:
     - name: Setup go