Skip to content

Conversation

@cgwalters
Copy link
Collaborator

@cgwalters cgwalters enabled auto-merge May 27, 2025 13:11
@cgwalters cgwalters requested a review from jmarrero May 27, 2025 14:17
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.

Looks sane to me.

Not sure what's going on with that failed test though. I tried to reproduce it in a stream9 container and everything is working fine for me. Only thing I noticed was this bit in the test:

https://github.com/bootc-dev/bootc/blob/main/ostree-ext/tests/it/main.rs#L1334-L1336

Which seems a bit weird to call export_container() twice like that. I didn't dig into it to fully wrap my head around it but maybe there's some potential for that to go wrong? I'll post a follow-up to fix that (unless there's some non-obvious reason it's like that).

@cgwalters cgwalters merged commit 69d5702 into bootc-dev:main May 27, 2025
30 of 31 checks passed
jeckersb added a commit to jeckersb/bootc that referenced this pull request May 27, 2025
Just something weird I noticed when looking at bootc-dev#1328

Signed-off-by: John Eckersberg <[email protected]>
jeckersb added a commit to jeckersb/bootc that referenced this pull request Jun 12, 2025
This was changed previously in bootc-dev#1328, this reverts to the old format.

Signed-off-by: John Eckersberg <[email protected]>
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