This is an automated email from the ASF dual-hosted git repository.

gyeongtae pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/master by this push:
     new 0f92acc64a [ZEPPELIN-6292] Fix e2e test build dependency order by 
including zeppelin-web-angular
0f92acc64a is described below

commit 0f92acc64a4165ec9ad3962aa8e8790c82b60993
Author: 강현욱 <43662405+hyu...@users.noreply.github.com>
AuthorDate: Tue Aug 26 20:45:29 2025 +0900

    [ZEPPELIN-6292] Fix e2e test build dependency order by including 
zeppelin-web-angular
    
    ### What is this PR for?
    The `run-e2e-tests-in-zeppelin-web` GitHub Actions workflow is currently 
failing with a `FileNotFoundException` error.
    This failure affects the build stability and prevents proper CI validation 
of frontend changes.
    
    The current CI workflow has a build dependency ordering issue:
      1. Current problematic workflow step:
         ```
         ./mvnw verify -pl zeppelin-web -Pweb-classic -Pspark-scala-2.12 
-Pspark-3.4 -Pweb-dist -Pweb-e2e
    
      2. Issue: The workflow only builds zeppelin-web module but skips 
zeppelin-web-angular
      3. Impact: zeppelin-web requires built artifacts from 
zeppelin-web-angular/dist/ directory
      4. Result: Zeppelin server startup fails because it cannot find the 
frontend build output
    
    ### What type of PR is it?
    Bug Fix
    
    ### Todos
    * [x] Add `zeppelin-web-angular` to Maven project list to ensure proper 
build dependency order
    * [x] Resolve `FileNotFoundException` for 
`/zeppelin-web-angular/dist/zeppelin` directory
    
    ### What is the Jira issue?
    * [[ZEPPELIN-6292]](https://issues.apache.org/jira/browse/ZEPPELIN-6292)
    
    ### How should this be tested?
    * github actions CI already including e2e tests
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the license files need to update? - No
    * Is there breaking changes for older versions? - No
    * Does this needs documentation? - No
    
    
    Closes #5043 from hyunw9/ZEPPELIN-6292.
    
    Signed-off-by: ParkGyeongTae <gyeong...@apache.org>
---
 .github/workflows/frontend.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/frontend.yml b/.github/workflows/frontend.yml
index ea8fdbd521..1d9990b889 100644
--- a/.github/workflows/frontend.yml
+++ b/.github/workflows/frontend.yml
@@ -53,7 +53,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-zeppelin-
       - name: Install application
-        run: ./mvnw clean install -DskipTests -am -pl zeppelin-web 
-Pweb-classic -Pspark-scala-2.12 -Pspark-3.4 -Pweb-dist ${MAVEN_ARGS}
+        run: ./mvnw clean install -DskipTests -am -pl 
zeppelin-web,zeppelin-web-angular -Pweb-classic -Pspark-scala-2.12 -Pspark-3.4 
-Pweb-dist ${MAVEN_ARGS}
       - name: Run headless test
         run: xvfb-run --auto-servernum --server-args="-screen 0 1024x768x24" 
./mvnw verify -pl zeppelin-web -Pweb-classic -Pspark-scala-2.12 -Pspark-3.4 
-Pweb-dist -Pweb-e2e ${MAVEN_ARGS}
       - name: Print zeppelin logs

Reply via email to