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

[BACKPORT][v1.5.5][IMPROVEMENT] Cannot read/write to block volume when the container is run as non-root #8123

Closed
github-actions bot opened this issue Mar 7, 2024 · 5 comments
Assignees
Labels
kind/backport Backport request kind/improvement Request for improvement of existing function require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Milestone

Comments

@github-actions
Copy link

github-actions bot commented Mar 7, 2024

backport #8088

@github-actions github-actions bot added kind/backport Backport request kind/improvement Request for improvement of existing function require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated labels Mar 7, 2024
@github-actions github-actions bot added this to the v1.5.5 milestone Mar 7, 2024
@chriscchien chriscchien self-assigned this Apr 9, 2024
@chriscchien
Copy link
Contributor

Hi @PhanLe1010,

I tested on v1.5.x head following the test plan.

When I try case 1 with a v2 volume, I hitted the permission error issue, could you take a look on this? thank you.

> k exec -it block-volume-test -- /bin/sh -c "dd if=/dev/longhorn/testblk bs=1M count=100 | md5sum"
dd: failed to open '/dev/longhorn/testblk': Permission denied
d41d8cd98f00b204e9800998ecf8427e  -

supportbundle_3c16e0fc-5ad5-4ff5-b9fd-650c98800934_2024-04-09T03-44-55Z.zip

@chriscchien
Copy link
Contributor

Hi @PhanLe1010 , I tried sue v1.5.x-head image to test again, still hit the permission error issue on v2 volume, the same cluster with v1.6.x images did not encounter this issue, thank you.

@PhanLe1010
Copy link
Contributor

The issue is that instance-manager has not been updated to point to new version of longhorn-spdk-engine and go-spdk-helper

@PhanLe1010
Copy link
Contributor

Hi @PhanLe1010 , I tried sue v1.5.x-head image to test again, still hit the permission error issue on v2 volume, the same cluster with v1.6.x images did not encounter this issue, thank you.

We decided to not backport this fix to v2 data engine. Please ignore the failed test for v2

@chriscchien
Copy link
Contributor

chriscchien commented Apr 13, 2024

Verified pass on longhorn v1.5.x (longhorn-engine 012165, longhorn-instance-manager 34d22c)

V1 volume worked well with test stpes, v2 volume not work on this ticke as this comment described.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backport Backport request kind/improvement Request for improvement of existing function require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Projects
None yet
Development

No branches or pull requests

2 participants