Skip to content

ci: bugbash — enable full *-cdp suites, disable *-cdp-remediated - #34531

Draft
cacieprins wants to merge 6 commits into
developfrom
bugbash/cdp-full-suite
Draft

ci: bugbash — enable full *-cdp suites, disable *-cdp-remediated#34531
cacieprins wants to merge 6 commits into
developfrom
bugbash/cdp-full-suite

Conversation

@cacieprins

Copy link
Copy Markdown
Collaborator
  • Closes N/A — temporary bugbash branch; do not merge as-is

Additional details

Temporary PR workflow config to run the full proxy-disabled (*-cdp) matrix for a CDP Fetch bugbash:

  • Removes all *-cdp-remediated known-green subset jobs from the PR workflow and all-jobs-passed
  • Sets require-opt-in: false on all full *-cdp jobs so they always run (without run-proxy-disabled-tests=true)
  • Leaves system-tests-cookies-cdp and unit-tests-proxy-disabled enabled
  • Plumbs require-opt-in through driver-integration-tests-chrome-inject-document-domain and driver-integration-memory-tests so those jobs accept the workflow parameter

Do not merge until remediations are restored. Full *-cdp jobs remain omitted from all-jobs-passed so failures stay advisory.

Steps to test

  1. Confirm CircleCI PR workflow starts the full *-cdp job set (driver chrome/electron/inject-document-domain/memory, system chrome/electron, app, launchpad).
  2. Confirm no *-cdp-remediated jobs appear in the workflow.
  3. Triage any *-cdp failures as flake vs real proxy-disabled gaps.

How has the user experience changed?

No change

PR Tasks

  • [na] Is there an associated issue with maintainer approval for PR submission?
  • [na] Have tests been added/updated?
  • [na] Has a PR for user-facing changes been opened in cypress-documentation?
  • [na] Have API changes been updated in the type definitions?

Made with Cursor

Temporary branch config so the full proxy-disabled matrix runs on every PR
workflow while remediations stay off; restore before merging.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cypress

cypress Bot commented Aug 7, 2026

Copy link
Copy Markdown

cypress    Run #73772

Run Properties:  status check passed Passed #73772  •  git commit 30936585fd: Merge remote-tracking branch 'origin/develop' into bugbash/cdp-full-suite
Project cypress
Branch Review bugbash/cdp-full-suite
Run status status check passed Passed #73772
Run duration 16m 39s
Commit git commit 30936585fd: Merge remote-tracking branch 'origin/develop' into bugbash/cdp-full-suite
Committer Bill Glesias
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 17
Tests that did not run due to a developer annotating a test with .skip  Pending 1236
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 35237
View all changes introduced in this branch ↗︎
UI Coverage  67.61%
  Untested elements 23  
  Tested elements 48  
Accessibility  98.97%
  Failed rules  0 critical   3 serious   1 moderate   0 minor
  Failed elements 18  

@cacieprins
cacieprins marked this pull request as ready for review August 10, 2026 17:27
@cacieprins
cacieprins requested a review from a team as a code owner August 10, 2026 17:27
@cacieprins
cacieprins marked this pull request as draft August 10, 2026 17:27
cacieprins and others added 4 commits August 10, 2026 13:28
It's already a pipeline parameter declared in both the setup config and
the continued pipeline.yml, so it's inherited automatically. Re-passing
it via continuation's parameters map collides with that and CircleCI
rejects the continuation with "Conflicting pipeline parameters" whenever
the pipeline is triggered with run-proxy-disabled-tests=true.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…uite

# Conflicts:
#	.circleci/src/pipeline/workflows/pull-request.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants