Handling charts with invalid chart names

[MTRNord]

Sadly in the reality of the internet not everyone seems to follow the chart naming conventions. As a result of this I ran into:

invalid chart reference: invalid chart name 'forgejoRunner': a valid name must be lower case letters and numbers and MAY be separated with dashes (-)

That is https://github.com/Mint-System/Kubernetes-Build/tree/main/forgejoRunner which seemingly doesnt follow the expected naming conventions. The helm command however seems to accept this name just fine.

Is there any way to make this work? Or is the only way to fork that chart or ask for a fix?
I would have expected a "ignore validation" flag in the HelmRelease definition for this case.

[matheuscscp]

From this org name I can tell they have no GHCR packages, you can't push images with uppercase letters lol. Why use things from them? Do you really need it? Take it as a red flag hahaha

This does seem to be on our side, by the way:

source-controller/internal/helm/chart/builder.go

Line 87 in 047f21d

return fmt.Errorf("invalid chart name '%s': a valid name must be lower case letters and numbers and MAY be separated with dashes (-), slashes (/) or periods (.)", r.Name)

Not sure why this is needed

[hiddeco]

It's the general convention, and at some point, Helm did not like it while attempting to package the chart (or when consuming it in certain places).

Because of this, it is caught by the builder as the issue would otherwise (vaguely and unexpectedly) appear in other places.

[matheuscscp]

Because of this, it is caught by the builder as the issue would otherwise (vaguely and unexpectedly) appear in other places.

Debug UX is a reason strong enough to keep it for me 👍