Skip to content

Automation: Disable harvester auto install e2e test and In Progress state checks#17544

Merged
yonasberhe23 merged 3 commits into
rancher:masterfrom
yonasberhe23:disable_harvester_auto_install_test
May 8, 2026
Merged

Automation: Disable harvester auto install e2e test and In Progress state checks#17544
yonasberhe23 merged 3 commits into
rancher:masterfrom
yonasberhe23:disable_harvester_auto_install_test

Conversation

@yonasberhe23
Copy link
Copy Markdown
Contributor

@yonasberhe23 yonasberhe23 commented May 7, 2026

Summary

Fixes #rancher/qa-tasks#2346

3 fixes to unblock CI:

  1. Disable harvester auto install e2e test.The test is currently failing because the Harvester UI extension auto-install is broken Harvester UI extension auto install fails #17543. Skipping the test until the underlying dashboard bug is fixed.

  2. Disable repositories e2e test 'In Progress' state checks until Repositories list no longer shows In Progress state while a new repo is being downloaded #17554 is resolved.

  3. Disable repositories detail page check until Repository detail page is blank and throws errors when opened while the repo is still downloading #17556 is resolved.

Checklist

  • The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
  • The PR has a Milestone
  • The PR template has been filled out
  • The PR has been self reviewed
  • The PR has a reviewer assigned
  • The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
  • The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes
  • The PR has been reviewed in terms of Accessibility
  • The PR has considered, and if applicable tested with, the three Global Roles Admin, Standard User and User Base

@yonasberhe23 yonasberhe23 added this to the v2.15.0 milestone May 7, 2026
@yonasberhe23 yonasberhe23 self-assigned this May 7, 2026
eva-vashkevich
eva-vashkevich previously approved these changes May 7, 2026
Copy link
Copy Markdown
Member

@eva-vashkevich eva-vashkevich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Maybe add a note to uncomment once harvester issue is resolved?

@yonasberhe23
Copy link
Copy Markdown
Contributor Author

Good thinking, i'll do that now

eva-vashkevich
eva-vashkevich previously approved these changes May 7, 2026
izaac
izaac previously approved these changes May 7, 2026
@yonasberhe23 yonasberhe23 changed the title Automation: Disable harvester auto install e2e test Automation: Disable harvester auto install e2e test and In Progress state checks May 7, 2026
@yonasberhe23 yonasberhe23 dismissed stale reviews from izaac and eva-vashkevich via 1dd41db May 7, 2026 23:06
@yonasberhe23 yonasberhe23 force-pushed the disable_harvester_auto_install_test branch from 1dd41db to fe7ad14 Compare May 7, 2026 23:07
izaac added a commit to izaac/dashboard-e2e-pw that referenced this pull request May 7, 2026
The "auto install harvester and begin process of importing a harvester
cluster" test exercises the harvester page's auto-install button, which
hits upstream bug rancher/dashboard#17543: installHarvesterExtension
throws "TypeError: Cannot read properties of undefined (reading
'conditions')" at uiplugins.ts:276. Upstream PR rancher/dashboard#17544
disables the corresponding cypress test for the same reason. Local 5x
runs match — :108 is hard-blocked on the dashboard fix.

Rest of the test body cleaned up to mirror upstream's simpler flow so
re-enabling later is straightforward:
- Drop the multi-page tour (extensions#installed → repos → harvester)
  in favor of upstream's single-pass nav. Each SPA re-render was a
  roll of the dice on the harvester extension's masthead-registration
  race; fewer navs = fewer chances to lose.
- Drop the uiplugin CR poll, the explicit page.reload after install,
  and the conditional reload-banner click.
- Add a waitForResponse for the JS bundle URL
  (raw.githubusercontent.com/harvester/harvester-ui-extension; same
  URL for community + prime charts) so the importBtn check doesn't
  race a slow github fetch when the test re-enables.
- Add a 500-status retry on the install-action POST (the marker for
  #17543) — mirrors upstream's intercept-based retry pattern.

The :271 (missing repo message) and :415 (update extension version)
tests use the extensions card install path and don't hit #17543. The
fixme leaves them passing reliably; full 4/5 valid runs ran 4/4 in
~1.7 min each.
@yonasberhe23 yonasberhe23 merged commit efd1eb2 into rancher:master May 8, 2026
96 of 97 checks passed
@yonasberhe23 yonasberhe23 deleted the disable_harvester_auto_install_test branch May 8, 2026 00:48
@github-actions github-actions Bot mentioned this pull request May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants