Add image_to_install cli option - #314
Conversation
6e9f5f1 to
5a9f1eb
Compare
|
Tests running into some user ns issue on Github CI. Possibly something to do with skopeo interaction with containers storage |
5a9f1eb to
6a70aaa
Compare
|
Every |
df4552c to
2f36cef
Compare
2f36cef to
2a833df
Compare
I don't think we should justify this change because of digests. Again, this is something to fix in bootc. That aside, yes we should make it easier to support (And rather than simulating we should make it easier to actually do anaconda installs w/bcvk I think) |
the main issue seems to have been not passing the Edit: |
We bind mount host container storage into an ephemeral VM in order to create a base disk. This has the unfortunate effect of changing the manifest digest of the final image. bootc-dev#313 tried to address this by first exporting to OCI dir, then installing, but that too changes the final manifest digest. With the `image_to_install` option, we can pass `docker://quay.io/...` as the image to be installed which will have the correct manifest digest as we are pulling directly from the registry. Also, we might need networking to pull an image from a registry, so we pass `--net=host` to the podman command if the transport is registry We do depend on skopeo with this change as we can't use podman to query remote registry image manifests Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
Was using for the `Transport` enum, but it has some linux specific functions which breaks MacOs builds Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
skopeo inspect requires unshare which fails in unprivileged CI environments. For local images (from containers-storage), use podman image inspect instead, and only fall back to skopeo for remote transports like docker:// Also, fix `libvirt to-disk` command by making sure image has a transport attached to it as now we're passing `--source-imgref` to bootc install Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
2a833df to
ee9ada5
Compare
Currently, there is a bug in bcvk/bootc where the remote digest isn't properly reported and this cause a failure in bootc-operator CI since the remote digest doesn't match with the one reported by bootc status. A temporary fix is available at bootc-dev/bcvk#314 and we now build bcvk from that branch. Signed-off-by: Alice Frosi <afrosi@redhat.com>
Currently, there is a bug in bcvk/bootc where the remote digest isn't properly reported and this cause a failure in bootc-operator CI since the remote digest doesn't match with the one reported by bootc status. A temporary fix is available at bootc-dev/bcvk#314 and we now build bcvk from that branch. Signed-off-by: Alice Frosi <afrosi@redhat.com>
|
#324 has the pure revert that was also included here.
But why would one want to do that? |
3 participants
We bind mount host container storage into an ephemeral VM in order to
create a base disk. This has the unfortunate effect of changing the
manifest digest of the final image.
#313 tried to address this by
first exporting to OCI dir, then installing, but that too changes the
final manifest digest.
With the
image_to_installoption, we can passdocker://quay.io/...as the image to be installed which will have the correct manifest digest
as we are pulling directly from the registry.
Also, we might need networking to pull an image from a registry, so we
pass
--net=hostto the podman command if the transport is registryWe do depend on skopeo with this change as we can't use podman to query
remote registry image manifests
Fixes: #307