Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom

[charts/csi-powerflex] Centralize all images to one place #263

Closed
wants to merge 3 commits into from
Closed

[charts/csi-powerflex] Centralize all images to one place #263

wants to merge 3 commits into from

Conversation

coulof
Copy link
Collaborator

@coulof coulof commented Aug 24, 2023

Is this a new chart?

No

What this PR does / why we need it:

This is related to PR: dell/csi-powerflex#207

Which issue(s) is this PR associated with:

  • KRV-12738

Special notes for your reviewer:

This template has been tested successfully on my lab

Checklist:

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

  • Chart Version bumped
  • Variables are documented in the chart README.md
  • Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

@coulof coulof changed the title [charts /csi-powerflex] Centralize all images to one place [charts/csi-powerflex] Centralize all images to one place Aug 24, 2023
niranjan-n1
niranjan-n1 reviewed Aug 24, 2023
View reviewed changes
charts/csi-vxflexos/values.yaml
replication: dellemc/dell-csi-replicator:v1.5.0
vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.2.0
podmon: dellemc/podmon:v1.6.0
authorization: dellemc/csm-authorization-sidecar:v1.7.0

Copy link
Contributor

Choose a reason for hiding this comment

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

These changes needs to be updated in https://github.com/dell/helm-charts/blob/release-v1.8.0/charts/container-storage-modules/values.yaml if applicable

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I know, that's why I have a WIP PR here : #285
I want to apply all the changes one shot

@coulof coulof mentioned this pull request Sep 1, 2023
Closed
3 tasks
@coulof coulof changed the base branch from release-v1.8.0 to release-v1.9.0 October 18, 2023 08:29
@coulof coulof changed the base branch from release-v1.9.0 to release-v1.8.0 October 18, 2023 08:29
@coulof
Copy link
Collaborator Author

coulof commented Oct 18, 2023

Replaced by #311

@coulof coulof closed this Oct 18, 2023
coulof added a commit that referenced this pull request Oct 31, 2023
@coulof
[charts/csi-powerflex] Centralize all images to one place #263 (#311)
91d94d2 
* 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
atye pushed a commit that referenced this pull request Nov 7, 2023
@coulof @atye
[charts/csi-powerflex] Centralize all images to one place #263 (#311)
b656c2b 
* 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
atye added a commit that referenced this pull request Nov 7, 2023
@atye @KshitijaKakde
@coulof @alankar-verma @suryagupta4 @VamsiSiddu-7 @panigs7
[charts/csi-unity] Centralize all images to one place (#312)
8621a19 
* PowerFlex v2.8.0 release (#250)

* Centralized images

* 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 <[email protected]>

* 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.

* Bump sidecars versions

* Bump up observability version for CSM 1.9 (#314)

* Bump-up observability version for CSM 1.9

* add patch verb to powerscale

* Add codeowners

* update attacher sidecar for powerflex and powerstore. (#315)

* [charts/csi-powerstore] Centralize all images to one place (#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 #263 (#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 (#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 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: alankar-verma <[email protected]>
Co-authored-by: Surya Gupta <[email protected]>
Co-authored-by: VamsiSiddu-7 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: panigs7 <[email protected]>
tdawe pushed a commit that referenced this pull request Dec 13, 2023
@coulof @tdawe
[charts/csi-powerflex] Centralize all images to one place #263 (#311)
032c152 
* 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
tdawe pushed a commit that referenced this pull request Dec 13, 2023
@atye @KshitijaKakde
@coulof @alankar-verma @suryagupta4 @VamsiSiddu-7 @panigs7 @tdawe
[charts/csi-unity] Centralize all images to one place (#312)
8eb4974 
* PowerFlex v2.8.0 release (#250)

* Centralized images

* 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 <[email protected]>

* 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.

* Bump sidecars versions

* Bump up observability version for CSM 1.9 (#314)

* Bump-up observability version for CSM 1.9

* add patch verb to powerscale

* Add codeowners

* update attacher sidecar for powerflex and powerstore. (#315)

* [charts/csi-powerstore] Centralize all images to one place (#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 #263 (#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 (#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 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: alankar-verma <[email protected]>
Co-authored-by: Surya Gupta <[email protected]>
Co-authored-by: VamsiSiddu-7 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: panigs7 <[email protected]>
tdawe added a commit to tdawe/helm-charts-dec13 that referenced this pull request Dec 13, 2023
@tdawe @suryagupta4
@VamsiSiddu-7 @panigs7 @coulof @boyamurthy @atye @KshitijaKakde @khareRajshree @aqu-dell @alikdell @alankar-verma @shaynafinocchiaro @xuluna @adarsh-dell @PeresKereotubo @AkshaySainiDell @HarishH-DELL
Release v1.9.0 (#1)
29b3515 
* [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 <[email protected]>

* Updating podmon to 1.7

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

* Update Observability values file for Authorization sidecar image version

---------

Signed-off-by: arnchiequ-dell <[email protected]>
Co-authored-by: Alik Saring <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: KshitijaKakde <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: Rajshree Khare <[email protected]>
Co-authored-by: Surya Gupta <[email protected]>
Co-authored-by: aqu-dell <[email protected]>
Co-authored-by: Alik Saring <[email protected]>
Co-authored-by: alankar-verma <[email protected]>
Co-authored-by: VamsiSiddu-7 <[email protected]>

* [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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: alankar-verma <[email protected]>
Co-authored-by: Surya Gupta <[email protected]>
Co-authored-by: VamsiSiddu-7 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: panigs7 <[email protected]>

* [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 <[email protected]>
Co-authored-by: Surya Gupta <[email protected]>
Co-authored-by: VamsiSiddu-7 <[email protected]>
Co-authored-by: panigs7 <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: boyamurthy <[email protected]>
Co-authored-by: Aaron Tye <[email protected]>
Co-authored-by: KshitijaKakde <[email protected]>
Co-authored-by: coulof <[email protected]>
Co-authored-by: Rajshree Khare <[email protected]>
Co-authored-by: aqu-dell <[email protected]>
Co-authored-by: Alik Saring <[email protected]>
Co-authored-by: alankar-verma <[email protected]>
Co-authored-by: shaynafinocchiaro <[email protected]>
Co-authored-by: Luna Xu <[email protected]>
Co-authored-by: Adarsh Kumar Yadav <[email protected]>
Co-authored-by: Peres Kereotubo <[email protected]>
Co-authored-by: Akshay Saini <[email protected]>
Co-authored-by: HarishH-DELL <[email protected]>
@coulof coulof deleted the pflex-driver-images branch April 17, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niranjan-n1 niranjan-n1 niranjan-n1 left review comments

@gallacher gallacher Awaiting requested review from gallacher gallacher is a code owner

@tdawe tdawe Awaiting requested review from tdawe tdawe is a code owner

@alikdell alikdell Awaiting requested review from alikdell alikdell is a code owner

@atye atye Awaiting requested review from atye atye is a code owner

@hoppea2 hoppea2 Awaiting requested review from hoppea2 hoppea2 is a code owner

@chaganti-rajitha chaganti-rajitha Awaiting requested review from chaganti-rajitha chaganti-rajitha is a code owner

@meggm meggm Awaiting requested review from meggm meggm is a code owner

@shaynafinocchiaro shaynafinocchiaro Awaiting requested review from shaynafinocchiaro shaynafinocchiaro is a code owner

@sharmilarama sharmilarama Awaiting requested review from sharmilarama sharmilarama is a code owner

@tssushma tssushma Awaiting requested review from tssushma tssushma is a code owner

@taohe1012 taohe1012 Awaiting requested review from taohe1012 taohe1012 is a code owner

@P-Cao P-Cao Awaiting requested review from P-Cao P-Cao is a code owner

@shanmydell shanmydell Awaiting requested review from shanmydell shanmydell is a code owner

@baoy1 baoy1 Awaiting requested review from baoy1 baoy1 is a code owner

@YianZong YianZong Awaiting requested review from YianZong YianZong is a code owner

@forrestxia forrestxia Awaiting requested review from forrestxia forrestxia is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@coulof @niranjan-n1