[ksm] Fix statefulset tag #9832

celenechang · 2021-08-03T19:32:55Z

Fix tag reporting for statefulset metrics

Better user experience with standardized tags

Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
PR title must be written as a CHANGELOG entry (see why)
Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
PR must have changelog/ and integration/ labels attached

ahmed-mez

Nice catch! Thank you!

codecov · 2021-08-04T13:52:56Z

Codecov Report

Merging #9832 (760ad51) into master (3ee898a) will not change coverage.
The diff coverage is n/a.

Flag	Coverage Δ
kubernetes_state	`89.69% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

[ksm] fix statefulset tag

9410477

celenechang requested review from a team as code owners August 3, 2021 19:32

celenechang added the changelog/Fixed label Aug 3, 2021

ghost added the integration/kubernetes_state label Aug 3, 2021

ahmed-mez previously approved these changes Aug 4, 2021

View reviewed changes

celenechang added 2 commits August 4, 2021 09:22

Merge branch 'master' into celene/fix_ksm_tags

be5874d

fix test

760ad51

celenechang dismissed ahmed-mez’s stale review via 760ad51 August 4, 2021 13:48

ahmed-mez approved these changes Aug 4, 2021

View reviewed changes

celenechang merged commit c120b20 into master Aug 4, 2021

celenechang deleted the celene/fix_ksm_tags branch August 4, 2021 13:56