Skip to content
This repository has been archived by the owner on Nov 7, 2024. It is now read-only.

deploy: Add a --no-imgref option #362

Merged
merged 1 commit into from
Sep 12, 2022

Conversation

cgwalters
Copy link
Member

In coreos/coreos-assembler#2523 we
taught coreos-assembler how to generate disk images with a
"pre-pulled" container image. This means that the first OS update
will use shared layers.

However...right now running e.g. rpm-ostree rebase quay.io/newimage
won't necessarily prune the previous image. (This may be considered
a bug)

But in practice, particularly for RHEL CoreOS we may not want to have
a default image reference - we don't (necessarily) want typing
rpm-ostree upgrade to do something.

With this, we can effectively pre-pull just the layers but not
the final image.

Copy link
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

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

lgtm

@lucab
Copy link
Member

lucab commented Sep 12, 2022

It would be good to refresh the quay.io/cgwalters/fcos-chunked image and then re-trigger CI before merging:

+ ostree-ext-cli container image deploy --sysroot /run/host --stateroot testos --imgref ostree-unverified-registry:quay.io/cgwalters/fcos-chunked:v1
error: Performing deployment: remote error: reading manifest v1 in quay.io/cgwalters/fcos-chunked: unknown: Tag v1 was deleted or has expired. To pull, revive via time machine

In coreos/coreos-assembler#2523 we
taught coreos-assembler how to generate disk images with a
"pre-pulled" container image.  This means that the *first* OS update
will use shared layers.

However...right now running e.g. `rpm-ostree rebase quay.io/newimage`
won't necessarily prune the previous image.  (This may be considered
a bug)

But in practice, particularly for RHEL CoreOS we may not want to have
a default image reference - we don't (necessarily) want typing
`rpm-ostree upgrade` to do something.

With this, we can effectively pre-pull just the layers but not
the final image.
@cgwalters
Copy link
Member Author

It would be good to refresh the quay.io/cgwalters/fcos-chunked image and then re-trigger CI before merging:

The CI was already fixed in #363 - this just needed a rebase 🏄

@cgwalters cgwalters merged commit f99c7fe into ostreedev:main Sep 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants