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.

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

add support for csm-metrics-powerscale #129

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

taohe1012
Copy link
Contributor

@taohe1012 taohe1012 commented Jun 28, 2022

Is this a new chart?

No

What this PR does / why we need it:

add support for 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.

Which issue(s) is this PR associated with:

Special notes for your reviewer:

  • This code change has been verified along with changes in csm-metrics-powerscale PR#2

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

@taohe1012 taohe1012 force-pushed the feature-343-PScaleMetrics branch 3 times, most recently from 5fe3597 to 266cccc Compare June 29, 2022 08:59
@taohe1012 taohe1012 changed the base branch from main to karavi-observability-release June 29, 2022 09:01
@taohe1012 taohe1012 force-pushed the feature-343-PScaleMetrics branch 3 times, most recently from bb8115a to e3dc4b8 Compare June 30, 2022 09:28
alikdell
alikdell previously approved these changes Jun 30, 2022
@taohe1012 taohe1012 force-pushed the feature-343-PScaleMetrics branch 3 times, most recently from 4e9429f to 939b879 Compare July 8, 2022 08:52
forrestxia
forrestxia previously approved these changes Jul 9, 2022
baoy1
baoy1 previously approved these changes Jul 11, 2022
Copy link
Collaborator

@baoy1 baoy1 left a comment

Choose a reason for hiding this comment

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

@taohe1012, This branch is out-of-date with the base branch

@taohe1012 taohe1012 dismissed stale reviews from baoy1 and forrestxia via 844ce9c July 11, 2022 02:53
@taohe1012
Copy link
Contributor Author

@taohe1012, This branch is out-of-date with the base branch

thanks, updated

forrestxia
forrestxia previously approved these changes Jul 11, 2022
baoy1
baoy1 previously approved these changes Jul 11, 2022
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.
@taohe1012 taohe1012 dismissed stale reviews from baoy1 and forrestxia via 9995312 July 11, 2022 07:34
@taohe1012 taohe1012 merged commit 822dbae into karavi-observability-release Jul 11, 2022
@taohe1012 taohe1012 deleted the feature-343-PScaleMetrics branch July 11, 2022 08:28
forrestxia added a commit that referenced this pull request Sep 21, 2022
* 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
…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
* 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
* 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants