Skip to content

ci: temporarily disable Desktop E2E — red on every PR since Aug 1 engines churn (#76627) - #76629

Merged
teknium1 merged 1 commit into
mainfrom
ci/disable-desktop-e2e-temporarily
Aug 2, 2026
Merged

ci: temporarily disable Desktop E2E — red on every PR since Aug 1 engines churn (#76627)#76629
teknium1 merged 1 commit into
mainfrom
ci/disable-desktop-e2e-temporarily

Conversation

@teknium1

@teknium1 teknium1 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Temporarily disables the Desktop E2E CI job — it is red on every PR regardless of diff since the Aug 1 night engines/npm churn, blocking the entire merge queue. Root fix + re-enable tracked in #76627 (Ari).

Changes

  • .github/workflows/ci.yml: e2e-desktop.if gated with false && (one-line re-enable: delete it). Comment block in the workflow carries the evidence and the tracking issue.

Validation

Evidence
Not diff-dependent identical spec failures on #76573 (Python-only, twice — before and after rebase) and #76582 (docs-only)
Harness-level mock-backend Electron window never gets a title (fails in ~200ms); dead-backend boot-failure spec still passes
Window matches #76499#76562#76575 engines/lockfile churn; main has no completed successful CI run since c7b4b4e1
Not a required check branch protection: none — skipped job cannot wedge merges

Infographic

e2e lane parked

…ines churn (#76627)

The Playwright suite fails identically on every PR regardless of diff
(verified on a Python-only PR and a docs-only PR): the mock-backend
Electron window never gets a title, so boot/chat/setup/interim specs all
fail; only the dead-backend boot-failure path still passes. Breakage
window matches the Aug 1 night engines/npm churn (#76499/#76562/#76575).

Gated with 'false &&' in the job condition — delete that to re-enable.
Root-fix + re-enable tracked in #76627 (Ari).
@alt-glitch alt-glitch added type/test Test coverage or test infrastructure P1 High — major feature broken, no workaround comp/desktop Electron desktop app (apps/desktop/*) sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation labels Aug 2, 2026
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

ran on b7f498c

ℹ️ Info

CI-sensitive file review · View job

PR touches sensitive files, but the ci-reviewed label has been added, approving them.

Sensitive files changed:

@teknium1 teknium1 added the ci-reviewed applied to manually approve dangerous changes label Aug 2, 2026
@teknium1
teknium1 merged commit 226e270 into main Aug 2, 2026
98 of 101 checks passed
@teknium1
teknium1 deleted the ci/disable-desktop-e2e-temporarily branch August 2, 2026 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-reviewed applied to manually approve dangerous changes comp/desktop Electron desktop app (apps/desktop/*) P1 High — major feature broken, no workaround sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation type/test Test coverage or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants