Some helm charts are now published with values.schema.json - a json schema to validate the provided values - case in point, the opentelemetry-collector chart that @RLazarus was trying to import in our repository.
Given this chart has "additionalProperties": false, our CI adding the listeners proxy fixture to tests for every chart/deployment generates an error.
I would suggest that for now we can look if there is a .no-global-fixtures file and don't inject the global fixtures.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T340551 distributed tracing epic | |||
| Open | None | T320549 distributed tracing v0 [minimum viable] | |||
| Resolved | Clement_Goubert | T320552 Package OpenTelemetry Collector atop our own base Docker images | |||
| Resolved | RLazarus | T320564 OpenTelemetry Collector running as a DaemonSet on Wikikube | |||
| Resolved | RLazarus | T324117 Helmchart for OpenTelemetry Collector | |||
| Resolved | Joe | T337359 deployment-charts CI should allow opting out of default fixture injections |
Change 922793 had a related patch set uploaded (by Giuseppe Lavagetto; author: Giuseppe Lavagetto):
[operations/deployment-charts@master] Add the possibility to override CI settings using a .fixturesctl.yaml files
Change 922793 merged by Giuseppe Lavagetto:
[operations/deployment-charts@master] Add the possibility to override CI settings using a .fixturesctl.yaml files