goharbor / harbor-helm

The helm chart to deploy Harbor
Apache License 2.0
1.17k stars 760 forks source link

`registry-dpl` template container's args only coincidentally work. #1801

Closed aznashwan closed 2 weeks ago

aznashwan commented 1 month ago

The registry-dpl's template currently only happens to coincidentally work with the upstream goharbor/registry-photon image because:

NOTE: The upstream image's entrypoint.sh also runs install_cert.sh, which will basically gets "lost" in the Helm chart setup if we directly set command: /usr/bin/registry_DO_NOT_USE_GC in this chart.

Possible fixes:

IMO the set of possible fixes, from "most to least correct" would be:

aznashwan commented 1 month ago

IMO the set of possible fixes, from "most to least correct" would be:

making the entrypoint.sh from the upstream Dockerfile transparently pass all args it gets to /usr/bin/registry_DO_NOT_USE_GC, with it defaulting to serve /etc/registry/config.yml if none are given

I have taken the liberty to propose what I though was the best fix here: https://github.com/goharbor/harbor/pull/20778

MinerYang commented 1 month ago

https://github.com/goharbor/harbor/pull/20778#issuecomment-2270847050