Skip to content

Article on data access authorization and responsibility#897

Merged
wwarriner merged 13 commits into
uabrc:mainfrom
bdu-birhanu:feat-data-access
Apr 4, 2025
Merged

Article on data access authorization and responsibility#897
wwarriner merged 13 commits into
uabrc:mainfrom
bdu-birhanu:feat-data-access

Conversation

@bdu-birhanu
Copy link
Copy Markdown
Contributor

Pull Request

Overview

The aim of this PR is to introduce data access authorization and responsibilities of researchers/users of personal and shared Cheaha storage spaces and security Exceptions related to access to data from current and former personnel of UAB.

Proposed Changes

  • Added two tables outlining the responsibilities of researchers/users for personal and shared Cheaha storage..

Related Issues

Fixes #635
Fixes #894

@bdu-birhanu bdu-birhanu marked this pull request as draft January 30, 2025 23:27
Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great start! I want us to add LTS responsibilities as well. I believe if we do this right, the Cheaha and LTS responsibilities should be mostly the same. We may need some rows of the table to apply specifically to one or the other.

I think with a bit of rewording, we can get the two existing tables to work for both GPFS and LTS.

Comment thread mkdocs.yml Outdated
Comment thread docs/data_management/res/cheaha_project_directory.csv Outdated
Comment thread docs/data_management/res/cheaha_individual_account.csv Outdated
Comment thread docs/data_management/res/cheaha_individual_account.csv Outdated
Comment thread docs/data_management/res/cheaha_project_directory.csv Outdated
@wwarriner
Copy link
Copy Markdown
Contributor

wwarriner commented Feb 5, 2025

I think this section of the storage main page will be relevant: https://docs.rc.uab.edu/data_management/storage/#data-responsibilities-and-procedures. It should probably become part of the page being developed here.

Then add two questions to the main page like "How can I effectively manage my data?" and "What are my responsibilities for data management?"

@wwarriner
Copy link
Copy Markdown
Contributor

We also need to add a section reminding folks of other responsibilities around data that come from Enterprise IT. These apply to all UAB-owned computer systems, including Research Computing systems.

@wwarriner
Copy link
Copy Markdown
Contributor

wwarriner commented Feb 5, 2025

In line with the Enterprise IT policy information, we need to add that information to the responsibility table(s).

Some additional rows to consider

  • Data management, PI
  • Creating and maintaining metadata, PI
  • Maintaining permissions and access controls, PI (this may be redundant)
  • Compliance with regulatory controls (HIPAA, NIST 800-171, etc.)
    • Administrative (authorization and access), PI
    • Technical (encryption, firewall, etc.), RC
    • Physical (locks, cameras, sign-ins, etc.), RC
  • Obtaining security exceptions when required, PI
  • Operating expenses, RC
  • Hardware capital expenses (up to default quota), RC
  • Hardware capital expenses (beyond default quota), PI

@wwarriner
Copy link
Copy Markdown
Contributor

One last thing for the current review: make sure the names of storage offerings matches what is in the data management pages. Individual vs shared, calling storage "allocations" instead of "accounts" or "spaces".

@wwarriner wwarriner added the pr: changes requested Review complete, needs changes label Feb 5, 2025
@bdu-birhanu bdu-birhanu marked this pull request as ready for review February 5, 2025 22:15
@bdu-birhanu
Copy link
Copy Markdown
Contributor Author

bdu-birhanu commented Feb 10, 2025

I think this section of the storage main page will be relevant: https://docs.rc.uab.edu/data_management/storage/#data-responsibilities-and-procedures. It should probably become part of the page being developed here.

Then add two questions to the main page like "How can I effectively manage my data?" and "What are my responsibilities for data management?"

Done.

@bdu-birhanu bdu-birhanu reopened this Feb 10, 2025
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: changes requested Review complete, needs changes labels Feb 18, 2025
Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, some changes that need to be made to avoid miscommunications, and a change to one of the tables.

Please also back out the .pre-commit-config.yaml change. You can keep the change on your local machine until we merge #896, but let's keep it out of this PR.

Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/res/cheaha_project_directory.csv Outdated
@wwarriner wwarriner added pr: changes requested Review complete, needs changes and removed pr: review PR is ready for review labels Feb 18, 2025
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: changes requested Review complete, needs changes labels Feb 24, 2025
Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good. A couple minor things and we can set it to be merged.

Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
@wwarriner wwarriner added pr: changes requested Review complete, needs changes and removed pr: review PR is ready for review labels Feb 25, 2025
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: changes requested Review complete, needs changes labels Feb 26, 2025
Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is coming along well!

I realized we could do with a couple of changes to the CSVs, and there are a few language and wording changes. The backup section could be a cross-link to where it is in the storage.md article.

Once these changes are in, I plan to present this to Ralph for review. He will have insights onto things we are missing, or language changes we may need.

Great work so far, thank you!

Comment thread docs/data_management/res/cheaha_project_directory.csv Outdated
Comment thread docs/data_management/res/cheaha_individual_account.csv Outdated
Comment thread docs/data_management/res/cheaha_individual_account.csv Outdated
Comment thread docs/data_management/res/cheaha_project_directory.csv
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
Comment thread docs/data_management/data_responsibilities.md Outdated
@wwarriner wwarriner added pr: changes requested Review complete, needs changes and removed pr: review PR is ready for review labels Mar 6, 2025
@bdu-birhanu bdu-birhanu marked this pull request as draft March 7, 2025 19:45
@bdu-birhanu bdu-birhanu mentioned this pull request Mar 10, 2025
@bdu-birhanu bdu-birhanu marked this pull request as ready for review March 14, 2025 16:05
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: changes requested Review complete, needs changes labels Mar 14, 2025
@wwarriner
Copy link
Copy Markdown
Contributor

Looks good! In the future, keep file content changes in separate commits from renaming/moving files. I have the same habit of committing content and moving files together. When we do that, it makes it very hard for diff tools to pick up the changes to the file. Instead it looks like we deleted one file and added another file.

To be specific, first change the file and commit the work. Then, change the name of the file (or move it) and commit again.

Don't worry about undoing anything this time, but please separate them for next time. I'll work on breaking my own habit as well!

@wwarriner wwarriner added pr: merge PR is ready to merge pr: review PR is ready for review and removed pr: review PR is ready for review pr: merge PR is ready to merge labels Mar 20, 2025
@wwarriner
Copy link
Copy Markdown
Contributor

This is ready to merge from my perspective. I'd like to have Ralph look it over prior to merging, so I will leave as review.

Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of changes from Ralph's notes and suggestions, then we can merge.

Comment thread docs/data_management/research_data_responsibilities.md Outdated
Comment thread docs/data_management/res/cheaha_project_directory.csv Outdated
@wwarriner wwarriner added pr: changes requested Review complete, needs changes and removed pr: review PR is ready for review labels Apr 4, 2025
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: changes requested Review complete, needs changes labels Apr 4, 2025
@wwarriner wwarriner merged commit 968af03 into uabrc:main Apr 4, 2025
3 checks passed
@wwarriner wwarriner added pr: merge PR is ready to merge and removed pr: review PR is ready for review labels May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: merge PR is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify data access authorization responsibilities. Add information on security exceptions for accessing data of former personnel who have left UAB

2 participants