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

use external kickstart file for rhel4edge image #5825

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

jscotka
Copy link
Collaborator

@jscotka jscotka commented Jan 23, 2024

Image refresh for rhel4edge ( replaces: #5810)

  • image-refresh rhel4edge

Fixes #5774

@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: 3-ci-srv-05: [no-test] use external kickstart file for rhel4edge image Jan 23, 2024
@cockpituous
Copy link
Contributor

@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: 3-ci-srv-05: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 23, 2024
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for figuring this out! This is a lot better! 🎉

It just bothers me a bit to keep a duplicate of that sizable .ks file in our tree -- it will quickly grow out of date. It's also not documented how it was extracted and what was modified. Supposedly it's 99% the same with just some modifications?

Would it be possible to instead do that dynamically? Extract the .ks with isoinfo, and then depending on how complex the changes are, use sed -i or patch to apply the modifications? Then it will keep itself up to date, and we'll immediately know when there's a conflict.

Thanks!

@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: 4-ci-srv-04: [no-test] use external kickstart file for rhel4edge image Jan 23, 2024
@martinpitt martinpitt removed the bot label Jan 23, 2024
@martinpitt martinpitt changed the title WIP: 4-ci-srv-04: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 23, 2024
@martinpitt martinpitt added the bot label Jan 23, 2024
@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: rhos-01-35: [no-test] use external kickstart file for rhel4edge image Jan 23, 2024
@cockpituous cockpituous changed the title WIP: rhos-01-35: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 23, 2024
@cockpituous
Copy link
Contributor

@martinpitt
Copy link
Member

That failure looked rather random, let's try again.

@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: 3-cockpit-11: [no-test] use external kickstart file for rhel4edge image Jan 23, 2024
@cockpituous cockpituous changed the title WIP: 3-cockpit-11: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 23, 2024
@cockpituous
Copy link
Contributor

@martinpitt
Copy link
Member

OK, this failed twice in a row now, apparently there's something wrong?

�[K�[1;24r�[23;1HInstalling boot loader�[1;23r�[23;1H
�[K�[1;24r�[23;1H.
ERROR    Cannot write data: Broken pipe
Domain installation does not appear to have been successful.
If it was, you can restart your domain by running:
  virsh --connect qemu:///session start rhel4edgeodehods9.qcow2-builder
otherwise, please restart your installation.
+ rm rhel4edge-1706023131.iso.ks
rm: cannot remove 'rhel4edge-1706023131.iso.ks': No such file or directory

@jscotka
Copy link
Collaborator Author

jscotka commented Jan 24, 2024

Hi, this error seems very odd. I'll look at it. Thinking about if it this cannot be caused by some external event, like that disk space in podman container were exhausted or some problem with terminal. I'm curious.

@jscotka jscotka added bot and removed bot labels Jan 24, 2024
@jscotka
Copy link
Collaborator Author

jscotka commented Jan 24, 2024

Try to reschedule build again

@jscotka jscotka added bot and removed bot labels Jan 24, 2024
@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: 2-cockpit-9: [no-test] use external kickstart file for rhel4edge image Jan 24, 2024
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: 2-cockpit-9: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 24, 2024
@jscotka jscotka added bot and removed bot labels Jan 24, 2024
@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: 2-ci-srv-06: [no-test] use external kickstart file for rhel4edge image Jan 24, 2024
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: 2-ci-srv-06: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 24, 2024
@cockpituous
Copy link
Contributor

Comment on lines 80 to 82
# Wait until installation ends (with maximum timeout 30*20 sec)
# check console output for errors if it appears to end early
# if machine succeed it goes to "shut off" state.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this what virt-install's --wait option is for? So --wait 10 should wait for 10 mins, like you do here, but without shell polling.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this what virt-install's --wait option is for? So --wait 10 should wait for 10 mins, like you do here, but without shell polling.

I think it could be used, but problem will be caused, when the error occurs, it will hang for 10 minutes. Instead of fail early. because fail is faster than installation.

@jscotka jscotka added bot and removed bot labels Jan 25, 2024
@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: rhos-01-32: [no-test] use external kickstart file for rhel4edge image Jan 25, 2024
@jscotka jscotka added bot and removed bot labels Jan 26, 2024
@jscotka jscotka changed the title WIP: rhos-01-32: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 26, 2024
@jscotka jscotka added the bot label Jan 26, 2024
@cockpituous cockpituous changed the title use external kickstart file for rhel4edge image WIP: rhos-01-21: [no-test] use external kickstart file for rhel4edge image Jan 26, 2024
@cockpituous
Copy link
Contributor

@cockpituous cockpituous changed the title WIP: rhos-01-21: [no-test] use external kickstart file for rhel4edge image use external kickstart file for rhel4edge image Jan 26, 2024
@cockpituous
Copy link
Contributor

jscotka and others added 2 commits January 26, 2024 16:47
add newly created issue related to this workaround

try to use method noninteractive method without expect

PR review, do not actively wait, let the timeout to virt-install
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jscotka for figuring this out! 🌟 I see a few small and unimportant niggles, but they really shouldn't block this.

I'm still rather disappointed how hard it is to get and install an Edge image..

@martinpitt martinpitt merged commit e20f648 into cockpit-project:main Jan 29, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Image refresh for rhel4edge
3 participants