-
Notifications
You must be signed in to change notification settings - Fork 356
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
infra: Build also live and updates images #5071
infra: Build also live and updates images #5071
Conversation
b578fd4
to
e2598c9
Compare
e2598c9
to
ceadb3e
Compare
Rebased to allow testing on my fork. A bunch of pushes will follow, sorry for the noise. |
2c0d639
to
098a328
Compare
Tested piecewise at VladimirSlavik#117. Very lovely. Now also with 100% more updates.img. |
098a328
to
d46219b
Compare
/kickstart-test --waive infra only |
/webui-test --waive infra only |
Hm, nice, the webui tests do not have correct status name. Not for this PR though. |
/webui-test --waive infra only |
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.
Looks good to me! Thanks a lot!
|
||
- name: Create updates image | ||
run: | | ||
./scripts/makeupdates |
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.
Hopefully this will work. Without the tag specification it takes the latest
which might be problematic in some cases. Also outdated PR might broke this even more.
I wonder if building RPMs and just adding them to the updates image wouldn't be better solution.
In any case, great improvement - just a bit fragile.
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.
Agreed; if people start using this, perhaps it can do the rebase, same as tests. However in the first approximation I wanted to stay consistent, none of the other image builds rebase. Which might be a problem, now that I think about it... Well, another PR.
Now run it from comment as: /build-image [--boot.iso] [--live] [--webui]
To do so: /build-image --updates.img
d46219b
to
2dc3903
Compare
/kickstart-test --waive infra only |
/webui-test --waive infra only |
Previous syntax:
/boot-iso [--webui]
New syntax:
/build-image [--boot.iso] [--live] [--webui] [--updates.img]
--webui
modifies boot.iso to use webui/build-iso --boot.iso --live
will build two imagesStacking with changes from #5052 for now.