Support supplying labels for kiali

General MR

Summary

Adds the ability to supply the kiali-required labels app and version via the umbrella chart.

Big Bang MR: big-bang/bigbang!4929 (closed)

Linked Issue

#104 (closed)

Upgrade Notices

N/A

assigned to @wyatt.fry

added statusreview label

tested, it's working

approved this merge request

added teamObservability label

requested review from @blairbowden

removed statusreview label

marked this merge request as draft

added statusdoing label

still need to update readme, changelog etc

added 1 commit

• 70d36269 - docs

Compare with previous version

reset approvals from @kylegalloway-seed by pushing to the branch

marked this merge request as ready

removed statusdoing label

added statusreview label

requested review from @steven.donald, @kliu, @jimmyungerman, @kscheunemann, and @piotr.machaj

added 1 commit

• 481d95f8 - housekeeping

Compare with previous version

changed the description

mentioned in merge request big-bang/bigbang!4917 (closed)

changed the description

Beginning review.

@wyatt.fry the helmrelease is failing on bigbang branch set-required-labels-for-kiali-eck-operator and eck-operator branch set-required-labels-for-kiali. The following error is shown:

│     message: "Helm install failed for release eck-operator/eck-operator-eck-operator                                                                                                                        │
│       with chart eck-operator@2.13.0-bb.3: 1 error occurred:\n\t* StatefulSet.apps                                                                                                                          │
│       \"elastic-operator\" is invalid: [metadata.labels: Invalid value: \"{{ .Chart.Name                                                                                                                    │
│       }}\": a valid label must be an empty string or consist of alphanumeric characters,                                                                                                                    │
│       '-', '_' or '.', and must start and end with an alphanumeric character (e.g.                                                                                                                          │
│       'MyValue',  or 'my_value',  or '12345', regex used for validation is '(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])?'),                                                                                  │
│       metadata.labels: Invalid value: \"{{ .Chart.AppVersion }}\": a valid label must                                                                                                                       │
│       be an empty string or consist of alphanumeric characters, '-', '_' or '.', and                                                                                                                        │
│       must start and end with an alphanumeric character (e.g. 'MyValue',  or 'my_value',                                                                                                                    │
│       \ or '12345', regex used for validation is '(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])?')]"

mentioned in merge request big-bang/bigbang!4929 (closed)

changed the description

approved this merge request

resolved all threads

@wyatt.fry bigbang pipeline for linked branch is failing. I also left some comments/suggestions there.

approved this merge request

resolved all threads

mentioned in commit 35acfbde

merged

mentioned in merge request big-bang/bigbang!4942 (merged)