-
Notifications
You must be signed in to change notification settings - Fork 17
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
minor typo fixed #258
Merged
Merged
minor typo fixed #258
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
khareRajshree
requested review from
gallacher,
tdawe,
alikdell,
atye,
hoppea2,
chaganti-rajitha,
coulof,
meggm,
niranjan-n1,
shaynafinocchiaro,
sharmilarama,
tssushma,
taohe1012,
P-Cao,
shanmydell,
baoy1,
YianZong and
forrestxia
as code owners
August 18, 2023 11:49
ashleyvjoy
approved these changes
Aug 18, 2023
niranjan-n1
approved these changes
Aug 18, 2023
atye
added a commit
that referenced
this pull request
Sep 14, 2023
* Update CODEOWNERS (#238) * add powerflex helm chart changes for v1.7.1. (#245) * add powerflex helm chart changes for v1.7.1. * add helm chart changes for v1.7.1 release. (#246) * Skip existing releases (#272) * updated helm-validations file (#243) * Powerscale v2.8.0 release (#242) * Powerscale v2.8.0 release * Added suppoert container-storage-modules * Added suppoert container-storage-modules * recommiting branch due to checks not working * dummy commit to trigger the checks * StorageCapacityTracking feature changes (#236) * Update _helpers.tpl Added changes related to StorageCapacityTracking * Update controller.yaml Added changes related to StorageCapacityTracking * Update csidriver.yaml Added changes related to StorageCapacityTracking * Update values.yaml Added changes related to StorageCapacityTracking * Update _helpers.tpl Storage Capacity Tracking K8s 1.24. * Update values.yaml Changed the version to v1.24 * Update csidriver.yaml Set the default value to false * Update controller.yaml Default value of the StorageCapacityTracking should be true * Update csidriver.yaml Default value should be true * Update controller.yaml Updated the controller file * Update csidriver.yaml Updated csidriver.yaml file * Update controller.yaml Done changes related to SCT * csm-replication: Removed init container usage (#244) * CSIUnity v2.8.0 release (#248) * Updated v2.8.0 unity driver version * addressed review comments on values.yaml file update * Added the Volumelimits for Powermax (#240) Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> * PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Adding k8 28 support for powermax (#253) * Adding K8s1.28 support for PowerFlex (#254) * K8s 1.28 support unity (#255) * support k8s v1.28 * support k8s v1.28 * minor typo fixed (#258) * PowerScale k8s-1.28 support (#260) * powerstore v2.8.0 & k8s 1.28 support (#262) * Adding volume limits support for CSIPowerstore * Adding volume limits support for CSIPowerstore * Adding maxPowerstoreVolumesPerNode in values.yaml inside container-storage-modules * Removed duplicate entry for X_CSI_DRIVER_NAME in node.yaml for CSI Powerstore (#273) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Update Authorization images version (#277) * Updated replication controller/replicator/migrator/rescanner SC image versions for CSM v1.8.0 (#278) * Updated csi-metadata-retriever version to 1.5.0 for CSI Powerstore (#280) * Update Observability image versions for CSM 1.8. (#281) * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * [charts/cosi] add COSI driver helm chart (#267) * add cosi driver helm chart * add cosi specific linters * remove go mod from cosi linters * fix typos * add cosi-ct-config.yaml * change workflow name * change log levels in cosi chart * update actions * change name from cosi-driver to cosi * add kubelinter for all charts * revert kubelinter * change log levels in chart's template --------- Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * [charts/container-storage-modules]: Uncomment Resiliency arguments (#287) * uncomment resiliency args * fix spacing * correct cosi log level setting (#288) * [charts/container-storage-modules]: new keys added for PowerFlex driver (#286) * PowerFlex new keys added * NFS ACL parameter removed (#289) * Adding new fields for powermax (#290) * bumped csi-volumegroup-snapshotter version (#292) * reset the CSM helm chart to main branch (#293) --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: chaganti-rajitha <111744573+chaganti-rajitha@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Co-authored-by: tssushma <98810659+tssushma@users.noreply.github.com> Co-authored-by: Santhosh Lakshmanan <111809751+santhoshatdell@users.noreply.github.com> Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Nitesh-Rewatkar <90749010+nitesh3108@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alikdell <52920355+alikdell@users.noreply.github.com> Co-authored-by: Don Khan <don.khan@dell.com> Co-authored-by: mdutka-dell <115641913+mdutka-dell@users.noreply.github.com> Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com> Co-authored-by: Rishabh Raj <120644626+rishabhatdell@users.noreply.github.com> Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com>
shaynafinocchiaro
added a commit
that referenced
this pull request
Sep 20, 2023
* updated helm-validations file (#243) * Powerscale v2.8.0 release (#242) * Powerscale v2.8.0 release * Added suppoert container-storage-modules * Added suppoert container-storage-modules * recommiting branch due to checks not working * dummy commit to trigger the checks * StorageCapacityTracking feature changes (#236) * Update _helpers.tpl Added changes related to StorageCapacityTracking * Update controller.yaml Added changes related to StorageCapacityTracking * Update csidriver.yaml Added changes related to StorageCapacityTracking * Update values.yaml Added changes related to StorageCapacityTracking * Update _helpers.tpl Storage Capacity Tracking K8s 1.24. * Update values.yaml Changed the version to v1.24 * Update csidriver.yaml Set the default value to false * Update controller.yaml Default value of the StorageCapacityTracking should be true * Update csidriver.yaml Default value should be true * Update controller.yaml Updated the controller file * Update csidriver.yaml Updated csidriver.yaml file * Update controller.yaml Done changes related to SCT * csm-replication: Removed init container usage (#244) * CSIUnity v2.8.0 release (#248) * Updated v2.8.0 unity driver version * addressed review comments on values.yaml file update * Added the Volumelimits for Powermax (#240) Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> * PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Adding k8 28 support for powermax (#253) * Adding K8s1.28 support for PowerFlex (#254) * K8s 1.28 support unity (#255) * support k8s v1.28 * support k8s v1.28 * minor typo fixed (#258) * PowerScale k8s-1.28 support (#260) * powerstore v2.8.0 & k8s 1.28 support (#262) * Adding volume limits support for CSIPowerstore * Adding volume limits support for CSIPowerstore * Adding maxPowerstoreVolumesPerNode in values.yaml inside container-storage-modules * Removed duplicate entry for X_CSI_DRIVER_NAME in node.yaml for CSI Powerstore (#273) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Update Authorization images version (#277) * Updated replication controller/replicator/migrator/rescanner SC image versions for CSM v1.8.0 (#278) * Updated csi-metadata-retriever version to 1.5.0 for CSI Powerstore (#280) * Update Observability image versions for CSM 1.8. (#281) * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * updated chart version (#282) * removed externalAccess from powerflex manifests. (#283) * [charts/cosi] add COSI driver helm chart (#267) * add cosi driver helm chart * add cosi specific linters * remove go mod from cosi linters * fix typos * add cosi-ct-config.yaml * change workflow name * change log levels in cosi chart * update actions * change name from cosi-driver to cosi * add kubelinter for all charts * revert kubelinter * change log levels in chart's template --------- Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * [charts/container-storage-modules]: Uncomment Resiliency arguments (#287) * uncomment resiliency args * fix spacing * correct cosi log level setting (#288) * [charts/container-storage-modules]: new keys added for PowerFlex driver (#286) * PowerFlex new keys added * NFS ACL parameter removed (#289) * Adding new fields for powermax (#290) * bumped csi-volumegroup-snapshotter version (#292) * reset the CSM helm chart to main branch (#293) * [feature-885]: Add workflow update to release branch (#297) * Update CODEOWNERS (#238) * add powerflex helm chart changes for v1.7.1. (#245) * add powerflex helm chart changes for v1.7.1. * add helm chart changes for v1.7.1 release. (#246) * Skip existing releases (#272) * updated helm-validations file (#243) * Powerscale v2.8.0 release (#242) * Powerscale v2.8.0 release * Added suppoert container-storage-modules * Added suppoert container-storage-modules * recommiting branch due to checks not working * dummy commit to trigger the checks * StorageCapacityTracking feature changes (#236) * Update _helpers.tpl Added changes related to StorageCapacityTracking * Update controller.yaml Added changes related to StorageCapacityTracking * Update csidriver.yaml Added changes related to StorageCapacityTracking * Update values.yaml Added changes related to StorageCapacityTracking * Update _helpers.tpl Storage Capacity Tracking K8s 1.24. * Update values.yaml Changed the version to v1.24 * Update csidriver.yaml Set the default value to false * Update controller.yaml Default value of the StorageCapacityTracking should be true * Update csidriver.yaml Default value should be true * Update controller.yaml Updated the controller file * Update csidriver.yaml Updated csidriver.yaml file * Update controller.yaml Done changes related to SCT * csm-replication: Removed init container usage (#244) * CSIUnity v2.8.0 release (#248) * Updated v2.8.0 unity driver version * addressed review comments on values.yaml file update * Added the Volumelimits for Powermax (#240) Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> * PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Adding k8 28 support for powermax (#253) * Adding K8s1.28 support for PowerFlex (#254) * K8s 1.28 support unity (#255) * support k8s v1.28 * support k8s v1.28 * minor typo fixed (#258) * PowerScale k8s-1.28 support (#260) * powerstore v2.8.0 & k8s 1.28 support (#262) * Adding volume limits support for CSIPowerstore * Adding volume limits support for CSIPowerstore * Adding maxPowerstoreVolumesPerNode in values.yaml inside container-storage-modules * Removed duplicate entry for X_CSI_DRIVER_NAME in node.yaml for CSI Powerstore (#273) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Update Authorization images version (#277) * Updated replication controller/replicator/migrator/rescanner SC image versions for CSM v1.8.0 (#278) * Updated csi-metadata-retriever version to 1.5.0 for CSI Powerstore (#280) * Update Observability image versions for CSM 1.8. (#281) * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * [charts/cosi] add COSI driver helm chart (#267) * add cosi driver helm chart * add cosi specific linters * remove go mod from cosi linters * fix typos * add cosi-ct-config.yaml * change workflow name * change log levels in cosi chart * update actions * change name from cosi-driver to cosi * add kubelinter for all charts * revert kubelinter * change log levels in chart's template --------- Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * [charts/container-storage-modules]: Uncomment Resiliency arguments (#287) * uncomment resiliency args * fix spacing * correct cosi log level setting (#288) * [charts/container-storage-modules]: new keys added for PowerFlex driver (#286) * PowerFlex new keys added * NFS ACL parameter removed (#289) * Adding new fields for powermax (#290) * bumped csi-volumegroup-snapshotter version (#292) * reset the CSM helm chart to main branch (#293) --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: chaganti-rajitha <111744573+chaganti-rajitha@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Co-authored-by: tssushma <98810659+tssushma@users.noreply.github.com> Co-authored-by: Santhosh Lakshmanan <111809751+santhoshatdell@users.noreply.github.com> Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Nitesh-Rewatkar <90749010+nitesh3108@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alikdell <52920355+alikdell@users.noreply.github.com> Co-authored-by: Don Khan <don.khan@dell.com> Co-authored-by: mdutka-dell <115641913+mdutka-dell@users.noreply.github.com> Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com> Co-authored-by: Rishabh Raj <120644626+rishabhatdell@users.noreply.github.com> Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: chaganti-rajitha <111744573+chaganti-rajitha@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Co-authored-by: tssushma <98810659+tssushma@users.noreply.github.com> Co-authored-by: Santhosh Lakshmanan <111809751+santhoshatdell@users.noreply.github.com> Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com> Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Nitesh-Rewatkar <90749010+nitesh3108@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alikdell <52920355+alikdell@users.noreply.github.com> Co-authored-by: Don Khan <don.khan@dell.com> Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: mdutka-dell <115641913+mdutka-dell@users.noreply.github.com> Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com> Co-authored-by: Rishabh Raj <120644626+rishabhatdell@users.noreply.github.com>
atye
added a commit
that referenced
this pull request
Nov 7, 2023
* PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Centralized images * minor typo fixed (#258) * powerstore v2.8.0 & k8s 1.28 support (#262) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * NFS ACL parameter removed (#289) * fix values --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof <florian.coulombel@dell.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com>
PeresKereotubo
pushed a commit
that referenced
this pull request
Nov 20, 2023
* PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Centralized images * minor typo fixed (#258) * powerstore v2.8.0 & k8s 1.28 support (#262) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * NFS ACL parameter removed (#289) * fix values --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof <florian.coulombel@dell.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com>
tdawe
pushed a commit
that referenced
this pull request
Dec 13, 2023
* PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Centralized images * minor typo fixed (#258) * powerstore v2.8.0 & k8s 1.28 support (#262) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * NFS ACL parameter removed (#289) * fix values --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof <florian.coulombel@dell.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com>
tdawe
added a commit
to tdawe/helm-charts-dec13
that referenced
this pull request
Dec 13, 2023
* [charts/csi-powerstore] Add PowerStore v2.9.0 support & sidecar version update (dell#303) * powerstore v2.9.0 support * sidecar version update * csi-powerflex v2.9.0 changes. (dell#308) * Bump up observability version for CSM 1.9 (dell#314) * Bump-up observability version for CSM 1.9 * add patch verb to powerscale * Add codeowners * update attacher sidecar for powerflex and powerstore. (dell#315) * [charts/csi-powerstore] Centralize all images to one place (dell#310) * One place to configure all images * Use latest versions of CSI official sidecars * Set Default container to the driver and avoid podmon logs by default * Fix annotation and remove driverVersion * [charts/csi-powerflex] Centralize all images to one place dell#263 (dell#311) * Centralize all images to one place * Use latest CSI sidecars and default container to driver * Add default annotation * Use bleeding edge helm validation action to workaround cosign issue * [charts/csi-powermax] Container images in one place (dell#309) * One place to configure all images * Fix the csi provisioner to use the full name * Use latest versions of CSI official sidecars * Removing linked proxy mode for proxy (dell#301) * Updatting latest sidecars for Powermax (dell#319) * [charts/powerscale] Powerscale driver images in one place (dell#313) * PowerFlex v2.8.0 release (dell#250) * PowerFlex SDC updated (dell#251) * Centralized images * minor typo fixed (dell#258) * powerstore v2.8.0 & k8s 1.28 support (dell#262) * update auth to 1.8 (dell#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (dell#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (dell#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (dell#261)" (dell#284) This reverts commit c1dd93b. * NFS ACL parameter removed (dell#289) * fix values --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof <florian.coulombel@dell.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> * [charts/csi-unity] Centralize all images to one place (dell#312) * PowerFlex v2.8.0 release (dell#250) * Centralized images * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (dell#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (dell#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (dell#261)" (dell#284) This reverts commit c1dd93b. * Bump sidecars versions * Bump up observability version for CSM 1.9 (dell#314) * Bump-up observability version for CSM 1.9 * add patch verb to powerscale * Add codeowners * update attacher sidecar for powerflex and powerstore. (dell#315) * [charts/csi-powerstore] Centralize all images to one place (dell#310) * One place to configure all images * Use latest versions of CSI official sidecars * Set Default container to the driver and avoid podmon logs by default * Fix annotation and remove driverVersion * [charts/csi-powerflex] Centralize all images to one place dell#263 (dell#311) * Centralize all images to one place * Use latest CSI sidecars and default container to driver * Add default annotation * Use bleeding edge helm validation action to workaround cosign issue * [charts/csi-powermax] Container images in one place (dell#309) * One place to configure all images * Fix the csi provisioner to use the full name * Use latest versions of CSI official sidecars * remove duplicate podmon image --------- Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof <florian.coulombel@dell.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: coulof <49235405+coulof@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> * [charts/csi-powerscale]: Remove duplicate images (dell#320) * remove duplicate images * auth 1.8 * New images location (dell#316) * [charts/csi-unity]: Update Unity version for CSM 1.9 (dell#322) * add unity v2.9.0 support * remove image description * bump up version (dell#323) * adding noderoot mount path (dell#324) * changed sdc version (dell#325) * sidecars version update for powerstore and powerflex (dell#326) * updating sidecar image version (dell#328) * Revert "changed sdc version (dell#325)" (dell#329) This reverts commit 543cbdc. * Update VGS version to 1.4.0 (dell#331) * Powerscale and sidecar version changes (dell#332) * Powerscale and sidecar version changes * csi-node-driver-registrar version changed to 2.9.1 * PowerMax 2.9.0 support (dell#321) * Updating latest sidecars for PowerMax (dell#334) * [charts/csi-powerflex] Added external access (dell#333) * external access powerflex * adding external access in csm chart * Update image tags for replication (dell#336) * Reset CSM to release v1.9.0 to main (dell#338) --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: coulof <49235405+coulof@users.noreply.github.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: Aaron Tye <tyeaaron@gmail.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof <florian.coulombel@dell.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <shayna_finocchiaro@dell.com> Co-authored-by: Luna Xu <10015938+xuluna@users.noreply.github.com> Co-authored-by: Adarsh Kumar Yadav <109620911+adarsh-dell@users.noreply.github.com> Co-authored-by: Peres Kereotubo <118770866+PeresKereotubo@users.noreply.github.com> Co-authored-by: Akshay Saini <109056238+AkshaySainiDell@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com>
atye
added a commit
that referenced
this pull request
Jan 11, 2024
* updated helm-validations file (#243) * Powerscale v2.8.0 release (#242) * Powerscale v2.8.0 release * Added suppoert container-storage-modules * Added suppoert container-storage-modules * recommiting branch due to checks not working * dummy commit to trigger the checks * StorageCapacityTracking feature changes (#236) * Update _helpers.tpl Added changes related to StorageCapacityTracking * Update controller.yaml Added changes related to StorageCapacityTracking * Update csidriver.yaml Added changes related to StorageCapacityTracking * Update values.yaml Added changes related to StorageCapacityTracking * Update _helpers.tpl Storage Capacity Tracking K8s 1.24. * Update values.yaml Changed the version to v1.24 * Update csidriver.yaml Set the default value to false * Update controller.yaml Default value of the StorageCapacityTracking should be true * Update csidriver.yaml Default value should be true * Update controller.yaml Updated the controller file * Update csidriver.yaml Updated csidriver.yaml file * Update controller.yaml Done changes related to SCT * csm-replication: Removed init container usage (#244) * CSIUnity v2.8.0 release (#248) * Updated v2.8.0 unity driver version * addressed review comments on values.yaml file update * Added the Volumelimits for Powermax (#240) Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> * PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Adding k8 28 support for powermax (#253) * Adding K8s1.28 support for PowerFlex (#254) * K8s 1.28 support unity (#255) * support k8s v1.28 * support k8s v1.28 * minor typo fixed (#258) * PowerScale k8s-1.28 support (#260) * powerstore v2.8.0 & k8s 1.28 support (#262) * Adding volume limits support for CSIPowerstore * Adding volume limits support for CSIPowerstore * Adding maxPowerstoreVolumesPerNode in values.yaml inside container-storage-modules * Removed duplicate entry for X_CSI_DRIVER_NAME in node.yaml for CSI Powerstore (#273) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Update Authorization images version (#277) * Updated replication controller/replicator/migrator/rescanner SC image versions for CSM v1.8.0 (#278) * Updated csi-metadata-retriever version to 1.5.0 for CSI Powerstore (#280) * Update Observability image versions for CSM 1.8. (#281) * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * updated chart version (#282) * removed externalAccess from powerflex manifests. (#283) * [charts/cosi] add COSI driver helm chart (#267) * add cosi driver helm chart * add cosi specific linters * remove go mod from cosi linters * fix typos * add cosi-ct-config.yaml * change workflow name * change log levels in cosi chart * update actions * change name from cosi-driver to cosi * add kubelinter for all charts * revert kubelinter * change log levels in chart's template --------- Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * [charts/container-storage-modules]: Uncomment Resiliency arguments (#287) * uncomment resiliency args * fix spacing * correct cosi log level setting (#288) * [charts/container-storage-modules]: new keys added for PowerFlex driver (#286) * PowerFlex new keys added * NFS ACL parameter removed (#289) * Adding new fields for powermax (#290) * bumped csi-volumegroup-snapshotter version (#292) * reset the CSM helm chart to main branch (#293) * [feature-885]: Add workflow update to release branch (#297) * Update CODEOWNERS (#238) * add powerflex helm chart changes for v1.7.1. (#245) * add powerflex helm chart changes for v1.7.1. * add helm chart changes for v1.7.1 release. (#246) * Skip existing releases (#272) * updated helm-validations file (#243) * Powerscale v2.8.0 release (#242) * Powerscale v2.8.0 release * Added suppoert container-storage-modules * Added suppoert container-storage-modules * recommiting branch due to checks not working * dummy commit to trigger the checks * StorageCapacityTracking feature changes (#236) * Update _helpers.tpl Added changes related to StorageCapacityTracking * Update controller.yaml Added changes related to StorageCapacityTracking * Update csidriver.yaml Added changes related to StorageCapacityTracking * Update values.yaml Added changes related to StorageCapacityTracking * Update _helpers.tpl Storage Capacity Tracking K8s 1.24. * Update values.yaml Changed the version to v1.24 * Update csidriver.yaml Set the default value to false * Update controller.yaml Default value of the StorageCapacityTracking should be true * Update csidriver.yaml Default value should be true * Update controller.yaml Updated the controller file * Update csidriver.yaml Updated csidriver.yaml file * Update controller.yaml Done changes related to SCT * csm-replication: Removed init container usage (#244) * CSIUnity v2.8.0 release (#248) * Updated v2.8.0 unity driver version * addressed review comments on values.yaml file update * Added the Volumelimits for Powermax (#240) Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> * PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Adding k8 28 support for powermax (#253) * Adding K8s1.28 support for PowerFlex (#254) * K8s 1.28 support unity (#255) * support k8s v1.28 * support k8s v1.28 * minor typo fixed (#258) * PowerScale k8s-1.28 support (#260) * powerstore v2.8.0 & k8s 1.28 support (#262) * Adding volume limits support for CSIPowerstore * Adding volume limits support for CSIPowerstore * Adding maxPowerstoreVolumesPerNode in values.yaml inside container-storage-modules * Removed duplicate entry for X_CSI_DRIVER_NAME in node.yaml for CSI Powerstore (#273) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Alik Saring <alik.saring@dell.com> * Update Authorization images version (#277) * Updated replication controller/replicator/migrator/rescanner SC image versions for CSM v1.8.0 (#278) * Updated csi-metadata-retriever version to 1.5.0 for CSI Powerstore (#280) * Update Observability image versions for CSM 1.8. (#281) * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * [charts/cosi] add COSI driver helm chart (#267) * add cosi driver helm chart * add cosi specific linters * remove go mod from cosi linters * fix typos * add cosi-ct-config.yaml * change workflow name * change log levels in cosi chart * update actions * change name from cosi-driver to cosi * add kubelinter for all charts * revert kubelinter * change log levels in chart's template --------- Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b. * [charts/container-storage-modules]: Uncomment Resiliency arguments (#287) * uncomment resiliency args * fix spacing * correct cosi log level setting (#288) * [charts/container-storage-modules]: new keys added for PowerFlex driver (#286) * PowerFlex new keys added * NFS ACL parameter removed (#289) * Adding new fields for powermax (#290) * bumped csi-volumegroup-snapshotter version (#292) * reset the CSM helm chart to main branch (#293) --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: chaganti-rajitha <111744573+chaganti-rajitha@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Co-authored-by: tssushma <98810659+tssushma@users.noreply.github.com> Co-authored-by: Santhosh Lakshmanan <111809751+santhoshatdell@users.noreply.github.com> Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Nitesh-Rewatkar <90749010+nitesh3108@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alikdell <52920355+alikdell@users.noreply.github.com> Co-authored-by: Don Khan <don.khan@dell.com> Co-authored-by: mdutka-dell <115641913+mdutka-dell@users.noreply.github.com> Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com> Co-authored-by: Rishabh Raj <120644626+rishabhatdell@users.noreply.github.com> Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> --------- Signed-off-by: arnchiequ-dell <arnchie.qu@dell.com> Co-authored-by: chaganti-rajitha <111744573+chaganti-rajitha@users.noreply.github.com> Co-authored-by: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Co-authored-by: tssushma <98810659+tssushma@users.noreply.github.com> Co-authored-by: Santhosh Lakshmanan <111809751+santhoshatdell@users.noreply.github.com> Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com> Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com> Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Nitesh-Rewatkar <90749010+nitesh3108@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring <alik.saring@dell.com> Co-authored-by: alikdell <52920355+alikdell@users.noreply.github.com> Co-authored-by: Don Khan <don.khan@dell.com> Co-authored-by: Niranjan N <110103643+niranjan-n1@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: mdutka-dell <115641913+mdutka-dell@users.noreply.github.com> Co-authored-by: mtrajfacki-dell <115641731+mtrajfacki-dell@users.noreply.github.com> Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com> Co-authored-by: Rishabh Raj <120644626+rishabhatdell@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.
Is this a new chart?
No
What this PR does / why we need it:
Its about a minor typo fix.
Which issue(s) is this PR associated with:
Special notes for your reviewer:
Checklist:
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[charts_dir/mychartname]
) if applicable