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

Restructure the "Data management" page #880

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

bdu-birhanu
Copy link
Contributor

@bdu-birhanu bdu-birhanu commented Dec 24, 2024

Pull Request

Overview

This pull request aims to restructure the Data Management pages.

Proposed Changes

  • Updated the current storage.md and renamed to index.md page of Data Management.
  • Moved Cheaha-specific content, from storage.md, to a new page to cheaha_storage.md
  • Renamed the "Transfer" page to "Data Transfer Tools" to more accurately reflect its content.
  • Update all internal links to reflect the new page structure.

Related Issues

Fixes #874
Fixes #886

@bdu-birhanu bdu-birhanu added the pr: review PR is ready for review label Dec 24, 2024
Copy link
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.

I think separating storage from only being associated with "Cheaha" is great, so I'm happy to see this. Looking at it side-by-side with "Storage Alternative" it feels a bit confusing. A naive question would be what is different between storage and storage alternatives that they need to be separate articles?

So, perhaps we should have

"Cheaha Storage" -> "Research Computing Storage Solutions"
"Storage Alternatives" -> "Alternative Storage Solutions"

I think this would help readers readily distinguish between them. Thoughts?

@wwarriner wwarriner added pr: needs changes Review complete, needs changes and removed pr: review PR is ready for review labels Jan 8, 2025
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: needs changes Review complete, needs changes labels Jan 10, 2025
@wwarriner
Copy link
Contributor

I missed a detail. Let's undo these changes. Per our discussion, let's get the main "storage.md" page as the index for data management (for now), splitting out the Cheaha-specific sections into their own page. The Cheaha-specific page would be titled "Cheaha Storage (GPFS)" and it would be a sibling of the LTS and Alternative pages

@bdu-birhanu bdu-birhanu changed the title Change "Cheaha Storage" to "Storage" Restructure the "Data management" page Jan 20, 2025
@bdu-birhanu bdu-birhanu marked this pull request as draft January 27, 2025 22:24
@bdu-birhanu bdu-birhanu marked this pull request as ready for review January 27, 2025 23:31
@bdu-birhanu bdu-birhanu added this to the Sprint 25-09 milestone Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: review PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Restructure the Data Management page Change Cheaha Storage Heading to Storage
2 participants