From f2f94421628147b87027768b53a3cc83540ef0ee Mon Sep 17 00:00:00 2001
From: AtomicFS <vojtech.vesely@9elements.com>
Date: Thu, 17 Oct 2024 14:57:31 +0200
Subject: [PATCH] revert(ci): remove edk2 matrix

---
 .github/workflows/docker-build-and-test.yml | 23 +++++++--------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/.github/workflows/docker-build-and-test.yml b/.github/workflows/docker-build-and-test.yml
index e1791f4c..b10d32bc 100644
--- a/.github/workflows/docker-build-and-test.yml
+++ b/.github/workflows/docker-build-and-test.yml
@@ -67,21 +67,14 @@ jobs:
     outputs:
       matrix: ${{ steps.get-matrix.outputs.matrix }}
 
-  get-matrix-edk2:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-      - name: get-matrix
-        id: get-matrix
-        run: |
-          # shellcheck disable=SC2046
-          echo matrix=$( yq --input-format yaml --output-format json '.services | keys[] | select(. | test("(e|u)dk2.*"))' docker/compose.yaml | sed 's/"//g' | jq -Rs 'split("\n") | del(.[-1])' | jq -c ) >> "${GITHUB_OUTPUT}"
-      - name: Check
-        run: |
-          jq . <<< '${{ steps.get-matrix.outputs.matrix }}'
-    outputs:
-      matrix: ${{ steps.get-matrix.outputs.matrix }}
+  # get-matrix-edk2:
+  #
+  # So ... I did try to also pre-compile edk2 BaseTools but there were simply too many problems
+  # - udk2017 uses Ubuntu Bionic (18.04) which is simply too old for GitHub actions, which require modern NodeJS
+  # - uploading almost 1GB of artifacts per edk2 variant is just too much
+  #
+  # Since the BaseTools do not take that long to compile, I will just make to so that the compilation happens
+  #   on the first use
 
   #=====================
   # Coreboot toolchains