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

F24 lmc ostree #117

Closed
wants to merge 6 commits into from
Closed

F24 lmc ostree #117

wants to merge 6 commits into from

Conversation

bcl
Copy link
Contributor

@bcl bcl commented Apr 27, 2016

These patches allow livemedia-creator --no-virt (and --no-virt inside a mock) to be used for --make-pxe-live and --make-ostree-live

bcl added 6 commits April 21, 2016 10:29
Also make sure that a --make_pxe_live --no-virt doesn't use autopart or
other mountpoints.
Instead of a partitioned disk image. This will allow a single pass
of lmc to be used to create an ostree live PXE image.
This moves the mounting inside make_live_images so that it isn't trying
to copy the fsimage while having it mounted.
umount tries to delete a mountpoint if it has lorax.imgutils in the
path. This doesn't work right if you try to umount something mounted
deeper on the path.

This adds a delete option, which is True by default, to skip the delete.
This makes sure the contents of /boot are at the expected locations in
/boot and in sys_root. For partitioned images it mounts the separate
/boot partition on /boot. For both fsimage and partitioned images ir
binf mounts it to sys_root so that the kernel+initrd can be found.

The boot directory isn't always named boot.0, so wildcard it and let the
count check handle failure if there is more than 1.
Because --no-virt uses a fsimage disk you can't create a /boot
partition. This kickstart will also work for creating the PXE files
inside a mock environment.

Also rename rhel-atomic-pxe-live.ks to fedora-atomic-pxe-live.ks
@jkonecny12
Copy link
Contributor

AFAICT this looks good to me.

@jkonecny12 jkonecny12 added the ACK label Apr 28, 2016
@bcl
Copy link
Contributor Author

bcl commented May 31, 2016

Pushed to master only, too much change for post-freeze F24.

@bcl bcl closed this May 31, 2016
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.

None yet

2 participants