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

Add permission fixing in finish module #212

Closed
saramonzon opened this issue Mar 18, 2024 · 2 comments
Closed

Add permission fixing in finish module #212

saramonzon opened this issue Mar 18, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request urgent

Comments

@saramonzon
Copy link
Member

saramonzon commented Mar 18, 2024

Although the HPC configuration should ensure that the permissions for folders is 775 and 664 for files, all of them assigned to group bi, somehow some intermediate or copied files are created with user specific permissions in the shared folders.
This is annoying for several reasons:

  • Sometimes any other user needs to modify that file, or include some file in that "wrong permissions" folder and we need to ask to the owner to change it.
  • Sometimes the owner may be not working in the unit anymore or can be on holidays so we need to create a ticket for Bruno to delete it.
  • Archive module strongly fails when it tries to handle a file or folder with wrong permissions making that we need to start all over the process, which is a real pain.

The solution would be to add a process after finish module so when each service analysis is finished, we fix the permissions ensuring that all folders are 2775, and all files 664 and adding bi group ownership to all files and folders. Ideally this permission and group configuration should be stablished in the config file so we can change it if needed without changing the code.

@saramonzon
Copy link
Member Author

It would also be helpful to add a helper script does fix this to any folder, like references or research so we can easily run it when we create a new folder in the HPC if the permissions are wrongly set. This script must be documented in the Wiki.

@Shettland
Copy link
Member

It would also be helpful to add a helper script does fix this to any folder, like references or research so we can easily run it when we create a new folder in the HPC if the permissions are wrongly set. This script must be documented in the Wiki.

I agree. I think it would be more useful to create a separate script so it can be executed from finish or copy-sftp without repeating lines of code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request urgent
Projects
None yet
Development

No branches or pull requests

3 participants