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

Rollback otel-collecter image version to 0.42.0 #131

Merged
merged 1 commit into from
Jul 8, 2022
Merged

Rollback otel-collecter image version to 0.42.0 #131

merged 1 commit into from
Jul 8, 2022

Conversation

taohe1012
Copy link
Contributor

@taohe1012 taohe1012 commented Jul 5, 2022 •
edited by gallacher
Loading

Is this a new chart?

No

What this PR does / why we need it:

Rollback otel-collecter and nginxProxy image version to 0.42.0 and 1.20;
Rollback tls settings for otel in otel-collector-config.yaml.

Which issue(s) is this PR associated with:

Special notes for your reviewer:

  • This code change has been verified along with changes in karavi-metrics-powerflex PR#130
  • Rollback otel-collecter image version to 0.42.0, because we upgrade the OpenTelemetry to v1.7.0 in karavi-metrics-powerflex, and its async UpDownCounter is working well to fetch the “current value”.

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

sharmilarama
sharmilarama approved these changes Jul 6, 2022
View reviewed changes
Copy link
Collaborator

@sharmilarama sharmilarama left a comment

Choose a reason for hiding this comment

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

LGTM

@taohe1012 taohe1012 merged commit e7738dc into karavi-observability-release Jul 8, 2022
@taohe1012 taohe1012 deleted the feature-356-pflexMetrics-upgrade-otel branch July 8, 2022 08:43
forrestxia added a commit that referenced this pull request Sep 21, 2022
@forrestxia @hoppea2
@shaynafinocchiaro @sharmilarama @YianZong @taohe1012 @atye @EvgenyUglov @baoy1 @bjiang27
charts/karavi-observability Karavi observability release v1.3.0 (#143)
2154435 
* Update otel-collector-config.yaml

updated tls settings for otel

* Update Chart.yaml

increment helm chart version

* Update otel-collector-config.yaml

updating correct tls settings

* Update otel-collector-config.yaml

update tls settings

* update observability metric service immage taags for CSM .3 release (#124)

* align authorization sidecar with CSM 1.3 release (#126)

* add support for PowerScale topology (#123)

* Rollback otel-collecter image version to 0.42.0 (#131)

* Update code owners list (#132)

* Add more code owners

* chart/csm-authorization: Release v1.3.0 updates (#128)

* [charts/csm-authorization] feature-261: Authorization helm chart (#109)
* use 1.3.0 (#125)

Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

* Correct codeowners format (#133)

* add support for csm-metrics-powerscale (#129)

Add powerscale configuration to support csm-metrics-powerscale:
  - Add karavi-metrics-powerscale.yaml, karavi-metrics-powerscale.yaml;
  - Add pscale configuration to karavi-observability-configmap.yaml,
  values.yaml, cert-manager.yaml and NOTES.txt.

* Adjust PowerScale polling enable and frequency (#134)

The new enable and frequency are:
capacityMetricsEnabled: "true"
performanceMetricsEnabled: "true"
clusterCapacityPollFrequencySeconds: 30
clusterPerformancePollFrequencySeconds: 20
quotaCapacityPollFrequencySeconds: 30

Co-authored-by: Yiming Bao <yiming.bao@emc.com>

* enable authorization for csm observability powerscale (#135)

* Update Authorization sidecar image to 1.4.0 (#138)

* Update observability image tags for CSM 1.4 release (#142)

Co-authored-by: Yiming Bao <yiming.bao@dell.com>

Co-authored-by: hoppea2 <33433874+hoppea2@users.noreply.github.com>
Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com>
Co-authored-by: Sharmila Ramamoorthy <sharmila.ramamoorthy@dell.com>
Co-authored-by: YianZong <63342088+YianZong@users.noreply.github.com>
Co-authored-by: taohe1012 <88763781+taohe1012@users.noreply.github.com>
Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>
Co-authored-by: baoy1 <105041111+baoy1@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@emc.com>
Co-authored-by: bjiang27 <105997542+bjiang27@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@dell.com>
forrestxia added a commit that referenced this pull request Sep 21, 2022
@baoy1 @hoppea2
@shaynafinocchiaro @sharmilarama @YianZong @taohe1012 @forrestxia @atye @EvgenyUglov @bjiang27
charts/karavi-observability: Version fix for csm-metrics-powerscale (#…
405a4de 
…148)

* Update otel-collector-config.yaml

updated tls settings for otel

* Update Chart.yaml

increment helm chart version

* Update otel-collector-config.yaml

updating correct tls settings

* Update otel-collector-config.yaml

update tls settings

* update observability metric service immage taags for CSM .3 release (#124)

* align authorization sidecar with CSM 1.3 release (#126)

* add support for PowerScale topology (#123)

* Rollback otel-collecter image version to 0.42.0 (#131)

* Update code owners list (#132)

* Add more code owners

* chart/csm-authorization: Release v1.3.0 updates (#128)

* [charts/csm-authorization] feature-261: Authorization helm chart (#109)
* use 1.3.0 (#125)

Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

* Correct codeowners format (#133)

* add support for csm-metrics-powerscale (#129)

Add powerscale configuration to support csm-metrics-powerscale:
  - Add karavi-metrics-powerscale.yaml, karavi-metrics-powerscale.yaml;
  - Add pscale configuration to karavi-observability-configmap.yaml,
  values.yaml, cert-manager.yaml and NOTES.txt.

* Adjust PowerScale polling enable and frequency (#134)

The new enable and frequency are:
capacityMetricsEnabled: "true"
performanceMetricsEnabled: "true"
clusterCapacityPollFrequencySeconds: 30
clusterPerformancePollFrequencySeconds: 20
quotaCapacityPollFrequencySeconds: 30

Co-authored-by: Yiming Bao <yiming.bao@emc.com>

* enable authorization for csm observability powerscale (#135)

* Update Authorization sidecar image to 1.4.0 (#138)

* Update observability image tags for CSM 1.4 release (#142)

Co-authored-by: Yiming Bao <yiming.bao@dell.com>

* Update csm-metrics-powerscale to 1.0.0 (#147)

Update csm-metrics-powerscale to 1.0.0 to accommodate csm-metrics-powerscale release

Co-authored-by: hoppea2 <33433874+hoppea2@users.noreply.github.com>
Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com>
Co-authored-by: Sharmila Ramamoorthy <sharmila.ramamoorthy@dell.com>
Co-authored-by: YianZong <63342088+YianZong@users.noreply.github.com>
Co-authored-by: taohe1012 <88763781+taohe1012@users.noreply.github.com>
Co-authored-by: forrestxia <105040588+forrestxia@users.noreply.github.com>
Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@emc.com>
Co-authored-by: bjiang27 <105997542+bjiang27@users.noreply.github.com>
atye added a commit that referenced this pull request Dec 13, 2022
@atye @baoy1
@hoppea2 @shaynafinocchiaro @sharmilarama @YianZong @taohe1012 @forrestxia @EvgenyUglov @bjiang27 @tdawe
Rebase main (#161)
33a15f8 
* changes for 1.4.0 (#145) (#146)

* charts/karavi-observability: Version fix for csm-metrics-powerscale (#148)

* Update otel-collector-config.yaml

updated tls settings for otel

* Update Chart.yaml

increment helm chart version

* Update otel-collector-config.yaml

updating correct tls settings

* Update otel-collector-config.yaml

update tls settings

* update observability metric service immage taags for CSM .3 release (#124)

* align authorization sidecar with CSM 1.3 release (#126)

* add support for PowerScale topology (#123)

* Rollback otel-collecter image version to 0.42.0 (#131)

* Update code owners list (#132)

* Add more code owners

* chart/csm-authorization: Release v1.3.0 updates (#128)

* [charts/csm-authorization] feature-261: Authorization helm chart (#109)
* use 1.3.0 (#125)

Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

* Correct codeowners format (#133)

* add support for csm-metrics-powerscale (#129)

Add powerscale configuration to support csm-metrics-powerscale:
  - Add karavi-metrics-powerscale.yaml, karavi-metrics-powerscale.yaml;
  - Add pscale configuration to karavi-observability-configmap.yaml,
  values.yaml, cert-manager.yaml and NOTES.txt.

* Adjust PowerScale polling enable and frequency (#134)

The new enable and frequency are:
capacityMetricsEnabled: "true"
performanceMetricsEnabled: "true"
clusterCapacityPollFrequencySeconds: 30
clusterPerformancePollFrequencySeconds: 20
quotaCapacityPollFrequencySeconds: 30

Co-authored-by: Yiming Bao <yiming.bao@emc.com>

* enable authorization for csm observability powerscale (#135)

* Update Authorization sidecar image to 1.4.0 (#138)

* Update observability image tags for CSM 1.4 release (#142)

Co-authored-by: Yiming Bao <yiming.bao@dell.com>

* Update csm-metrics-powerscale to 1.0.0 (#147)

Update csm-metrics-powerscale to 1.0.0 to accommodate csm-metrics-powerscale release

Co-authored-by: hoppea2 <33433874+hoppea2@users.noreply.github.com>
Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com>
Co-authored-by: Sharmila Ramamoorthy <sharmila.ramamoorthy@dell.com>
Co-authored-by: YianZong <63342088+YianZong@users.noreply.github.com>
Co-authored-by: taohe1012 <88763781+taohe1012@users.noreply.github.com>
Co-authored-by: forrestxia <105040588+forrestxia@users.noreply.github.com>
Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@emc.com>
Co-authored-by: bjiang27 <105997542+bjiang27@users.noreply.github.com>

* [charts/csm-application-mobility]: Application Mobility v0.2.0 Release (#159)

* update authorization chart and image versions (#155)

Co-authored-by: baoy1 <105041111+baoy1@users.noreply.github.com>
Co-authored-by: hoppea2 <33433874+hoppea2@users.noreply.github.com>
Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com>
Co-authored-by: Sharmila Ramamoorthy <sharmila.ramamoorthy@dell.com>
Co-authored-by: YianZong <63342088+YianZong@users.noreply.github.com>
Co-authored-by: taohe1012 <88763781+taohe1012@users.noreply.github.com>
Co-authored-by: forrestxia <105040588+forrestxia@users.noreply.github.com>
Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@emc.com>
Co-authored-by: bjiang27 <105997542+bjiang27@users.noreply.github.com>
Co-authored-by: Trevor Dawe <trevor.dawe@dell.com>
sharmilarama added a commit that referenced this pull request Dec 13, 2022
@sharmilarama @hoppea2
@shaynafinocchiaro @YianZong @taohe1012 @forrestxia @atye @EvgenyUglov @baoy1 @bjiang27
charts/karavi-observability: CSM 1.5 release (#165)
8f92b22 
* Update otel-collector-config.yaml

updated tls settings for otel

* Update Chart.yaml

increment helm chart version

* Update otel-collector-config.yaml

updating correct tls settings

* Update otel-collector-config.yaml

update tls settings

* update observability metric service immage taags for CSM .3 release (#124)

* align authorization sidecar with CSM 1.3 release (#126)

* add support for PowerScale topology (#123)

* Rollback otel-collecter image version to 0.42.0 (#131)

* Update code owners list (#132)

* Add more code owners

* chart/csm-authorization: Release v1.3.0 updates (#128)

* [charts/csm-authorization] feature-261: Authorization helm chart (#109)
* use 1.3.0 (#125)

Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

Co-authored-by: sharmilarama <72404078+sharmilarama@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>

* Correct codeowners format (#133)

* add support for csm-metrics-powerscale (#129)

Add powerscale configuration to support csm-metrics-powerscale:
  - Add karavi-metrics-powerscale.yaml, karavi-metrics-powerscale.yaml;
  - Add pscale configuration to karavi-observability-configmap.yaml,
  values.yaml, cert-manager.yaml and NOTES.txt.

* Adjust PowerScale polling enable and frequency (#134)

The new enable and frequency are:
capacityMetricsEnabled: "true"
performanceMetricsEnabled: "true"
clusterCapacityPollFrequencySeconds: 30
clusterPerformancePollFrequencySeconds: 20
quotaCapacityPollFrequencySeconds: 30

Co-authored-by: Yiming Bao <yiming.bao@emc.com>

* enable authorization for csm observability powerscale (#135)

* Update Authorization sidecar image to 1.4.0 (#138)

* Update observability image tags for CSM 1.4 release (#142)

Co-authored-by: Yiming Bao <yiming.bao@dell.com>

* Update csm-metrics-powerscale to 1.0.0 (#147)

Update csm-metrics-powerscale to 1.0.0 to accommodate csm-metrics-powerscale release

* Update cert-manager to 1.10.0 for Observability (#151)

* update authorization sidecar image (#156)

* Update to latest images

Co-authored-by: hoppea2 <33433874+hoppea2@users.noreply.github.com>
Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com>
Co-authored-by: YianZong <63342088+YianZong@users.noreply.github.com>
Co-authored-by: taohe1012 <88763781+taohe1012@users.noreply.github.com>
Co-authored-by: forrestxia <105040588+forrestxia@users.noreply.github.com>
Co-authored-by: Aaron Tye <tyeaaron@gmail.com>
Co-authored-by: EvgenyUglov <63835199+EvgenyUglov@users.noreply.github.com>
Co-authored-by: baoy1 <105041111+baoy1@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@emc.com>
Co-authored-by: bjiang27 <105997542+bjiang27@users.noreply.github.com>
Co-authored-by: Yiming Bao <yiming.bao@dell.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baoy1 baoy1 baoy1 approved these changes

@forrestxia forrestxia forrestxia approved these changes

@sharmilarama sharmilarama sharmilarama approved these changes

@shaynafinocchiaro shaynafinocchiaro shaynafinocchiaro approved these changes

@alikdell alikdell alikdell approved these changes

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

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

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

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

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

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

@coulof coulof Awaiting requested review from coulof coulof 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
@taohe1012 @alikdell @shaynafinocchiaro @sharmilarama @forrestxia @baoy1