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/powerscale] Powerscale driver images in one place #313

Merged
merged 12 commits into from
Nov 7, 2023
Merged

[charts/powerscale] Powerscale driver images in one place #313

merged 12 commits into from
Nov 7, 2023

Conversation

atye
Copy link
Contributor

@atye atye commented Oct 20, 2023 •
edited by shaynafinocchiaro
Loading

Is this a new chart?

No

What this PR does / why we need it:

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

  • 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 Pscale driver images test [charts/powerscale] Pscale driver images test Oct 23, 2023
@coulof coulof changed the title [charts/powerscale] Pscale driver images test [charts/powerscale] Powerscale driver images in one place Oct 23, 2023
This was referenced Oct 23, 2023
Closed
Merged
shanduur-dell
shanduur-dell previously approved these changes Oct 26, 2023
View reviewed changes
gallacher
gallacher previously approved these changes Oct 31, 2023
View reviewed changes
coulof
coulof previously approved these changes Nov 7, 2023
View reviewed changes
Copy link
Collaborator

@coulof coulof left a comment

Choose a reason for hiding this comment

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

LGTM

KshitijaKakde and others added 10 commits November 7, 2023 10:38
@KshitijaKakde @atye
PowerFlex v2.8.0 release (#250)
41a3114
@KshitijaKakde @atye
PowerFlex SDC updated (#251)
d1c49be
@coulof @atye
Centralized images
f778ef1
@khareRajshree @atye
minor typo fixed (#258)
a79996b
@suryagupta4 @atye
powerstore v2.8.0 & k8s 1.28 support (#262)
5c2f6af
@aqu-dell @alikdell @atye
update auth to 1.8 (#271)
aa3eed7 
* 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>
@alankar-verma @suryagupta4 @atye
Adding CSI Powerstore driver’s effective pod security profile for sup…
d278fc8 
…porting ephemeral volumes as part of Openshift 4.13 support (#261)

Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com>
@VamsiSiddu-7 @atye
removed externalAccess from powerflex manifests. (#283)
a8a9eea
@alankar-verma @atye
Revert "Adding CSI Powerstore driver’s effective pod security profile…
b29d98c 
… for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284)

This reverts commit c1dd93b.
@khareRajshree @atye
NFS ACL parameter removed (#289)
73bd0ce
@atye atye dismissed stale reviews from shanduur-dell, coulof, and gallacher via 73bd0ce November 7, 2023 15:38
@atye atye force-pushed the pscale-driver-images-test branch from eb85059 to 73bd0ce Compare November 7, 2023 15:38
@atye atye force-pushed the pscale-driver-images-test branch from a28e0b1 to 3bd2301 Compare November 7, 2023 15:44
coulof
coulof approved these changes Nov 7, 2023
View reviewed changes
Copy link
Collaborator

@coulof coulof left a comment

Choose a reason for hiding this comment

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

LGTM

@atye atye merged commit 0bc8c31 into release-v1.9.0 Nov 7, 2023
4 checks passed
@atye atye deleted the pscale-driver-images-test branch November 7, 2023 15:59
PeresKereotubo pushed a commit that referenced this pull request Nov 20, 2023
@atye @KshitijaKakde
@coulof @khareRajshree @suryagupta4 @aqu-dell @alikdell @alankar-verma @VamsiSiddu-7 @PeresKereotubo
[charts/powerscale] Powerscale driver images in one place (#313)
83310b8 
* 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
@atye @KshitijaKakde
@coulof @khareRajshree @suryagupta4 @aqu-dell @alikdell @alankar-verma @VamsiSiddu-7 @tdawe
[charts/powerscale] Powerscale driver images in one place (#313)
7e9abd6 
* 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
@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 <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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shaynafinocchiaro shaynafinocchiaro shaynafinocchiaro approved these changes

@coulof coulof coulof approved these changes

@shanduur-dell shanduur-dell shanduur-dell left review comments

@gallacher gallacher gallacher left review comments

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

@alikdell alikdell Awaiting requested review from alikdell alikdell 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

@niranjan-n1 niranjan-n1 Awaiting requested review from niranjan-n1 niranjan-n1 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

@P-Cao P-Cao Awaiting requested review from P-Cao

@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

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

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

@shefali-malhotra shefali-malhotra Awaiting requested review from shefali-malhotra shefali-malhotra is a code owner

@panigs7 panigs7 Awaiting requested review from panigs7

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

@HarishH-DELL HarishH-DELL Awaiting requested review from HarishH-DELL HarishH-DELL 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.