-
Notifications
You must be signed in to change notification settings - Fork 15
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
Ro pvc feature changes #768
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removed a troubleshooting point as we are adding support of RO pvc
shefali-malhotra
requested review from
bharathsreekanth,
chimanjain,
Deepak-Ghivari,
gallacher,
mgandharva,
mjsdell,
prablr79,
rajendraindukuri,
rajkumar-palani,
rsedlock1958,
shanmydell and
sharont58
as code owners
August 22, 2023 11:42
chimanjain
previously approved these changes
Aug 22, 2023
rajkumar-palani
previously approved these changes
Aug 22, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
sharont58
previously approved these changes
Aug 22, 2023
shefali-malhotra
dismissed stale reviews from sharont58, rajkumar-palani, and chimanjain
via
August 22, 2023 13:36
f7bbc39
shefali-malhotra
requested review from
chimanjain,
rajkumar-palani and
sharont58
August 22, 2023 13:38
sharont58
approved these changes
Aug 22, 2023
chimanjain
approved these changes
Aug 22, 2023
mgandharva
approved these changes
Aug 22, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
gallacher
added a commit
that referenced
this pull request
Sep 20, 2023
* Adding RHEL 9.0 support for Powermax (#707) * Powerstore-Centralized-helm-Changes (#708) * Updated replication install doc for centralized helm charts (#711) * Release notes template (#714) * Using common helm charts for installation (#713) * Adding operator support for powerstore in wizard (#710) * add SCT to installation wizard pmax (#715) * Configurable Volume Attributes (#712) * PowerMax storage capacity Tracking documentation (#717) * Update powermax.md Added documentation for StorageCapacityTracking feature * Update powermax.md Added the StorageCapacityTracking * Update _index.md Made the StorageCapacityTracking to True * Update powermax.md Did changes related to SCT * Update powermax.md Added the supporting document for SCT * Update powermax.md Added row related to SCT * Update powermax.md Added document for SCT * Update powermax.md Added doc support for SCT * Update powermax.md Added doc support to PowerMax SCT * Update powermax.md Added doc support for SCT * Update powermax.md * Update powermax.md Added supporting feature for PowerMax driver * Update powermax.md Resolved the conflict * Update powermax.md Added supporting doc * Update powermax.md Modified the file * Update powermax.md Added Storage Capacity Tracking feature for Powermax * Update powermax.md --------- Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> * Health monitor enable steps (#722) * Update CODEOWNERS (#725) * Obs rep ocp (#724) * OpenShift 4.13 support for PowerMax (#709) * Cert Manager should display tooltip about the pre-requisite. (#721) * update resiliency support for ocp 4.13 (#727) * Doc updates for removing replication init container (#729) * add powerscale 9.5 support (#731) * Adding Operator Support for PowerScale in wizard (#733) * Automate cert-manager install with observability (#737) * Update observability.md to reflect automated cert-manager install * formatting * Update observability.md * Update observability.md * add link to cert-manager docs * Added example of running volumeHealthMetric suite (#734) * Adding the same example in /v1 i.e. for older release * Updated Google Anthos support for PowerMax (#740) * Powerflex-Centralized-helm-Changes (#742) * Updated v2.8.0 changes (#744) * Updated the resiliency module for Powerstore operator support in Installation Wizard UI (#745) * Update ocp version to 4.13: csi-powerflex (#741) * update ocp version to 4.13 * updated release notes with ocp4.13 support * added known issues * Documentation changes related to volume limit feature PowerMax (#749) * Update powermax.md Added changes related to Volume limits for PowerMax * Update powermax.md Added changes related to PowerMax volume limits feature * Update powermax.md Added changes related to Volume limit feature to PowerMax * Update powermax.md Added changes related to volume limit feature for PowerMax * Update powermax.md Added changes related to volume limit feature for PowerMax * Powerstore-Openshift-version-Changes (#747) * Updated PowerScale supported version for replication and observability (#750) * add prereq to install (#751) * Added docs for SCT feature Unity XT (#752) * Added example of few suites (#746) * Volume limit support in CSI Powerflex Driver (#748) * volume limit support in CSI Powerflex Driver * Incorporate review comments * Add support for kubernetes 1.28.0 - Observability and Replication (#756) Added Support for 1.28 for observability * PowerScale 9.5.0.4 support (#758) * PowerScale module operator support in Installation wizard (#757) * Adding k8 28 support for Powermax (#759) * Powermax operator support wizard (#754) * Update sdc to 3.6.1 (#762) * update sdc to 3.6.1 * updated release notes * Add offline bundle support in csm-operator (#760) * powerflex: support added for NFS (#763) * Added Support for k8s 1.28 in PowerStore (#767) * Helm changes unity v2.8.0 driver (#766) * Updated v2.8.0 driver version * Updated v2.8.0 driver version * Addressing the review comments on helm install * file replication (#769) * Added docs for SCT feature Powerflex (#761) * add sct changes to csm installation wizard. * powerflex storage capacity docs changes. --------- Co-authored-by: Ashley_Joy <Ashley.Joy@dell.com> * Csm installation wizard doc update (#771) * updated installation wizard doc changes * updated installation wizard doc changes * updated installation wizard doc changes * updated installation wizard doc changes * Updated UI for module support in powerscale (#765) * Updated UI for module support in powerscale * fix authorization issue * Updated support matrix (#768) Removed a troubleshooting point as we are adding support of RO pvc * Updating powermax templates (#773) * Powerstore OpenShift 4.13 known issues (#774) * Adding known issues related to Openshift 4.13 for CSI Powerstore * Update powerstore.md * Replication installation wizard (#775) * Testing release-notes tool (#778) Co-authored-by: csmbot <karavi@dell.com> * Add deprecation notice to dell-csi-operator (#780) * Volume limits support in CSI Powerstore * add isDefault to samples (#785) * PowerStore: Volume limits support in Installation Wizard (#781) * volume limits support in installation wizard for powerstore * indentation fix * updated csm-version check * updated comparator Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> * increased width of max-volumes-per-node field --------- Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> * Config file update (#783) * NFS documentation updated (#786) * CSI Powerstore 2.8.0 documentation changes * Removed arrayConnectivityConnectionLossThreshold from podmon-node (#790) * Helm changes and Operator upgrade changes (#795) * Update offline install to include step for driver install (#794) * uncomment resiliency args (#797) * Unity installation changes for central helm (#789) * Updated the unity helm installation changes in document * Updated the unity helm installation changes in document * Addressed review comment * Addressed review comment on gut hub ulr update * CSM Installation Wizard:Disable Operator option for Powerflex and Unity (#784) * disabled operator support for powerflex & unity, added operator prerequiste note * minor change * fixed linter issues * fixed minor bug * fixed linter issue * fixed linter issue * fixed linter issue * Added COSI documentation (#779) * docs(cosi): added COSI documentation Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> * fix(review/1): applied suggestions Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> * feat: added defaults for Bucket provisioning Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> * fix: command for helm install Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> * fix(review/2): clarified obtaining values Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> * fix(review/3): one more how-to clarification Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> --------- Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> * Documentation updates for CSI Unity (#791) * CSI Unity docs update * volume limit for unity in csm installation wizard * Removed please * Removed please * Removed please * Removed the known issue that is fixed * Replication Yaml creation enhancements (#804) * CSM Installation Wizard: Updated release notes (#799) * updated release notes * updated release notes * updated release notes * updated release notes * updated release notes * addressed review comment * added CSM installation wizard release notes link in common release notes * updated release notes * add max volume limit param in powerscale (#806) * Powerscale Upgrade (#807) * Powerscale Upgrade * Powerscale Upgrade * Powerscale Upgrade * Documentation updates for CSI Powerflex (#805) * updated powerflex docs * removed special chars * reverted orchestrator version bump up in csi-operator * adressed review comments * adressed review comments * updated example file name * addressed review comments * corrected n-2 version in csm-operator * fixed intendation (#810) * update dellctl images docs for CSM 1.8 (#812) * updated authorization and resiliency docs for release 1.8 (#800) * updated authorization and resiliency docs for release 1.8 * fixed support matrix for CSM components * Fixed Auth RPM and karavictl deprecation notes to refer to CSM 2.0 * fix: helm inconsistency (#816) * Observability updates for 1.8 release (#808) * Replication 1.6.0 updates for CSM 1.8.0 (#817) * Installation Wizard: PowerFlex new keys added (#809) * PowerFlex features added * Update known issue on Volume limits for Unity, PowerScale and Powerflex (#814) * update known issue * added vol limit known issue in powerscale * added vol limit known issue in powerscale * Updating volume limits known issue in Powerstore (#815) * Updated CSI-PowerScale Release notes and documentation (#821) * PowerMax documentation changes-release1.8 (#802) * Adding new changes for Powermax (#824) * csi-powerflex: NFS documentation updated (#822) * NFS parameters updated * updated resiliency version (#825) * remove-incorrect-section (#718) * Update csm-powermax-1.8.0.template (#829) * Bumped csi-volumegroup-snapshotter version (#828) * Bumped csi-volumegroup-snapshotter version * fixed CRDs * bump up version --------- Co-authored-by: JacobGros <jacobgrosner4@gmail.com> * remove incorrect line (#770) * remove incorrect line * Update powerflex.md * Replication/Observability 1.6 updates for Installation Wizard (#830) * Update release-notes for release-v1.8.0 (#832) * Update known issue powermax (#833) * Rebase Main with Release 1.8.0 branch. * Removing 1.7.1 Comment from Index Page * Removing the 1.7.1 Placeholder Content from the Top Banner. * Helm charts version update (#835) * helm-charts version update * helm-charts version update * Removing powerpath known issue (#838) * Remove 1.28 support for 1.8 (#837) * update docs. (#839) * Rephrase the CSM Wizard steps (#787) * Rephrase the CSM Wizard steps * Address comments on the phrasing --------- Signed-off-by: shanduur-dell <mateusz.urbanek@dell.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: sakshi-garg1 <74704849+sakshi-garg1@users.noreply.github.com> Co-authored-by: Santhosh Lakshmanan <111809751+santhoshatdell@users.noreply.github.com> Co-authored-by: Adarsh Kumar Yadav <109620911+adarsh-dell@users.noreply.github.com> Co-authored-by: chaganti-rajitha <111744573+chaganti-rajitha@users.noreply.github.com> Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> Co-authored-by: Rishabh Raj <120644626+rishabhatdell@users.noreply.github.com> Co-authored-by: tssushma <98810659+tssushma@users.noreply.github.com> Co-authored-by: Nitesh-Rewatkar <90749010+nitesh3108@users.noreply.github.com> Co-authored-by: gallacher <35462391+gallacher@users.noreply.github.com> Co-authored-by: AnandatDell <105747555+anandrajak1@users.noreply.github.com> Co-authored-by: Luna Xu <10015938+xuluna@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com> Co-authored-by: Chiman Jain <36687396+chimanjain@users.noreply.github.com> Co-authored-by: Jooseppi Luna <jooseppi_luna@dell.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com> Co-authored-by: ashleyvjoy <110008193+ashleyvjoy@users.noreply.github.com> Co-authored-by: Rajendra Indukuri <82365588+rajendraindukuri@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: rensyct <80810999+rensyct@users.noreply.github.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: Ashley_Joy <Ashley.Joy@dell.com> Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> Co-authored-by: shefali-malhotra <91597668+shefali-malhotra@users.noreply.github.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: CSM Bot <105446864+csmbot@users.noreply.github.com> Co-authored-by: csmbot <karavi@dell.com> Co-authored-by: Konduri Satya Narayana <109734493+satyakonduri@users.noreply.github.com> Co-authored-by: Aaron Tye <tyeaaron@gmail.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Co-authored-by: Mateusz Urbanek <110409467+shanduur-dell@users.noreply.github.com> Co-authored-by: KerryKovacevic <140824885+KerryKovacevic@users.noreply.github.com> Co-authored-by: Don Khan <don.khan@dell.com> Co-authored-by: JacobGros <jacobgrosner4@gmail.com> Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com> Co-authored-by: coulof <49235405+coulof@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
GitHub Issues
List the GitHub issues impacted by this PR:
Checklist: