search

bitnami / charts-syncer

Tool for synchronizing Helm Chart repositories.
https://bitnami.com
Apache License 2.0
169 stars 31 forks source link

UTF-8 octet causes error in pushing to repo #215

Open jamesdavismf opened 2 months ago

jamesdavismf commented 2 months ago

Error syncing some charts from HELM to OCI repo - have seen with kuberhealthy being pulled from https://kuberhealthy.github.io/kuberhealthy/helm-repos and kube-prometheus-stack from https://prometheus-community.github.io/helm-charts

Error syncing charts: Failed to push Helm chart: failed to push Helm chart: cannot load values.yaml: error converting YAML to JSON: yaml: invalid trailing UTF-8 octet

Suspect that this is because a few characters within the values.yaml files, for example within the kuberhealthy values.yaml file contains μ

Sample config file: 

source:
  repo:
    kind: HELM
    url: https://kuberhealthy.github.io/kuberhealthy/helm-repos
target:
  repo: 
    kind: OCI
    url: oci://registry-1.docker.io/docker
charts:
  - kuberhealthy
TimMnz09 commented 1 month ago

I am facing the same issue, after upgrading from v0.20.3. @jamesdavismf did you find a workaround for this?

jamesdavismf commented 1 month ago

Glad not only me, the work around that, I have done is to use yq to change to Local; and push it up with helm on a failure, it's not pretty but it works.

On Tue, Aug 13, 2024 at 3:20 PM Tim M @.***> wrote:

I am facing the same issue, after upgrading from v0.20.3. @jamesdavismf https://github.com/jamesdavismf did you find a workaround for this?

— Reply to this email directly, view it on GitHub https://github.com/bitnami/charts-syncer/issues/215#issuecomment-2286380366, or unsubscribe https://github.com/notifications/unsubscribe-auth/APL3REFFRZQ2K5ZC76VBOETZRIIZLAVCNFSM6AAAAABKM57UMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWGM4DAMZWGY . You are receiving this because you were mentioned.Message ID: @.***>