Skip to content

[On Week] Replace Loki with Playwright-based approach and run in CI#9594

Draft
weronikaolejniczak wants to merge 30 commits intoelastic:mainfrom
weronikaolejniczak:chore/vrt-playwright
Draft

[On Week] Replace Loki with Playwright-based approach and run in CI#9594
weronikaolejniczak wants to merge 30 commits intoelastic:mainfrom
weronikaolejniczak:chore/vrt-playwright

Conversation

@weronikaolejniczak
Copy link
Copy Markdown
Contributor

@weronikaolejniczak weronikaolejniczak commented Apr 15, 2026

Summary

Dynamic jobs

Now I know we might not typically want a dynamic job injection but because "building and deploying a website" is something completely independent from "building and deploying a Storybook", we might want to run both in parallel, have separate bash scripts for them AND it means we save some CI time:

Screenshot 2026-04-28 at 19 08 23

API Changes

N/A

Screenshots

N/A

Impact Assessment

Note: Most PRs should be tested in Kibana to help gauge their Impact before merging.

  • 🔴 Breaking changes — What will break? How many usages in Kibana/Cloud UI are impacted?
  • 💅 Visual changes — May impact style overrides; could require visual testing. Explain and estimate impact.
  • 🧪 Test impact — May break functional or snapshot tests (e.g., HTML structure, class names, default values).
  • 🔧 Hard to integrate — If changes require substantial updates to Kibana, please stage the changes and link them here.

Impact level: 🟢 None

Release Readiness

  • Documentation: {link to docs page(s)}
  • Figma: {link to Figma or issue}
  • Migration guide: {steps or link, for breaking/visual changes or deprecations}
  • Adoption plan (new features): {link to issue/doc or outline who will integrate this and where}

QA instructions for reviewer

  • If there are new snapshots, a commit is added chore(eui): add VRT baseline screenshots
  • If VRT is not failing, the whole pipeline passes and there are no GitHub comments made
  • If VRT fails, the whole pipeline fails and there is a GitHub comment with a diff table
  • When the blocked step to update VRT snapshots is approved, a commit is added chore(eui): update VRT baseline screenshots

Checklist before marking Ready for Review

Reviewer checklist

  • Approved Impact Assessment — Acceptable to merge given the consumer impact.
  • Approved Release Readiness — Docs, Figma, and migration info are sufficient to ship.

@weronikaolejniczak weronikaolejniczak changed the title Chore/vrt playwright [On Week] Replace Loki with Playwright-based approach and run in CI Apr 15, 2026
@weronikaolejniczak weronikaolejniczak self-assigned this Apr 15, 2026
@weronikaolejniczak weronikaolejniczak force-pushed the chore/vrt-playwright branch 3 times, most recently from 06a29a1 to 0140de5 Compare April 15, 2026 13:40
@weronikaolejniczak weronikaolejniczak added the skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) label Apr 15, 2026
@weronikaolejniczak weronikaolejniczak force-pushed the chore/vrt-playwright branch 2 times, most recently from aaea671 to 1267344 Compare April 15, 2026 15:27
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

History

cc @weronikaolejniczak

@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented Apr 28, 2026

💚 Build Succeeded

History

cc @weronikaolejniczak

@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented May 5, 2026

💔 Build Failed

Failed CI Steps

History

cc @weronikaolejniczak

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EuiIconTip] The story for VRT test doesn't work as expected

2 participants