qa: make the xfstests tmp directories to be readable and excutable#50217
qa: make the xfstests tmp directories to be readable and excutable#50217
Conversation
|
Both the relevant qa acls tests and qa fscrypt tests are running. |
Both passed. |
|
Only updated the commit comment since last tests. |
|
Only updated the comments in code since last tests. |
There was a problem hiding this comment.
I also suggest some improvements to the commit message -
The user namespace test case will execute some commands as the other
user. The temporary directory now has the rx permission for
other users by default. The tests then will fail otherwise.
Add back the generic/317 test.
Copying old commit message here for reference -
The user namespace test case will excute some commands as the other
user. And the temporary direcotry have now the rx permission for
other users by default and then will fail the tests.
This will chmod by making it to be readable and excutable.
|
jenkins test make check |
1 similar comment
|
jenkins test make check |
The |
without that patch, this PR would still fail at make check, maybe merge both PRs? |
Which commit introduced the failure? |
The user namespace test case will excute some commands as the other user. And the temporary direcotry now has the rx permission for other users by default. The test then will fail otherwise. Fixes: https://tracker.ceph.com/issues/58760 Signed-off-by: Xiubo Li <xiubli@redhat.com>
Since the permission denied bug has been fixed, just add the 317 test back. Fixes: https://tracker.ceph.com/issues/58760 Signed-off-by: Xiubo Li <xiubli@redhat.com>
|
Rebased it. |
The user namespace test case will excute some commands as the other user. And the temporary direcotry have now the rx permission for other users by default and then will fail the tests.
This will chmod by making it to be readable and excutable.
Fixes: https://tracker.ceph.com/issues/58760
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume toxjenkins test windows