Skip to content

Revert "Do not check for journal.xml presence, temporarily (#4648)"#4652

Open
happz wants to merge 1 commit intomainfrom
revert-beakerlib-journal-xml
Open

Revert "Do not check for journal.xml presence, temporarily (#4648)"#4652
happz wants to merge 1 commit intomainfrom
revert-beakerlib-journal-xml

Conversation

@happz
Copy link
Contributor

@happz happz commented Mar 6, 2026

This reverts commit f2510a3.

@happz happz added this to planning Mar 6, 2026
@happz happz 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 Mar 6, 2026
@github-project-automation github-project-automation bot moved this to backlog in planning Mar 6, 2026
Copy link
Contributor

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

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 reverts a temporary change, re-enabling checks for journal.xml in tests/execute/basic/test.sh and tests/execute/result/subresults.sh. The changes are correct and no issues were found.

Copy link
Contributor

@thrix thrix left a comment

Choose a reason for hiding this comment

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

Straightforward revert of the temporary workaround. Re-enables journal.xml presence checks.

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 test coverage Improvements or additions to test coverage of tmt itself

Projects

Status: backlog

Development

Successfully merging this pull request may close these issues.

2 participants