Skip to content

kola: Add --qemu-bind-ro#3593

Merged
cgwalters merged 1 commit intocoreos:mainfrom
cgwalters:kola-bind
Sep 13, 2023
Merged

kola: Add --qemu-bind-ro#3593
cgwalters merged 1 commit intocoreos:mainfrom
cgwalters:kola-bind

Conversation

@cgwalters
Copy link
Copy Markdown
Member

I'm trying to debug
kola run --ssh-on-test-failure ext.rpm-ostree.destructive.container-image interactively but one problem is that because the test is classified as disconnected, we run qemu with no networking. This makes it impossible to get remote debug tools.

With this I can --qemu-bind-ro /usr to get access to the debug tools in my dev container's /usr. But one needs to manually mount inside the VM, because injecting the mount into the Ignition config in kola is a bit trickier.

For reference it looks like e.g.
mount -t 9p -o trans=virtio,version=9p2000.L /kola/usr /run/hostusr in the ssh shell.

@cgwalters
Copy link
Copy Markdown
Member Author

impossible

(Well, not really, I mean I could inject them into the disk image or base64 encode them and paste into the ssh session; or scp externally and find the credentials...but, as far as getting data into a VM, 9p and hopefully in the future virtio are much more efficient and ergonomic)

I'm trying to debug
`kola run --ssh-on-test-failure ext.rpm-ostree.destructive.container-image`
interactively but one problem is that because the test is classified
as disconnected, we run qemu with no networking.  This makes
it impossible to get remote debug tools.

With this I can `--qemu-bind-ro /usr` to get access to the debug
tools in my dev container's `/usr`.  But one needs to manually mount
inside the VM, because injecting the mount into the Ignition
config in kola is a bit trickier.

For reference it looks like e.g.
`mount -t 9p -o trans=virtio,version=9p2000.L /kola/usr /run/hostusr`
in the ssh shell.
Copy link
Copy Markdown
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

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

lgtm

@cgwalters cgwalters merged commit d812406 into coreos:main Sep 13, 2023
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Sep 13, 2023
PR coreos#3593 was written and opened before coreos#3428 but merged after it. CI had
passed but was not rerun, so this went in. This would be fixed by using
a merge bot on this repo that reruns CI before merge (See also
https://bors.tech/essay/2017/02/02/pitch/.)

Fixes: d812406 ("kola: Add `--qemu-bind-ro`")
dustymabe pushed a commit that referenced this pull request Sep 13, 2023
PR #3593 was written and opened before #3428 but merged after it. CI had
passed but was not rerun, so this went in. This would be fixed by using
a merge bot on this repo that reruns CI before merge (See also
https://bors.tech/essay/2017/02/02/pitch/.)

Fixes: d812406 ("kola: Add `--qemu-bind-ro`")
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