azimuth-cloud / capi-helm-charts

Apache License 2.0
17 stars 21 forks source link

Fix broken Calico metrics #401

Closed mkjpryor closed 2 months ago

mkjpryor commented 2 months ago

In 3.28 Calico starting creating the calico-typha-metrics service itself, breaking our existing metric integration. This PR fixes that, protects against similar things happening in the future and introduces a new test that would have caught the issue.

NOTE - the "deploy from previous tag" in the chart-upgrade test will fail because of this bug, so the failure of that test will not block merging.