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

Update/Simplify CSM Support Matrix #1029

Merged
merged 50 commits into from
Mar 20, 2024
Merged

Conversation

gallacher
Copy link
Contributor

Description

This PR defines a CSM-based support matrix under a new prerequisites page rather than having it defined at the csi-driver level. The support matrices from the individual sections for orchestrators and storge platforms have also been removed and consolidated in the CSM-based support matrix. The section for cert-csi has been moved the root level to provide more visibility as well.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

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?

| Mirantis Kubernetes Engine | 3.7.x |
| Google Anthos | 1.15 |
| Rancher Kubernetes Engine | 1.4.x|
| Amazon Elastic Kubernetes Service | ?? |
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is Amazon Elastic Kubernetes Anywhere,not Amazon EKS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. Removed this from the support matrix since Amazon Elastic Kubernetes Anywhere is connected to a backed Kubernetes version

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.

Please address the review comments

Copy link
Collaborator

Choose a reason for hiding this comment

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

This can be added to the support matrix page for better access

weight: 1
---

## Supported Storage Platforms
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, there is no matrix to reveal co-relation between modules and drivers - Which modules are supported by each driver?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Look at the notes section

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was easier to call out what was not supported

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.

We lost the support matrix of which storage supports which CSM modules.

We have that table : http://10.x.x.x:1313/csm-docs/docs/deployment/csmoperator/#supported-csm-components
But this is only for the CSM Operator.

Btw, we probably want to leave the latest version only in that specific table

@@ -1,6 +1,7 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

There are 19 items on the left menu!

Can we combine:

  • Troubleshooting
  • License
  • Release notes
  • Cert-CSI
  • Security Policy
  • Support
  • References
    Under Support ?

Copy link
Contributor

Choose a reason for hiding this comment

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

As of now, I don't see a problem with the 19 items. But, if we decide to move the items under Support, let Cert-CSI and Troubleshooting be in the main left menu since they are/will be frequently used.

@@ -0,0 +1,42 @@
---
title: "Prerequisites"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we also move : http://10.x.x.x:1313/csm-docs/docs/csidriver/#features-and-capabilities
under the same chapter and maybe rename it support matrix ?

The idea is to have one page with everything.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@coulof - keeping the scope of these changes narrowed for now but this can be addressed in another iteration. Thank you for your feedback.

@gallacher
Copy link
Contributor Author

We lost the support matrix of which storage supports which CSM modules.

We have that table : http://10.x.x.x:1313/csm-docs/docs/deployment/csmoperator/#supported-csm-components But this is only for the CSM Operator.

Btw, we probably want to leave the latest version only in that specific table

If you look at the notes section, if called out what's supported from a modules perspective

{{</table>}}

> Notes:
> * The host operating system/version being used must align with what each Dell Storage platform supports. Please visit [E-Lab Navigator](https://elabnavigator.dell.com/eln/modernHomeSSM) for specific Dell Storage platform host operating system level support matrices.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we mention that there is a "Container Storage Interface(CSI)" section under Software Solutions which should not be referenced?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't want to get too verbose here so we can leave it as is for now

gallacher and others added 11 commits March 15, 2024 12:06
* Update _index.md

Added few lines to demonstrate the need for cert-csi, re-iterate support for self-certified customers, and enable customers to submit test results.

* Update _index.md

Removed line - 'As we begin to see more flavors of operating systems and orchestrator platforms, qualifying Dell CSI drivers against different storage arrays is a challenging task. Validating five different storage arrays with different permutations of orchestrators, operating systems, and their versions is not feasible.'
Copy link

github-actions bot commented Mar 20, 2024

Test Results

76 tests  ±0   76 ✅ ±0   3s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 62368eb. ± Comparison against base commit 3499bff.

♻️ This comment has been updated with latest results.

@gallacher gallacher merged commit fec7e09 into release-1.10.0 Mar 20, 2024
5 checks passed
@gallacher gallacher deleted the update-support-matrix branch March 20, 2024 15:00
hoppea2 pushed a commit that referenced this pull request Mar 20, 2024
* CSM 1.10 driver version changes (#949)

* Remove k3s debian to community supported (#951)

* Update PowerFlex version to 4.5.x (#953)

* Release notes for CSM 1.9 (#954)

* Updated the troubleshooting for Powerstore (#956)

* Architecture change of CSM deployment pages (#957)

* [feature-1091]: Update Authorization Redis storage class and proxy server address information (#964)

* MKE version update (#969)

* Update _MKE version

Update _MKE version

* Update _index.md

---------

Co-authored-by: Yamunadevi N Shanmugam <[email protected]>

* Adding medusa plus support for PowerStore (#970)

* Secret update for Powerflex driver (#971)

* Updating Authorization documentation (#972) (#973)

* updating generating token

* updating generating token

* updating generating token doc

* updating troubleshooting page

* update index (#974)

* Update support for all PowerScale 9.5.0.x minor versions (#980)

* Update support for PowerScale 9.5.0.x minor versions

* Update support for PowerScale 9.5.0.x minor versions

* Update support for PowerScale 9.5.0.x minor versions

* Update support for PowerScale 9.5.0.x minor versions

* Update support for PowerScale 9.5.0.x minor versions

* Updating Release notes (#982)

* added sample link for sc in csm-operator (#988)

* Updating prerequisite for Powerflex driver (#989)

* Updating doc for csi-powerflex (#993)

* Updating latest sidecar versions (#996)

* Responsive interactive tutorial (#998)

* Update app mobility version (#1001)

* Document automatic certificate creation during observability install via csm-operator (#1002)

* Update observability.md

* Update observability.md

* Update observability.md

* Update observability.md

* Update observability.md

* Update observability.md

* Update observability.md

* Add known issue (#1003)

* add known issue

* remove dash

* update known issues

* use table format

* `

* update release notes

* CSM Operator documentation missing steps to enable Authorization sidecar for Observability (#1008)

* Update resiliency uninstallation instructions (#1009)

* Add resiliency uninstallation steps (#1011)

* CSM Operator docs update (#1012)

* csm-operators docs update

* Update _index.md

* Fixing broken links (#1014)

* Fixing broken links (#1016)

Co-authored-by: Yamunadevi N Shanmugam <[email protected]>

* versioning the docs

* Update powerflex.md (#1027)

* Update support for Amazon EKS (#1025)

* Removed driver installation section from operator doc (#1030)

* add known issue about unity registration (#1034)

* update app mobility version (#1036)

* Update/Simplify CSM Support Matrix (#1029)

* Create _index.html

* Added CSM prerequisites

* Update _index.html

* Added notes

* Added cert-csi doc link

* Removed support matrix from csi drivers section

* Removed support matrix

* Removed orchestrator and platform support matrix

* Added ObjectScale and note about Auth support

* Removed support matrix

* Added Observability note

* Removed support matrix

* Added replication note

* Create cert-csi page

* Delete content/docs/csidriver/test/certcsi.md

* Created community qualified page

* Removed community qualified table

* Update qualified.md

* Create _index.md

* Delete content/docs/prerequisites/_index.html

* Update _index.md

* Create _index.md

* Delete content/docs/cert-csi/_index.html

* Removed Tanzu from Support Matrix

* Update qualified.md

* changed weight

* changed weight

* Update _index.md

* Update _index.md

* updated links

* Update _index.md

* moved images

* Update _index.md

* Update _index.md

* Added EKS qual

* Removed Elastic Kubernetes Anywhere

* Added VMware Tanzu

* updated cert-csi note

* Removed storage platforms for page

* Removed support matrix

* Update _index.md (#1031)

* Update _index.md

Added few lines to demonstrate the need for cert-csi, re-iterate support for self-certified customers, and enable customers to submit test results.

* Update _index.md

Removed line - 'As we begin to see more flavors of operating systems and orchestrator platforms, qualifying Dell CSI drivers against different storage arrays is a challenging task. Validating five different storage arrays with different permutations of orchestrators, operating systems, and their versions is not feasible.'

* Added module support matrix

* Added links to table

* updated link

* Added tested host operating systems

* Removed host operating system note

* Updated cert-csi note

* Update _index.md

* Update qualified.md

---------

Co-authored-by: vyshakav-dell <[email protected]>

* Updating Release notes (#1035)

Co-authored-by: csmbot <[email protected]>

---------

Co-authored-by: coulof <[email protected]>
Co-authored-by: gallacher <[email protected]>
Co-authored-by: CSM Bot <[email protected]>
Co-authored-by: tssushma <[email protected]>
Co-authored-by: Aaron Tye <[email protected]>
Co-authored-by: boyamurthy <[email protected]>
Co-authored-by: Adarsh Kumar Yadav <[email protected]>
Co-authored-by: Peres Kereotubo <[email protected]>
Co-authored-by: shefali-malhotra <[email protected]>
Co-authored-by: Bartosz Ciesielczyk <[email protected]>
Co-authored-by: Jooseppi Luna <[email protected]>
Co-authored-by: shaynafinocchiaro <[email protected]>
Co-authored-by: KerryKovacevic <[email protected]>
Co-authored-by: panigs7 <[email protected]>
Co-authored-by: vyshakav-dell <[email protected]>
Co-authored-by: csmbot <[email protected]>
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.

6 participants