Skip to content

Conversation

cgwalters
Copy link
Collaborator

@cgwalters cgwalters commented Oct 2, 2025

I was looking at something here and felt it could use some docs, so decided to work on that.

Assisted-by: Claude Code

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.

needs cargo fmt otherwise lgtm

Assisted-by: Claude Code
Signed-off-by: Colin Walters <walters@verbum.org>
@jeckersb jeckersb merged commit f3cdb75 into containers:main Oct 3, 2025
14 checks passed
cgwalters added a commit to cgwalters/composefs-rs that referenced this pull request Oct 18, 2025
The previous change to use 'create-image --bootable --stat-root' was
incorrect. The working version (PR containers#186) used 'compute-id --bootable'.

Revert to the working approach.

Signed-off-by: Claude <noreply@anthropic.com>
cgwalters added a commit to cgwalters/composefs-rs that referenced this pull request Oct 18, 2025
The changes to use stdout=PIPE and communicate() in testthing.py
caused the QEMU process to block until exit, preventing SSH
connections from being established during VM runtime.

Revert to the original working version from PR containers#186 that uses
wait() instead of communicate().

Signed-off-by: Claude <noreply@anthropic.com>
cgwalters added a commit to cgwalters/composefs-rs that referenced this pull request Oct 21, 2025
The previous change to use 'create-image --bootable --stat-root' was
incorrect. The working version (PR containers#186) used 'compute-id --bootable'.

Revert to the working approach.

Signed-off-by: Claude <noreply@anthropic.com>
Signed-off-by: Colin Walters <walters@verbum.org>
cgwalters added a commit to cgwalters/composefs-rs that referenced this pull request Oct 21, 2025
The changes to use stdout=PIPE and communicate() in testthing.py
caused the QEMU process to block until exit, preventing SSH
connections from being established during VM runtime.

Revert to the original working version from PR containers#186 that uses
wait() instead of communicate().

Signed-off-by: Claude <noreply@anthropic.com>
Signed-off-by: Colin Walters <walters@verbum.org>
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