Feat linux permissions #940
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Everything looks good to me. Below are few typos and comments. |
bdu-birhanu
reviewed
Apr 18, 2025
|
|
||
| Linux has two features for controlling who can access what data: permissions and access control lists (ACLs). As a researcher responsible for research data, it is up to you to use these tools to control access to data within your shared storage allocations. | ||
|
|
||
| While permissions and ACLs may seem overwhelming at first, there are only a few common setups that cover most use cases, and we will cover these first. |
There was a problem hiding this comment.
Consider using "standard linux permissions" or a similar term instead of just "permissions" to help readers quickly understand.
bdu-birhanu
reviewed
Apr 18, 2025
| - Data accessible only to a specific subset of people. | ||
| - Read-only data. | ||
|
|
||
| You can check the permissions of any |
There was a problem hiding this comment.
This part looks incomplete.
bdu-birhanu
reviewed
Apr 18, 2025
|
|
||
| You can check the permissions of any | ||
|
|
||
| Further along on this page are the details you can put together to cover other cases. This article assumes the good practice of keeping data for each of your research projects in separate directories. |
There was a problem hiding this comment.
Would be good if we merged #920 first and cross-link here.
bdu-birhanu
reviewed
Apr 18, 2025
| ## Terminology | ||
|
|
||
| - **permission**: A set of binary bits that instruct the Linux operating system who can access what. | ||
| - **mode**: The |
There was a problem hiding this comment.
It seems this sentence got cut off
bdu-birhanu
reviewed
Apr 18, 2025
| - Each possible combination of the decimal representation | ||
| digits. The following conversion table will show how. | ||
|
|
||
| `file type` |
There was a problem hiding this comment.
This part also seems incomplete.
bdu-birhanu
added
pr: changes requested
|
I just noticed this PR is in "draft" . I have started the review, but I will check it again once it is ready. Thank you. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request
Overview
Add a document on how to manage permissions in shared storage on GPFS.
Proposed Changes
Related Issues