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.

Sign up for GitHub

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

Jump to bottom

[replication] Added PowerScale replication section #139

Merged
merged 8 commits into from
Feb 25, 2022
Merged

[replication] Added PowerScale replication section #139

merged 8 commits into from
Feb 25, 2022

Conversation

walker2
Copy link
Contributor

@walker2 walker2 commented Feb 14, 2022

Description

  • Added PowerScale replication section
  • Fixed a couple of formatting issues
  • Added PowerScale actions descriptions

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#116

Checklist:

  • Have you run a grammar and spell checks against your submission?
  • Have you tested the changes locally?
  • Have you tested whether the hyperlinks are working properly?
  • Did you add the examples wherever applicable?
  • Have you added high-resolution images?

maxan98
maxan98 previously approved these changes Feb 15, 2022
View reviewed changes
IliaRN
IliaRN previously approved these changes Feb 18, 2022
View reviewed changes
shanmydell
shanmydell requested changes Feb 18, 2022
View reviewed changes
Copy link
Collaborator

@shanmydell shanmydell left a comment

Choose a reason for hiding this comment

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

Link is broken in storage classes

@walker2
Copy link
Contributor Author

walker2 commented Feb 18, 2022

Link is broken in storage classes

Thanks, fixed the broken link

maxan98
maxan98 previously approved these changes Feb 21, 2022
View reviewed changes
coulof
coulof previously approved these changes Feb 22, 2022
View reviewed changes
Copy link
Collaborator

@coulof coulof left a comment

Choose a reason for hiding this comment

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

I committed a few changes directly to this branch.
Hope that's fine for you.

coulof
coulof previously approved these changes Feb 23, 2022
View reviewed changes
Copy link
Collaborator

@coulof coulof left a comment

Choose a reason for hiding this comment

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

lgtm

maxan98
maxan98 previously approved these changes Feb 23, 2022
View reviewed changes
@rsedlock1958
Update powerscale.md (#151)
9c7ee1f 
Made several grammatical edits.
@gallacher gallacher dismissed stale reviews from maxan98 and coulof via 9c7ee1f February 23, 2022 15:36
AronAtDell
AronAtDell previously approved these changes Feb 23, 2022
View reviewed changes
@walker2 walker2 merged commit c52c152 into release-1.2 Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmpost dmpost dmpost approved these changes

@maxan98 maxan98 maxan98 approved these changes

@shanmydell shanmydell shanmydell approved these changes

@AronAtDell AronAtDell AronAtDell left review comments

@coulof coulof coulof left review comments

@gallacher gallacher Awaiting requested review from gallacher

@rsedlock1958 rsedlock1958 Awaiting requested review from rsedlock1958

@IliaRN IliaRN Awaiting requested review from IliaRN

Assignees

@walker2 walker2

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@walker2 @dmpost @maxan98 @AronAtDell @coulof @shanmydell @IliaRN @rsedlock1958