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

jongyoul 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 597652e4c2 [ZEPPELIN-6558] Drop the redundant Chromium-channel 
Playwright projects from CI
597652e4c2 is described below

commit 597652e4c2e21a90511805abaf43bcc1619751b3
Author: HwangRock <[email protected]>
AuthorDate: Sat Jul 25 11:48:51 2026 +0900

    [ZEPPELIN-6558] Drop the redundant Chromium-channel Playwright projects 
from CI
    
    ### What is this PR for?
    
    `zeppelin-web-angular/playwright.config.js` declared five browser projects, 
three of them Chromium — the bundled build plus the `chrome` and `msedge` 
channels — so the e2e suite ran against Chromium three times per invocation. 
`run-playwright-e2e-tests (auth)` at ~53m is the critical path of the hour-long 
`frontend` workflow.
    
    ZEPPELIN-6314 set the goal as "Multi-browser support: Chromium / Firefox / 
WebKit" — three engines. Nothing under `e2e/` references the `Google Chrome` or 
`Microsoft Edge` projects, nor the `chrome`/`msedge` channels; every 
browser-specific branch keys on `webkit` or `firefox` (`dark-mode.spec.ts`, 
`notebook-keyboard-shortcuts.spec.ts`, `editor-search-page.ts`). The two 
channel projects add a 3x Chromium pass without adding engine coverage.
    
    This drops them, keeping `setup`, `chromium`, `firefox`, and `webkit`. 
WebKit is untouched, so ZEPPELIN-6450 is unaffected. 
`playwright.classic.config.js` is unaffected — it uses a single Chrome project. 
Follow-up from the review discussion on #5335.
    
    ### What type of PR is it?
    
    Improvement
    
    ### What is the Jira issue?
    
    [ZEPPELIN-6558](https://issues.apache.org/jira/browse/ZEPPELIN-6558)
    
    ### How should this be tested?
    
    `frontend` workflow runs the Playwright e2e suite. Expect 
`run-playwright-e2e-tests (auth)` to drop from ~53m to ~27m with Chromium / 
Firefox / WebKit coverage retained.
    
    ### Questions:
    
    * Does the license files need to update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    
    Closes #5338 from HwangRock/ZEPPELIN-6558.
    
    Signed-off-by: Jongyoul Lee <[email protected]>
---
 zeppelin-web-angular/playwright.config.js | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/zeppelin-web-angular/playwright.config.js 
b/zeppelin-web-angular/playwright.config.js
index 617d5776ed..6633e6cd55 100644
--- a/zeppelin-web-angular/playwright.config.js
+++ b/zeppelin-web-angular/playwright.config.js
@@ -47,16 +47,6 @@ module.exports = defineConfig({
       },
       dependencies: ['setup']
     },
-    {
-      name: 'Google Chrome',
-      use: {
-        ...devices['Desktop Chrome'],
-        channel: 'chrome',
-        permissions: ['clipboard-read', 'clipboard-write'],
-        storageState: 'playwright/.auth/user.json'
-      },
-      dependencies: ['setup']
-    },
     {
       name: 'firefox',
       use: {
@@ -75,16 +65,6 @@ module.exports = defineConfig({
         storageState: 'playwright/.auth/user.json'
       },
       dependencies: ['setup']
-    },
-    {
-      name: 'Microsoft Edge',
-      use: {
-        ...devices['Desktop Edge'],
-        channel: 'msedge',
-        permissions: ['clipboard-read', 'clipboard-write'],
-        storageState: 'playwright/.auth/user.json'
-      },
-      dependencies: ['setup']
     }
   ],
   webServer: process.env.CI

Reply via email to