Skip to content
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

Merged
merged 3 commits into from
Aug 29, 2023

Conversation

VladimirSlavik
Copy link
Contributor

@VladimirSlavik VladimirSlavik commented Aug 23, 2023

Previous syntax: /boot-iso [--webui]

New syntax: /build-image [--boot.iso] [--live] [--webui] [--updates.img]

  • --webui modifies boot.iso to use webui
  • if no type selected, builds boot.iso
  • builds all specified types, so /build-iso --boot.iso --live will build two images

Stacking with changes from #5052 for now.

@github-actions github-actions bot added infrastructure Changes affecting mostly infrastructure f39 labels Aug 23, 2023
@VladimirSlavik VladimirSlavik force-pushed the master-live-action branch 2 times, most recently from b578fd4 to e2598c9 Compare August 23, 2023 15:27
@rhinstaller rhinstaller deleted a comment from github-actions bot Aug 23, 2023
@rhinstaller rhinstaller deleted a comment from github-actions bot Aug 23, 2023
@VladimirSlavik VladimirSlavik added the manual testing required This issue can't be merged without manual testing label Aug 23, 2023
@VladimirSlavik
Copy link
Contributor Author

Rebased to allow testing on my fork. A bunch of pushes will follow, sorry for the noise.

@VladimirSlavik VladimirSlavik force-pushed the master-live-action branch 4 times, most recently from 2c0d639 to 098a328 Compare August 24, 2023 17:36
@VladimirSlavik
Copy link
Contributor Author

Tested piecewise at VladimirSlavik#117. Very lovely. Now also with 100% more updates.img.

@VladimirSlavik VladimirSlavik marked this pull request as ready for review August 24, 2023 17:37
@VladimirSlavik VladimirSlavik removed the manual testing required This issue can't be merged without manual testing label Aug 24, 2023
@VladimirSlavik
Copy link
Contributor Author

/kickstart-test --waive infra only

@VladimirSlavik
Copy link
Contributor Author

/webui-test --waive infra only

@VladimirSlavik
Copy link
Contributor Author

Hm, nice, the webui tests do not have correct status name. Not for this PR though.

@VladimirSlavik VladimirSlavik changed the title infra: Build also live images infra: Build also live and updates images Aug 24, 2023
@VladimirSlavik
Copy link
Contributor Author

/webui-test --waive infra only

Copy link
Member

@jkonecny12 jkonecny12 left a 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!

.github/workflows/build-image.yml Outdated Show resolved Hide resolved
.github/workflows/build-image.yml Outdated Show resolved Hide resolved
.github/workflows/build-image.yml Outdated Show resolved Hide resolved

- name: Create updates image
run: |
./scripts/makeupdates
Copy link
Member

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.

Copy link
Contributor Author

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
@VladimirSlavik
Copy link
Contributor Author

/kickstart-test --waive infra only

@VladimirSlavik
Copy link
Contributor Author

/webui-test --waive infra only

@VladimirSlavik VladimirSlavik merged commit 383431b into rhinstaller:master Aug 29, 2023
14 checks passed
@VladimirSlavik VladimirSlavik deleted the master-live-action branch August 29, 2023 14:35
@VladimirSlavik VladimirSlavik restored the master-live-action branch August 29, 2023 14:36
@VladimirSlavik VladimirSlavik deleted the master-live-action branch August 29, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f39 infrastructure Changes affecting mostly infrastructure
Development

Successfully merging this pull request may close these issues.

2 participants