Skip to content

Support bootc installation for mrack plugin #3728

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.

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

skycastlelily
Copy link
Collaborator

@skycastlelily skycastlelily commented May 13, 2025

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

@skycastlelily skycastlelily marked this pull request as draft May 13, 2025 08:41
@psss psss changed the title Implement the bootcbeaker provision plugin Implement the bootc-beaker provision plugin May 13, 2025
@skycastlelily skycastlelily changed the title Implement the bootc-beaker provision plugin Support bootc installation for mrack plugin May 14, 2025
@skycastlelily skycastlelily marked this pull request as ready for review May 15, 2025 06:52
@skycastlelily
Copy link
Collaborator Author

hi @henrywang , it would be great if you could review this mr when you get some time to spare, as I get many clue from you boot-beaker-test code,thanks:)

@skycastlelily
Copy link
Collaborator Author

Hi, @henrywang @thrix @ukulekek this is finally ready for review, feel free to check on your side:)
You will end up with a warn result, but it seems will not prevent the following steps on the tmt plan to run.
But it's ugly and will pretty likely confuse the users, to resolve that we need to
add a check-system task to the job, and tmt doesn't support add task, so I opened a mr
And there is more stones: beaker doesn't support fetch-url, so I opened mr
It seems that beaker maintainer is not active, if it's not going to be merged, I may could find a workaround from mrack side, whose maintainer is pretty welcoming for patch:)

Please note, we only support fedora now, and if we agree on it's gonna to implement in this way,I'm gonna to open
more mrs to support rhel and cs and switch task:)

@skycastlelily
Copy link
Collaborator Author

FYI, the beaker mr is merged 🎉 thanks to the maintainer, and he also said we could get it land soon as a minor update :)

skycastlelily added a commit to skycastlelily/mrack that referenced this pull request May 23, 2025
Block tmt's bootc installation on beaker feature:
teemtee/tmt#3728
dav-pascual pushed a commit to neoave/mrack that referenced this pull request May 27, 2025
Block tmt's bootc installation on beaker feature:
teemtee/tmt#3728
@skycastlelily
Copy link
Collaborator Author

mrack mr to support dry was merged and landed long time ago, so enable tests/provision/beaker test and extend the test for this mr , mrack mr to support fetch_url is also merged and landed,btw:)

@skycastlelily skycastlelily mentioned this pull request Jun 7, 2025
8 tasks
@thrix thrix added this to planning Jun 24, 2025
@thrix thrix moved this to implement in planning Jun 24, 2025
@skycastlelily
Copy link
Collaborator Author

Please note,we should create a check-system task for tmt users, instead of using bootc-beaker-test's:)

@teemtee teemtee deleted a comment from happz Jun 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: implement
Development

Successfully merging this pull request may close these issues.

1 participant