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

Csm operator pflex support and v1 #390

Merged
merged 25 commits into from
Nov 15, 2022

Conversation

jooseppi-luna
Copy link
Contributor

@jooseppi-luna jooseppi-luna commented Nov 15, 2022

Description

Adding support for PowerFlex in csm-operator docs, removing tech preview since we are going v1.

@mbasha-dell @kumarkgosa @abhi16394 if one of you could take a look at the SDC deployment section, I would really appreciate that, I'm not as knowledgeable in that area.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#477

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?

Copy link

@rsedlock1958 rsedlock1958 left a comment

Choose a reason for hiding this comment

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

I have reviewed this. Several edits are required.,

@jooseppi-luna
Copy link
Contributor Author

I have reviewed this. Several edits are required.,

Edits made, thank you!

mbasha-dell
mbasha-dell previously approved these changes Nov 15, 2022
kumarkgosa
kumarkgosa previously approved these changes Nov 15, 2022
Copy link

@kumarkgosa kumarkgosa left a comment

Choose a reason for hiding this comment

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

Awesome!

Copy link
Contributor

@JacobGros JacobGros left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@kumarkgosa kumarkgosa left a comment

Choose a reason for hiding this comment

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

Great work!

Copy link

@rsedlock1958 rsedlock1958 left a comment

Choose a reason for hiding this comment

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

I have reviewed this. No edits are required.

@jooseppi-luna jooseppi-luna merged commit f267f31 into release-1.5 Nov 15, 2022
shanmydell pushed a commit that referenced this pull request Nov 25, 2022
* add pflex and 1.25 support

* Create powerflex.md

* powerflex.md updates

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* remove tech preview

* remove tech preview

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md

* Update powerflex.md
@shanmydell shanmydell deleted the csm-operator-pflex-support-and-v1 branch July 20, 2023 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants