-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add test for bootupd/bootc ostree containers #1037
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I think the main information that want from this test is a confirmation that the final system is bootable. Can you reuse here some parts of the reboot
test to verify that?
The /etc directory is writable from %post also with ostree, while /usr/libexec only on normal systems.
7cad1d1
to
5ee5331
Compare
Still WIP, can't get the validation to work. Given that both ostree and reboot tests have special detection, that's... expected, I guess |
5ee5331
to
91994b0
Compare
Here's a couple of findings related to earlier discussions with @VladimirSlavik which should likely be taken into account when finishing the test:
|
Taking over the test development from Vladimir, will continue working on it after some higher priority tasks I've on my plate right now are done. |
Thank you Jirko, I think eventually this test will replace the rpm-ostree-container-rhel9 which does not test booting of the target system, just presence of the payload). I am fixing it for now in #1052. |
FYI, there is some tests renaming / consolidation undergoing: #1054 |
Follow-up in #1096 |
WIP, needs some more checking...