Skip to content

Revert ansible test on rawhide skip#4967

Open
LecrisUT wants to merge 3 commits into
mainfrom
revert/4966
Open

Revert ansible test on rawhide skip#4967
LecrisUT wants to merge 3 commits into
mainfrom
revert/4966

Conversation

@LecrisUT

@LecrisUT LecrisUT commented Jun 8, 2026

Copy link
Copy Markdown
Member

LecrisUT added 3 commits June 8, 2026 10:05
Signed-off-by: Cristian Le <git@lecris.dev>
Signed-off-by: Cristian Le <git@lecris.dev>
@LecrisUT LecrisUT added test coverage Improvements or additions to test coverage of tmt itself ci | full test Pull request is ready for the full test execution labels Jun 8, 2026
@LecrisUT LecrisUT mentioned this pull request Jun 8, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request removes the workaround that skipped Ansible tests on Fedora Rawhide due to an upstream issue in both tests/finish/ansible/test.sh and tests/prepare/ansible/test.sh. I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@LecrisUT LecrisUT added the status | discuss Needs more discussion before closing label Jun 9, 2026
@LecrisUT

LecrisUT commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

Welp upstream's recommendation 1 complicates things. Basically older ansible in provision-* jobs are not expected to have rawhide and python3.15 support. This would basically break on testing-farm though. Maybe if we can at least get the fixes currently in rawhide into F44 the issue will not be that bad.

Footnotes

  1. https://github.com/ansible/ansible/issues/87081#issuecomment-4661576409

@LecrisUT LecrisUT added the status | blocking other work An important pull request, blocking other pull requests or issues label Jun 9, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Jun 10, 2026
@therazix therazix moved this from backlog to review in planning Jun 10, 2026
bajertom pushed a commit that referenced this pull request Jun 10, 2026
Unblock testing
- `dataclasses._MISSING_TYPE` was dropped
- Skip `ansible` on rawhide
ansible/ansible#87081
  - revert PR: #4967

---------

Signed-off-by: Cristian Le <git@lecris.dev>
Base automatically changed from fix/python3.15 to main June 10, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution status | blocking other work An important pull request, blocking other pull requests or issues status | discuss Needs more discussion before closing test coverage Improvements or additions to test coverage of tmt itself

Projects

Status: review

Development

Successfully merging this pull request may close these issues.

2 participants