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

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


The following commit(s) were added to refs/heads/master by this push:
     new 3f4f772efef ci(storybook): use every core for story tests and install 
deps for Chromium only (#45600)
3f4f772efef is described below

commit 3f4f772efef717cda94f7acb11e9442ac4ad2130
Author: Evan Rusackas <[email protected]>
AuthorDate: Fri Oct 23 02:58:34 2026 +0100

    ci(storybook): use every core for story tests and install deps for Chromium 
only (#45600)
---
 .github/workflows/superset-frontend.yml                        | 4 ++--
 superset-frontend/.storybook/playwright.storybook-ci.config.ts | 4 ++++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/superset-frontend.yml 
b/.github/workflows/superset-frontend.yml
index 7b960e3bf1b..dbaabe0cc43 100644
--- a/.github/workflows/superset-frontend.yml
+++ b/.github/workflows/superset-frontend.yml
@@ -204,8 +204,8 @@ jobs:
 
       - name: Build Storybook and Run Tests
         run: |
-          docker run --rm $TAG bash -c \
-          "npx playwright install-deps && npx playwright install chromium && 
npm run test-storybook"
+          docker run --rm -e CI=true $TAG bash -c \
+          "npx playwright install --with-deps chromium && npm run 
test-storybook"
 
   actions-timeline:
     needs: [report-coverage, lint-frontend, validate-frontend, test-storybook]
diff --git a/superset-frontend/.storybook/playwright.storybook-ci.config.ts 
b/superset-frontend/.storybook/playwright.storybook-ci.config.ts
index b8f218ca943..14fb4fd2873 100644
--- a/superset-frontend/.storybook/playwright.storybook-ci.config.ts
+++ b/superset-frontend/.storybook/playwright.storybook-ci.config.ts
@@ -26,6 +26,10 @@ export default defineConfig({
   globalSetup: 'setup-stories.ts',
   use: { baseURL: storybookUrl },
   fullyParallel: true,
+  // Playwright defaults to half the cores. The story smoke tests are light and
+  // CPU-bound, so use every core on CI; locally keep the default so a dev
+  // machine stays responsive.
+  workers: process.env.CI ? '100%' : undefined,
   webServer: {
     command: 'npm run storybook -- --ci',
     url: `${storybookUrl}/index.json`,

Reply via email to