Skip to content

Comments

Cargo: update ostree-ext to b76d75d6#5553

Merged
jmarrero merged 1 commit intocoreos:mainfrom
jmarrero:update-ostree-ext
Jan 19, 2026
Merged

Cargo: update ostree-ext to b76d75d6#5553
jmarrero merged 1 commit intocoreos:mainfrom
jmarrero:update-ostree-ext

Conversation

@jmarrero
Copy link
Member

@jmarrero jmarrero commented Jan 15, 2026

This updates ostree-ext and related dependencies (composefs-rs, bootc-internal-utils) to address an issue where G_MESSAGES_DEBUG=all causes packages to be incorrectly shown as "Removed" during rebase operations.

The root cause was that GLib debug output was being written to stdout instead of stderr, corrupting the ostree commit hash parsing in ostree-ext's container layer import. When the subprocess output is polluted with debug messages, the merge commit ends up missing content from derived layers.

Fixes: OCPBUGS-64692
Fixes: RHEL-130454
See: bootc-dev/bootc#1917

This updates ostree-ext and related dependencies (composefs-rs,
bootc-internal-utils) to address an issue where G_MESSAGES_DEBUG=all
causes packages to be incorrectly shown as "Removed" during rebase
operations.

The root cause was that GLib debug output was being written to stdout
instead of stderr, corrupting the ostree commit hash parsing in
ostree-ext's container layer import. When the subprocess output is
polluted with debug messages, the merge commit ends up missing content
from derived layers.

Fixes: OCPBUGS-64692
Fixes: RHEL-130454
See: bootc-dev/bootc#1917
Copy link

@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 updates ostree-ext and related dependencies to fix an issue with GLib debug output corrupting stdout parsing. The changes correctly update the commit hashes for the git dependencies in Cargo.toml and Cargo.lock. The update is well-described and the changes are consistent with the stated goal. I find no issues with this pull request.

@jeckersb
Copy link
Collaborator

The pr-merge failure looks like an infra flake, how do we retry that one?

Copy link
Collaborator

@jeckersb jeckersb left a comment

Choose a reason for hiding this comment

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

LGTM other than CI failure

@jmarrero jmarrero merged commit 6528e39 into coreos:main Jan 19, 2026
16 of 18 checks passed
@jmarrero jmarrero mentioned this pull request Jan 22, 2026
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