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

commit: Always enable generation of composefs metadata if possible #4495

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

alexlarsson
Copy link
Collaborator

There is no particular cost to always adding composefs metadata to rpm-ostree composes, so always do so if libostree is new enough and has the support compiled in.

@openshift-ci
Copy link

openshift-ci bot commented Jul 10, 2023

Hi @alexlarsson. Thanks for your PR.

I'm waiting for a coreos member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

There is no particular cost to always adding composefs metadata to
rpm-ostree composes, so always do so if libostree is new enough and has
the support compiled in.
@alexlarsson alexlarsson changed the title commit: Always enable generation pf composefs metadata if possible commit: Always enable generation of composefs metadata if possible Jul 10, 2023
@alexlarsson
Copy link
Collaborator Author

Got this failure in "CI / Container build (pull_request) ":

rpm-ostree: /lib64/libostree-1.so.1: version `LIBOSTREE_2023.4' not found (required by rpm-ostree)
Error: error building at STEP "RUN cd /etc/yum.repos.d/ && curl -LO https://pkgs.tailscale.com/stable/fedora/tailscale.repo &&     rpm-ostree install tailscale &&     systemctl enable tailscaled &&     ostree container commit": error while running runtime: exit status 1

Are we mixing ostree versions used during build and at runtime?

@cgwalters
Copy link
Member

Are we mixing ostree versions used during build and at runtime?

Yes, in this job. It will be a little annoying to fix. If we wait a little bit until the new ostree hits fcos stable, the issue will go away on its own.

@alexlarsson
Copy link
Collaborator Author

What is the status of this? Did the new ostree hit fcos?

@cgwalters cgwalters closed this Aug 15, 2023
@cgwalters cgwalters reopened this Aug 15, 2023
@cgwalters
Copy link
Member

What is the status of this? Did the new ostree hit fcos?

$ podman run --pull=newer --rm -ti quay.io/fedora/fedora-coreos:stable rpm -q ostree
ostree-2023.4-2.fc38.x86_64

So yes

@cgwalters
Copy link
Member

/ok-to-test

@cgwalters cgwalters merged commit e2103c5 into coreos:main Aug 15, 2023
13 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.

2 participants