CSM helm charts 1.7 #197

niranjan-n1 · 2023-04-03T13:19:46Z

Is this a new chart?

Yes

What this PR does / why we need it:

Added helm charts for csi-unity, csi-powerscale, csi-poweflex
Reverted namespace template changes back to Release.namespace

Which issue(s) is this PR associated with:

[FEATURE]: CSM Installation Wizard support for CSI PowerScale and PowerFlex and Unity drivers and modules through Helm csm#698

Special notes for your reviewer:

Checklist:

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

Chart Version bumped

Screenshots:

* added the custom namespace * Edited the copyright year * removed charts and addressed the review comments * removed charts and chart.lock * renamed the chart and addressed review comments for observability * Addressed review comments for authorization and renamed csm chart * updated replication module chart version * reverted replicator version * deleted authorization chart

…nds (#172)

…M installation Wizard (#174) * Added powermax, authorization and updated charts with latest changes * changed kubeversion in powerstore * Updated charts with latest changes * updated the charts with latest changes from respective repos * updated the charts with latest changes * updated the charts with latest changes * updated charts with new versions * updated powermax, replication with latest changes and removed authorization from csm chart

* Update mailing list (#169) * update mailing list * upgrade checkout action to v3 because of node 12 deprecation * remove url policies * remove powerscale volume create --------- Co-authored-by: shaynafinocchiaro <[email protected]>

* Update mailing list (#169) * update mailing list * upgrade checkout action to v3 because of node 12 deprecation * support for sdc-approve policy * removed url.rego and url_test.rego --------- Co-authored-by: shaynafinocchiaro <[email protected]>

…185)

…es (#184)

Signed-off-by: arnchiequ-dell <[email protected]>

shefali-malhotra

Please Share the test performed for PowerScale (if any) along with the results.

niranjan-n1 · 2023-04-10T05:08:36Z

Please Share the test performed for PowerScale (if any) along with the results.

I have updated the PR with the screenshots

chaganti-rajitha

LGTM

meggm

LGTM

tdawe · 2023-04-12T16:13:39Z

charts/container-storage-modules/Chart.yaml


 dependencies:
 - name: csi-powerstore
  version: 2.6.0
  repository: file://../csi-powerstore
  condition: csi-powerstore.enabled

+- name: csi-powermax
+  version: 2.6.0
+  repository: file://../csi-powermax


This is fine for testing, but when the chart is released it will need to contain the full path to our helm repository. See example: #195

tdawe · 2023-04-12T16:15:06Z

charts/container-storage-modules/Chart.yaml

@@ -30,25 +30,45 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.1.0


Any updates to the umbrella container-storage-modules chart will need to be merged into the main branch separately, after all of the other charts are released. If not, this will prevent proper packaging of the dependencies.

@niranjan-n1 You may want to keep changes for this umbrella chart in a separate release branch, so this can be released after all the dependencies are released. The release-v1.7.0 branch is common to all the driver and module releases.

@tdawe @sharmilarama We are planning to keep the chart in the same branch until the code freeze and remove the charts before code-freeze and merge it to main after all the charts are merged. Since, we have requested the respective teams to test the charts it will be difficult to manage with 2 branches.

* [charts/csm-authorization]: feature-725: Remove tenant service ingress (#196) * remove tenant service ingress * remove tenant service ingress * update chart version * Ingress removal storage service (#198) * remove tenant service ingress * remove tenant service ingress * update chart version * removed storage service Ingress * removed storage service from ingress * updating storage-service.yaml * Trigger Build * Trigger Build * Update NOTES.txt * Update ingress.yaml * added release workflow --------- Co-authored-by: atye <[email protected]> * updated CodeOwners (#201) * [csm-authorization/csm-authorization] Remove role-service ingress (#203) * removed storage service Ingress * remove role svc ingress * merge with storage --------- Co-authored-by: root <[email protected]> * CSM helm charts 1.7 (#197) * add support for pmax topology (#168) * Support Observability for PowerMax (#166) * Helm chart changes for CSM installation Wizard (#167) * added the custom namespace * Edited the copyright year * removed charts and addressed the review comments * removed charts and chart.lock * renamed the chart and addressed review comments for observability * Addressed review comments for authorization and renamed csm chart * updated replication module chart version * reverted replicator version * deleted authorization chart * Rename arrayCapacityPollFrequencySeconds to capacityPollFrequencySeconds (#172) * Rename the performance poll frequency (#173) * Powerstore, Powermax and their supported modules chart changes for CSM installation Wizard (#174) * Added powermax, authorization and updated charts with latest changes * changed kubeversion in powerstore * Updated charts with latest changes * updated the charts with latest changes from respective repos * updated the charts with latest changes * updated the charts with latest changes * updated charts with new versions * updated powermax, replication with latest changes and removed authorization from csm chart * [charts/csm-authorization]: Remove URL policies (#176) * Update mailing list (#169) * update mailing list * upgrade checkout action to v3 because of node 12 deprecation * remove url policies * remove powerscale volume create --------- Co-authored-by: shaynafinocchiaro <[email protected]> * update policies for zero quota (#182) * support for sdc-approve policy (#180) * Update mailing list (#169) * update mailing list * upgrade checkout action to v3 because of node 12 deprecation * support for sdc-approve policy * removed url.rego and url_test.rego --------- Co-authored-by: shaynafinocchiaro <[email protected]> * sidecar version updates in powermax, powerstore and csm (#183) * Update Application Mobility chart with new image and chart versions (#185) * Updated Powerstore, Powermax and Replication charts with latest changes (#184) * updated chart version for app-mobility in CSM charts (#189) * updated Powerstore helm charts (#190) * Image and chart version bumped up. (#186) * Update release to 1.6.0 for auth (#188) Signed-off-by: arnchiequ-dell <[email protected]> * Bump up images for Observability (#191) * Adding Powerscale helm charts * added helm cahrt for csi-unity * updated values.yaml * updated values.yaml * added powerflex helm charts * updated powerscale and powerflex helm charts * reverted namespace changes * replaced custom namespace with Release namespace * replaced all custom namespace with Release namespace * test lint * resolved conflicts * minor corrections * minor corrections * added whitespaces * Update helm-validations.yml --------- Signed-off-by: arnchiequ-dell <[email protected]> Co-authored-by: YianZong <[email protected]> Co-authored-by: baoy1 <[email protected]> Co-authored-by: chaganti-rajitha <[email protected]> Co-authored-by: taohe1012 <[email protected]> Co-authored-by: P-Cao <[email protected]> Co-authored-by: Aaron Tye <[email protected]> Co-authored-by: shaynafinocchiaro <[email protected]> Co-authored-by: ashleyvjoy <[email protected]> Co-authored-by: Trevor Dawe <[email protected]> Co-authored-by: alexemc <[email protected]> Co-authored-by: aqu-dell <[email protected]> Co-authored-by: chaganti-rajitha <[email protected]> * Change K8s registry to registry.k8s.io (#200) * Update cert-manager to 1.11.0 (#204) Signed-off-by: arnchiequ-dell <[email protected]> * Adding k8 27 support for Powermax (#205) * Adding k8s 1.27 Support for PowerStore (#206) * powerstore changes for k8s 1.27 * updating helpers.tpl * fixed merge conflict * [charts/csi-unity] : 2.7.0 csi-unity support for helm in csm-installation-wizard (#208) * Added support for csi-unity driver 2.7.0 * Updated driver version to 2.7.0 * [charts/csm-replication]: Updated image versions (#207) * [charts/csi-vxflexos]:2.7.0 csi-vxflexos support for helm in csm-installation-wizard (#210) * Add support for csi-powerflex driver 2.7.0 * address review comments. * removed unused files and dependencies (#211) * [charts/csi-isilon]Added Powerscale 2.7.0 Support (#212) * add powerscale 2.7.0 support * Add enable storage capacity feature in csm values.yaml * changes for powerstore 2.7 (#213) * Updating auto proxy code (#214) * Upgradring driver and Chart version (#215) * update to 1.7 (#216) Co-authored-by: Luna Xu <[email protected]> * [charts/csi-powerstore] Sidecars and csi-metadata-retriever version update (#217) * sidecar version update * added csi-metadata-retriever * Update powerscale driver sidecars (#218) * container-storage-modules chart versions update (#220) * removed unused files and dependencies * updated values.yaml * updated values.yaml * Update latest sidecars in the unity driver (#219) * Updated latest sidecars and image versions * Updated latest sidecars and image versions * sidecar versions updated for powerflex driver (#221) * sidecar versions updated for powerflex driver * Remove container storage modules chart (#222) * removed unused files and dependencies * removed container-storage-module helm chart * rebased the container-storage-modules chart with main branch --------- Signed-off-by: arnchiequ-dell <[email protected]> Co-authored-by: Aaron Tye <[email protected]> Co-authored-by: IsaiasA1 <[email protected]> Co-authored-by: Niranjan N <[email protected]> Co-authored-by: Luna Xu <[email protected]> Co-authored-by: root <[email protected]> Co-authored-by: YianZong <[email protected]> Co-authored-by: baoy1 <[email protected]> Co-authored-by: chaganti-rajitha <[email protected]> Co-authored-by: taohe1012 <[email protected]> Co-authored-by: P-Cao <[email protected]> Co-authored-by: shaynafinocchiaro <[email protected]> Co-authored-by: ashleyvjoy <[email protected]> Co-authored-by: Trevor Dawe <[email protected]> Co-authored-by: alexemc <[email protected]> Co-authored-by: chaganti-rajitha <[email protected]> Co-authored-by: boyamurthy <[email protected]> Co-authored-by: Rishabh Raj <[email protected]> Co-authored-by: karthikk92 <[email protected]> Co-authored-by: Santhosh Lakshmanan <[email protected]> Co-authored-by: VamsiSiddu-7 <[email protected]> Co-authored-by: Chiman Jain <[email protected]> Co-authored-by: Luna Xu <[email protected]> Co-authored-by: Surya Gupta <[email protected]> Co-authored-by: Rajshree Khare <[email protected]>

YianZong and others added 30 commits January 17, 2023 11:48

add support for pmax topology (#168)

bed523d

Support Observability for PowerMax (#166)

c79f12f

Rename arrayCapacityPollFrequencySeconds to capacityPollFrequencySeco…

8d4295d

…nds (#172)

Rename the performance poll frequency (#173)

9c24724

update policies for zero quota (#182)

ed2278c

support for sdc-approve policy (#180)

01123dd

* Update mailing list (#169) * update mailing list * upgrade checkout action to v3 because of node 12 deprecation * support for sdc-approve policy * removed url.rego and url_test.rego --------- Co-authored-by: shaynafinocchiaro <[email protected]>

sidecar version updates in powermax, powerstore and csm (#183)

acc8928

Update Application Mobility chart with new image and chart versions (#…

6b8529c

…185)

Updated Powerstore, Powermax and Replication charts with latest chang…

cbca6de

…es (#184)

updated chart version for app-mobility in CSM charts (#189)

4a6b8af

updated Powerstore helm charts (#190)

9d88008

Image and chart version bumped up. (#186)

0325e99

Update release to 1.6.0 for auth (#188)

1f027a9

Signed-off-by: arnchiequ-dell <[email protected]>

Bump up images for Observability (#191)

8b64fbf

Adding Powerscale helm charts

4b3b5cf

added helm cahrt for csi-unity

7c33914

updated values.yaml

0fd6781

updated values.yaml

d4d5240

added powerflex helm charts

5323c44

updated powerscale and powerflex helm charts

c9db529

reverted namespace changes

ec6bf34

replaced custom namespace with Release namespace

fb55b57

replaced all custom namespace with Release namespace

5e71f7a

resolved conflicts

76e9590

test lint

0aec5dc

resolved conflicts

9ce420e

minor corrections

e043760

niranjan-n1 requested review from taohe1012, P-Cao, baoy1, YianZong and forrestxia as code owners April 3, 2023 13:19

niranjan-n1 assigned chaganti-rajitha Apr 3, 2023

niranjan-n1 requested a review from Prabhu-Dell April 3, 2023 13:22

niranjan-n1 changed the title ~~Csm helm charts 1.7~~ CSM helm charts 1.7 Apr 4, 2023

niranjan-n1 added 2 commits April 5, 2023 18:05

Merge branch 'release-v1.7.0' into csm-helm-charts-1.7

4985369

Update helm-validations.yml

03704af

niranjan-n1 requested review from AkshaySainiDell and adarsh-dell April 6, 2023 08:27

shefali-malhotra reviewed Apr 6, 2023

View reviewed changes

niranjan-n1 requested a review from shefali-malhotra April 10, 2023 05:08

Merge branch 'release-v1.7.0' into csm-helm-charts-1.7

d5170b7

niranjan-n1 assigned niranjan-n1 and unassigned niranjan-n1 and chaganti-rajitha Apr 12, 2023

niranjan-n1 requested review from meggm and chaganti-rajitha April 12, 2023 12:07

Merge branch 'release-v1.7.0' into csm-helm-charts-1.7

4fcc6aa

chaganti-rajitha approved these changes Apr 12, 2023

View reviewed changes

meggm approved these changes Apr 12, 2023

View reviewed changes

tdawe reviewed Apr 12, 2023

View reviewed changes

Merge branch 'release-v1.7.0' into csm-helm-charts-1.7

4b3912b

niranjan-n1 merged commit ac8378b into release-v1.7.0 Apr 13, 2023

niranjan-n1 deleted the csm-helm-charts-1.7 branch April 13, 2023 12:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CSM helm charts 1.7 #197

CSM helm charts 1.7 #197

niranjan-n1 commented Apr 3, 2023 •

edited by chaganti-rajitha

Loading

shefali-malhotra left a comment •

edited

Loading

niranjan-n1 commented Apr 10, 2023

chaganti-rajitha left a comment

meggm left a comment

tdawe Apr 12, 2023

tdawe Apr 12, 2023

sharmilarama Apr 12, 2023

niranjan-n1 Apr 13, 2023

CSM helm charts 1.7 #197

CSM helm charts 1.7 #197

Conversation

niranjan-n1 commented Apr 3, 2023 • edited by chaganti-rajitha Loading

Is this a new chart?

What this PR does / why we need it:

Which issue(s) is this PR associated with:

Special notes for your reviewer:

Checklist:

shefali-malhotra left a comment • edited Loading

Choose a reason for hiding this comment

niranjan-n1 commented Apr 10, 2023

chaganti-rajitha left a comment

Choose a reason for hiding this comment

meggm left a comment

Choose a reason for hiding this comment

tdawe Apr 12, 2023

Choose a reason for hiding this comment

tdawe Apr 12, 2023

Choose a reason for hiding this comment

sharmilarama Apr 12, 2023

Choose a reason for hiding this comment

niranjan-n1 Apr 13, 2023

Choose a reason for hiding this comment

niranjan-n1 commented Apr 3, 2023 •

edited by chaganti-rajitha

Loading

shefali-malhotra left a comment •

edited

Loading