Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Test virtiofsd as root #3623

Closed
wants to merge 3 commits into from
Closed

Conversation

cgwalters
Copy link
Member

I want to reproduce the failure from coreos/rpm-ostree#4584 (comment) So I can verify the fix works here.

This is just better behavior for the case of e.g.
`cosa run -x "somecmd" > out.txt` to execute a command and
capture the output - we don't want to intermix status stuff in
there.

Further, this makes it easier to test the code when not on
a tty (in e.g. CI).
I want to reproduce the failure from coreos/rpm-ostree#4584 (comment)
So I can verify the fix works here.
@cgwalters
Copy link
Member Author

[2023-09-18T20:12:44.572Z] [2023-09-18T20:12:44Z ERROR virtiofsd] can't apply the child capabilities: failed to sync capabilities with the kernel

🎉 successfully failed.

Work around https://gitlab.com/virtio-fs/virtiofsd/-/merge_requests/197

Basically virtiofsd should not be trying to drop capabilities
when it can't do so, but we should be able to ask it to stop doing that.
@jlebon
Copy link
Member

jlebon commented Sep 19, 2023

I think we can close this now that #3625 is merged?

@cgwalters cgwalters closed this Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants