Skip to content

Fail early if Chart catalog cannot be populated

Mitchell Nielsen requested to merge better-error-for-charts-path into master

Returns a failure when attempting to populate the Chart catalog. If it's empty, we return an error that no charts could be found.

This is to address some vague errors we've seen when testing locally, in a scenario where the Helm Charts may not have been retrieved using ./scripts/retrieve_gitlab_charts.sh.

Additionally, it's best not to ignore errors anyways. Even though the final Operator image has the charts built in, there are always edge cases that should be surfaced without getting a vague "nil object" error.

Changelog: fixed

Closes #311 (closed)

Related to #688

Edited by Mitchell Nielsen

Merge request reports