-
Notifications
You must be signed in to change notification settings - Fork 338
DAOS-18324 control: Apply group perms to hugepage and dpdk dirs #17260
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
Conversation
Features: control Allow-unstable-test: true Signed-off-by: Tom Nabarro <[email protected]>
|
Ticket title is 'DLCK & DDB could use setuid/setgid to be easily runnable by any user in the system' |
|
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17260/1/testReport/ |
| echo "RUN: chown -R ${_TARGET_USER}:${_TARGET_USER} /dev/hugepages" | ||
| chown -R "${_TARGET_USER}:${_TARGET_USER}" /dev/hugepages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor - Might be good to put the command in a variable, so the printed message stays in sync with what's actually run
Co-authored-by: Jan Michalski <[email protected]> Signed-off-by: Tom Nabarro <[email protected]>
|
Test stage Functional on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17260/1/execution/node/1075/log |
| echo "RUN: chmod -R g+rw /tmp/dpdk" | ||
| echo "RUN: chmod -R g+rwx /tmp/dpdk" | ||
| chmod -R g+rw /tmp/dpdk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, forgot to update the actual command that's run to g+rwx.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My bad I've committed Jan's suggested change but forgot to check this.
…dk-fileperms-fix Signed-off-by: Tom Nabarro <[email protected]>
Features: control Signed-off-by: Tom Nabarro <[email protected]>
|
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17260/3/testReport/ |
|
CI failing on unrelated permissions/acl failures in NLT test, all others passed. Requesting force Landing. |
Allow dlck off-line debug tool to create hugepage and dpdk files if
run as a member of the daos_server user-group.
Features: control
Allow-unstable-test: true
Steps for the author:
After all prior steps are complete: