Revert CI changes, install pnpm via maven plugin

Author: devabhishekpal

.github/workflows/check.yml

@@ -45,12 +45,6 @@ on:
         default: ''
         required: false
 
-      pnpm-version:
-        type: string
-        description: "pnpm version to use for builds (default: none)"
-        default: ''
-        required: false
-
       needs-maven-cache:
         type: boolean
         description: "Whether to restore Maven cache before run (default: yes)"
@@ -173,7 +167,6 @@ jobs:
         with:
           path: |
             ~/.pnpm-store
-            **/node_modules
           key: ${{ runner.os }}-pnpm-${{ hashFiles('**/pnpm-lock.yaml') }}
           restore-keys: |
             ${{ runner.os }}-pnpm-
@@ -226,12 +219,6 @@ jobs:
           distribution: 'temurin'
           java-version: ${{ inputs.java-version }}
 
-      - name: Setup pnpm version ${{ inputs.pnpm-version }}
-        if: ${{ inputs.pnpm-version }}
-        uses: pnpm/action-setup@v4
-        with:
-          version: ${{ inputs.pnpm-version }}
-
       - name: Execute pre-test steps
         if: ${{ inputs.pre-script }}
         run: |

.github/workflows/ci.yml

@@ -30,7 +30,6 @@ env:
   BUILD_ARGS: "-Pdist -Psrc -Dmaven.javadoc.skip=true -Drocks_tools_native"
   # Minimum required Java version for running Ozone is defined in pom.xml (javac.version).
   TEST_JAVA_VERSION: 21 # JDK version used by CI build and tests; should match the JDK version in apache/ozone-runner image
-  PNPM_VERSION: 8.15.7 # This is defined in the pom.xml file with pnpm.version, should be updated accordingly
   # MAVEN_ARGS and MAVEN_OPTS are duplicated in check.yml, please keep in sync
   MAVEN_ARGS: --batch-mode --settings ${{ github.workspace }}/dev-support/ci/maven-settings.xml
   MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3
@@ -55,7 +54,6 @@ jobs:
       # `env` context cannot be used when calling reusable workflow, so we need to convert these to `outputs`
       build-args: ${{ env.BUILD_ARGS }}
       java-version: ${{ env.TEST_JAVA_VERSION }}
-      pnpm-version: ${{ env.PNPM_VERSION }}
       with-coverage: ${{ env.OZONE_WITH_COVERAGE }}
     steps:
       - name: "Checkout ${{ github.ref }} / ${{ github.sha }} (push)"
@@ -116,7 +114,6 @@ jobs:
     if: needs.build-info.outputs.needs-build == 'true' || needs.build-info.outputs.needs-integration-tests == 'true'
     uses: ./.github/workflows/check.yml
     with:
-      pnpm-version: ${{ needs.build-info.outputs.pnpm-version }}
       java-version: ${{ needs.build-info.outputs.java-version }}
       needs-npm-cache: true
       ratis-args: ${{ inputs.ratis_args }}

hadoop-ozone/recon/pom.xml

@@ -415,16 +415,15 @@
           <npmInheritsProxyConfigFromMaven>false</npmInheritsProxyConfigFromMaven>
           <installDirectory>target</installDirectory>
           <workingDirectory>${basedir}/src/main/resources/webapps/recon/ozone-recon-web</workingDirectory>
+          <nodeVersion>v${nodejs.version}</nodeVersion>
+          <pnpmVersion>${pnpm.version}</pnpmVersion>
         </configuration>
         <executions>
           <execution>
             <id>Install node and npm locally to the project</id>
             <goals>
-              <goal>install-node-and-npm</goal>
+              <goal>install-node-and-pnpm</goal>
             </goals>
-            <configuration>
-              <nodeVersion>v${nodejs.version}</nodeVersion>
-            </configuration>
           </execution>
           <execution>
             <id>set pnpm@${pnpm.version} store path</id>