Add matrix.jdk to unit tests(main) and remove all-jdk step

We will likely need to parallelize the runs

Author: abhishekrb19

.github/workflows/ci.yml

@@ -28,9 +28,9 @@ jobs:
     uses: ./.github/workflows/worker.yml
     with:
       script: .github/scripts/run_unit-tests -Dtest=!QTest,'${{ matrix.pattern }}' -Dmaven.test.failure.ignore=true
-      jdk: 17
       artifact_prefix: "unit-test-reports"
-      key: "test-jdk17-[${{ matrix.pattern }}]"
+      jdk: [ "17", "21" ]
+      key: "test-jdk${{ matrix.jdk }}-[${{ matrix.pattern }}]"
 
   validate-dist:
     uses: ./.github/workflows/worker.yml
@@ -71,21 +71,6 @@ jobs:
       artifacts_to_download: "unit-test-reports-*"
       key: "coverage-jacoco"
 
-  run-unit-tests-all-jdk:
-    needs: test-report
-    name: "unit tests(all jdk)"
-    if: ${{ !cancelled() && ( contains( github.event.pull_request.labels.*.name, 'tests:all-jdk') || github.event_name == 'push' ) }}
-    strategy:
-      fail-fast: false
-      matrix:
-        jdk: [ "21" ]
-        pattern: [ "C*", "H*,U*,V*", "N*,Q*,S*", "B*,O*,R*", "G*,J*,K*", "F*,L*,M*", "A*,D*,I*,X*,Y*,Z*", "E*,P*,T*,W*"]
-    uses: ./.github/workflows/worker.yml
-    with:
-      script: .github/scripts/run_unit-tests -Dtest=!QTest,'${{ matrix.pattern }}' -fae
-      jdk: ${{ matrix.jdk }}
-      key: "test-jdk${{ matrix.jdk }}-[${{ matrix.pattern }}]"
-
   # this will be running in parallel with the ITs later; but until that migration happens - run them in parallel with normal tests
   run-qtest:
     strategy: