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

Fix URL path for reanaconda #43

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jkonecny12
Copy link
Member

@jkonecny12 jkonecny12 commented Feb 28, 2024

The path to download installable tree is not correct so run of this tool will fail.

Unfortunately, I wasn't able to make it run even with this fix but at least it's closer to get it working so I'm proposing it anyway when I have it locally already.

There is also minor improvement for dependency checking.

Tell user early that the execution of reanaconda is not possible because
dependencies are not installed.
vmlinuz and initrd was in the past stored in `isolinux` directory but
now it is stored in `image/pxeboot`. Keep the backward compatibility by
trying the old path if download fails.
@jkonecny12
Copy link
Member Author

jkonecny12 commented Feb 28, 2024

I started to play with this tool to get easy way of running a VM with updates image. However, during playing with this I've got to conclusion that this tool might be too complicated for not that big benefit.

I'm thinking if the save-store solution is really saving so much of the time because in general the boot is fast (a few seconds usually), however, what takes time is download of external sources. My idea for improvement here is to drop the save-store functionality (or leave it there as side solution) and add an easy way to start VM from URL with updates image and kickstart with a caching of the downloaded artifacts (initrd, vmlinuz, stage2 image).

One of the biggest pain with Anaconda development is how to get updates image into booted VM. And for that this tool is pretty nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant