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

Bump up observability version for CSM 1.9 #314

Merged
merged 3 commits into from
Oct 26, 2023
Merged

Bump up observability version for CSM 1.9 #314

merged 3 commits into from
Oct 26, 2023

Conversation

panigs7
Copy link
Contributor

@panigs7 panigs7 commented Oct 23, 2023

Is this a new chart?

No

What this PR does / why we need it:

This PR bumps up the csm observability versions for CSM 1.9

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

santhoshatdell
santhoshatdell previously approved these changes Oct 23, 2023
View reviewed changes
shefali-malhotra
shefali-malhotra previously approved these changes Oct 25, 2023
View reviewed changes
Copy link
Contributor

@shefali-malhotra shefali-malhotra left a comment

Choose a reason for hiding this comment

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

LGTM

@panigs7 panigs7 merged commit 3562928 into release-v1.9.0 Oct 26, 2023
4 checks passed
atye pushed a commit that referenced this pull request Nov 7, 2023
@panigs7 @atye
Bump up observability version for CSM 1.9 (#314)
dfa6a4a 
* Bump-up observability version for CSM 1.9

* add patch verb to powerscale

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

* 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 <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>
PeresKereotubo pushed a commit that referenced this pull request Nov 20, 2023
@panigs7 @PeresKereotubo
Bump up observability version for CSM 1.9 (#314)
bc10e3d 
* Bump-up observability version for CSM 1.9

* add patch verb to powerscale

* Add codeowners
tdawe pushed a commit that referenced this pull request Dec 13, 2023
@panigs7 @tdawe
Bump up observability version for CSM 1.9 (#314)
27ef03e 
* Bump-up observability version for CSM 1.9

* add patch verb to powerscale

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

* 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 <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>
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>
@anandrajak1 anandrajak1 deleted the bump-up-observability-version branch October 17, 2024 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@santhoshatdell santhoshatdell santhoshatdell approved these changes

@niranjan-n1 niranjan-n1 niranjan-n1 approved these changes

@nitesh3108 nitesh3108 nitesh3108 approved these changes

@rajendraindukuri rajendraindukuri rajendraindukuri approved these changes

@shefali-malhotra shefali-malhotra shefali-malhotra 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

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

@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

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

Successfully merging this pull request may close these issues.
6 participants
@panigs7 @rajendraindukuri @nitesh3108 @shefali-malhotra @niranjan-n1 @santhoshatdell