Rename arrayCapacityPollFrequencySeconds to capacityPollFrequencySeconds #172
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
taohe1012
requested review from
gallacher,
tdawe,
alikdell,
atye,
hoppea2,
coulof,
shaynafinocchiaro,
sharmilarama,
P-Cao,
baoy1,
YianZong and
forrestxia
as code owners
forrestxia
approved these changes
Jan 28, 2023
baoy1
approved these changes
Jan 30, 2023
IsaiasA1
added a commit
that referenced
this pull request
Mar 27, 2023
* 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) * Update values.yaml * Update Chart.yaml * Update Chart.yaml * Update Chart.yaml * Update values.yaml * Update Chart.yaml * Update Chart.yaml * Updaating Files * Linting more files * Updating helm-validation version --------- 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]>
niranjan-n1
added a commit
that referenced
this pull request
Apr 13, 2023
* 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]>
aqu-dell
added a commit
that referenced
this pull request
Jun 23, 2023
* [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]>
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:
Rename arrayCapacityPollFrequencySeconds to capacityPollFrequencySeconds to support capacity metrics for Observability PowerMax
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