Skip to content

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Sep 19, 2025

This reverts commit 712634c.

We can have v2025.11 in cosa now that we have new libselinux.

We don't strictly need something from it but there's no point in keeping a pin either.

This reverts commit 712634c.

We can have v2025.11 in cosa now that we have new libselinux.

We don't strictly need something from it but there's no point in keeping
a pin either.
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request reverts a commit that pinned the rpm-ostree package version. The change is correct as the reason for the pin is no longer valid. I've added one comment suggesting a minor follow-up cleanup to remove the now-unused frozendeps variable for better code maintainability.

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@dustymabe
Copy link
Member

now that we have new libselinux.

for clarity, this is because we tagged https://bodhi.fedoraproject.org/updates/FEDORA-2025-2a0988eb82 into the f42-coreos-continuous koji tag (backed by a distrepo that makes a yum repo out of the builds in the tag):

$ koji list-tagged f42-coreos-continuous | grep libselinux
libselinux-3.8-3.fc42                     f42-coreos-continuous  plautrba

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@jlebon
Copy link
Member Author

jlebon commented Sep 22, 2025

CI should be fixed by coreos/fedora-coreos-config#3778.

@dustymabe dustymabe merged commit e9034ee into coreos:main Sep 22, 2025
5 of 6 checks passed
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.

2 participants