Skip to content

Mock provision: make detection of finished processes more deterministic #4437

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
mkoncek wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Mock provision: make detection of finished processes more deterministic #4437

mkoncek wants to merge 17 commits into from

Conversation

@mkoncek
Copy link
Collaborator

@mkoncek mkoncek commented Dec 17, 2025

Based on: #4173
I found out that various environments (fedora-rawhide-x86_64) print different number of empty strings on their stdout as the commands spawned in the chroots finish. This causes some environments to hang.
This PR makes the detection more deterministic.

Pull Request Checklist

  • implement the feature
  • write the documentation
  • extend the test coverage
  • update the specification
  • adjust plugin docstring
  • modify the json schema
  • mention the version
  • include a release note

mkoncek added 17 commits December 16, 2025 11:01
@mkoncek mkoncek added bug Something isn't working step | provision Stuff related to the provision step plugin | mock Related to mock provision plugin labels Dec 17, 2025
@happz happz added the ci | full test Pull request is ready for the full test execution label Jan 7, 2026
@happz happz added this to planning Jan 7, 2026
@github-project-automation github-project-automation bot moved this to backlog in planning Jan 7, 2026
@happz happz moved this from backlog to review in planning Jan 7, 2026
@happz happz added the status | blocked The merging of PR is blocked on some other issue label Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working ci | full test Pull request is ready for the full test execution plugin | mock Related to mock provision plugin status | blocked The merging of PR is blocked on some other issue step | provision Stuff related to the provision step

Projects

planning
Status: review

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mkoncek @happz