Skip to content

Conversation

@albinsun
Copy link
Contributor

@albinsun albinsun commented Jan 2, 2026

Which issue(s) this PR fixes:

Issue #2373

What this PR does / why we need it:

  1. Easier to track corresponding issue of known/skipped test cases on test report.
  2. Refactor to unify pytest.mark.skip_version_* marks to the more explicit pytest.mark.skip_version_if.
  3. For known issues, now we use skip_version_if() if issue milestone is decided. For those still in Planning, may use xfail (expected fail)

Special notes for your reviewer:

To leverage this enhancement, pytest.mark.skip() or pytest.mark.xfail() need to have issue URL in reason field

Additional documentation or context

Verification (harvester-install-and-test-e2e-daily#28)
image

@albinsun albinsun force-pushed the mark_e2e_known_issues branch from 0306396 to 1ea2e30 Compare January 2, 2026 09:22
@albinsun albinsun changed the title Add report content if links in Skipped and XFailed reason Show Link if there is URL in Skipped and XFailed reason Jan 2, 2026
@albinsun albinsun force-pushed the mark_e2e_known_issues branch 5 times, most recently from 891e0ca to 134fbd9 Compare January 5, 2026 17:26
Signed-off-by: Albin Sun <albin.sun@suse.com>
@albinsun albinsun force-pushed the mark_e2e_known_issues branch from 134fbd9 to 85644f0 Compare January 5, 2026 17:30
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.

1 participant