matrix job to check all the api specs

eclipse-edc · Jun 20, 2024 · 62ae5d6 · 62ae5d6
1 parent 886f894
commit 62ae5d6
Show file tree

Hide file tree

Showing 2 changed files with 51 additions and 17 deletions.
diff --git a/.github/workflows/verify-management-api.yml b/.github/workflows/verify-management-api.yml
@@ -4,29 +4,63 @@ on:
   pull_request:
 
 jobs:
-  verify-version-change:
+  prepare-api-groups:
     runs-on: ubuntu-latest
+    outputs:
+      api_groups: ${{ steps.outputStep.outputs.matrix }}
     steps:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
       - uses: eclipse-edc/.github/.github/actions/setup-build@main
+      - run: ./gradlew resolve
+      - uses: actions/upload-artifact@v4
+        with:
+          name: openapi-spec
+          path: resources/openapi/yaml
+      - name: get api groups to verify and create matrix for next job
+        id: outputStep
+        run: |
+          if test -d resources/openapi/yaml; then
+            API_GROUPS_ARRAY=$(ls -l resources/openapi/*.version | awk '{print $9}' | awk -F "[/\.]" '{print $3}' | sed ':a; N; $!ba; s/\n/","/g' | sed 's/.*/["&"]/')
+            echo "matrix={\"apiGroup\": ${API_GROUPS_ARRAY}}" >> $GITHUB_OUTPUT
+          fi
+
+  verify-spec:
+    if: ${{ needs.prepare-api-groups.outputs.api_groups != '' }}
+    needs: prepare-api-groups
+    runs-on: ubuntu-latest
+    strategy:
+      matrix: ${{ fromJson(needs.prepare-api-groups.outputs.api_groups) }}
+    env:
+      versionFile: resources/openapi/${{ matrix.apiGroup }}.version
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+      - uses: eclipse-edc/.github/.github/actions/setup-build@main
+      - uses: actions/download-artifact@v4
+        with:
+          name: openapi-spec
+          path: resources/openapi/yaml
       - run: |
           CHANGELOG=$(git diff-tree --name-only HEAD..origin/main -r)
-          VERSION_FILE=$(cat resources/openapi/management-api.version)
-          if ! (echo $CHANGELOG | grep -q "$VERSION_FILE") ; then 
-            ./gradlew resolve -q
+          if (echo $CHANGELOG | grep -q "${{ env.versionFile }}") ; then
+            echo "Version file for context ${{ matrix.apiGroup }} has been modified, no need to check differences"
+             echo "skip=true" >> $GITHUB_OUTPUT
+          fi
+
+      - if: ( steps.verify-spec.outputs.skip != 'true' )
+        run: |
+          VERSION=$(cat ${{ env.versionFile }} | xargs cat | grep version | awk -F '"' '{print $4}')
+          ./gradlew -Pversion=$VERSION -PapiTitle="${{ matrix.apiGroup }}" -PapiDescription="REST API documentation for the ${{ matrix.apiGroup }}" :mergeApiSpec --input=resources/openapi/yaml/${{ matrix.apiGroup }} --output=${{ matrix.apiGroup }}-new.yaml
+          
+          git show origin/gh-pages:openapi/${{ matrix.apiGroup }}/${{ matrix.apiGroup }}.yaml > ${{ matrix.apiGroup }}.yaml
+          
+          diff ${{ matrix.apiGroup }}.yaml ${{ matrix.apiGroup }}-new.yaml > diff
           
-            VERSION=$(cat $VERSION_FILE | grep version | awk -F '"' '{print $4}')
-            ./gradlew -Pversion=$VERSION -PapiTitle="management-api" -PapiDescription="REST API documentation for the management-api" :mergeApiSpec --input=resources/openapi/yaml/management-api --output=management-api-new.yaml
-            
-            git show origin/gh-pages:openapi/management-api/management-api.yaml > management-api.yaml
-            
-            diff management-api.yaml management-api-new.yaml > diff
-            
-            if [ -s diff ]; then
-                echo "management-api openapi spec diverges from the published one, please update the version.json file"
-                cat diff
-                exit 1
-            fi
+          if [ -s diff ]; then
+              echo "${{ matrix.apiGroup }} openapi spec diverges from the published one, please update the version.json file"
+              cat diff
+              exit 1
           fi
diff --git a/DEPENDENCIES b/DEPENDENCIES
@@ -47,7 +47,7 @@ maven/mavencentral/com.github.docker-java/docker-java-transport-zerodep/3.3.6, A
 maven/mavencentral/com.github.docker-java/docker-java-transport/3.3.6, Apache-2.0, approved, #7942
 maven/mavencentral/com.github.java-json-tools/btf/1.3, Apache-2.0 AND GPL-1.0-or-later AND LGPL-3.0-only AND Apache-2.0 AND LGPL-3.0-only, restricted, #15201
 maven/mavencentral/com.github.java-json-tools/jackson-coreutils-equivalence/1.0, LGPL-3.0 OR Apache-2.0, approved, clearlydefined
-maven/mavencentral/com.github.java-json-tools/jackson-coreutils/2.0, Apache-2.0 AND LGPL-2.1-or-later AND LGPL-3.0-only AND (Apache-2.0 AND GPL-1.0-or-later AND LGPL-3.0-only) AND Apache-2.0 AND LGPL-3.0-only, restricted, #15186
+maven/mavencentral/com.github.java-json-tools/jackson-coreutils/2.0, Apache-2.0 OR LGPL-3.0-or-later, approved, #15186
 maven/mavencentral/com.github.java-json-tools/json-patch/1.13, Apache-2.0 OR LGPL-3.0-or-later, approved, CQ23929
 maven/mavencentral/com.github.java-json-tools/json-schema-core/1.2.14, Apache-2.0 AND LGPL-2.1-or-later AND LGPL-3.0-only AND (Apache-2.0 AND GPL-1.0-or-later AND LGPL-3.0-only) AND Apache-2.0 AND LGPL-3.0-only, restricted, #15282
 maven/mavencentral/com.github.java-json-tools/json-schema-validator/2.2.14, Apache-2.0 OR LGPL-3.0-or-later, approved, CQ20779