Skip to content

Store guest in phase results#4320

Merged
psss merged 2 commits intomainfrom
phase-result-guest
Dec 9, 2025
Merged

Store guest in phase results#4320
psss merged 2 commits intomainfrom
phase-result-guest

Conversation

@happz
Copy link
Copy Markdown
Contributor

@happz happz commented Nov 14, 2025

Pull Request Checklist

  • implement the feature

@happz happz added this to planning Nov 14, 2025
@happz happz added the area | results Related to how tmt stores and shares results label Nov 14, 2025
@github-project-automation github-project-automation Bot moved this to backlog in planning Nov 14, 2025
@happz happz added the ci | full test Pull request is ready for the full test execution label Nov 14, 2025
@happz happz moved this from backlog to review in planning Nov 14, 2025
Copy link
Copy Markdown
Member

@LecrisUT LecrisUT left a comment

Choose a reason for hiding this comment

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

LGTM. Please also patch tmt.steps.execute.Execute.create_results

Comment thread tmt/result.py
Comment thread tmt/result.py
@happz
Copy link
Copy Markdown
Contributor Author

happz commented Nov 18, 2025

LGTM. Please also patch tmt.steps.execute.Execute.create_results

Why so?

@happz happz force-pushed the phase-result-guest branch 2 times, most recently from fc77988 to 8b6b98f Compare November 20, 2025 13:42
@LecrisUT
Copy link
Copy Markdown
Member

LGTM. Please also patch tmt.steps.execute.Execute.create_results

Why so?

Ah, I thought it was using a guest instance, but yeah those are just strings, nvm it.

@happz happz added the status | blocking other work An important pull request, blocking other pull requests or issues label Dec 1, 2025
@happz happz force-pushed the phase-result-guest branch 3 times, most recently from dfae0ef to f47f202 Compare December 4, 2025 14:20
@psss psss added this to the 1.64 milestone Dec 4, 2025
Copy link
Copy Markdown
Contributor

@skycastlelily skycastlelily left a comment

Choose a reason for hiding this comment

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

LGTM^^, expect the typo-ish

Comment thread tmt/steps/finish/__init__.py Outdated
@happz happz force-pushed the phase-result-guest branch from f47f202 to d349fe9 Compare December 8, 2025 07:05
@LecrisUT LecrisUT moved this from review to merge in planning Dec 9, 2025
@LecrisUT
Copy link
Copy Markdown
Member

LecrisUT commented Dec 9, 2025

/packit retest-failed

@psss psss force-pushed the phase-result-guest branch from d349fe9 to b747261 Compare December 9, 2025 09:32
@psss
Copy link
Copy Markdown
Member

psss commented Dec 9, 2025

Red job is an irrelevant guest failure, merging.

@psss psss merged commit b5ed2e2 into main Dec 9, 2025
27 of 28 checks passed
@psss psss deleted the phase-result-guest branch December 9, 2025 11:44
@github-project-automation github-project-automation Bot moved this from merge to done in planning Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area | results Related to how tmt stores and shares results 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

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

5 participants