search

twuni / docker-registry.helm

Helm chart for a Docker registry. Successor to stable/docker-registry chart.
Apache License 2.0
309 stars 146 forks source link

Add support for storage.redirect.disable=true #91

Open FredNass opened 1 year ago

FredNass commented 1 year ago

Hello,

Thank you for providing this Helm Chart to the community. We'd like to see support for storage.redirect.disable=true as it would help in situations where S3 storage is not directly accessible to clients. Sorry for not providing the PR as it's a little beyond my skills.

0xDEC0DE commented 1 year ago

Workaround

In your local config:

docker-registry:
  configData:
    storage:
      redirect:
        disable: true
FredNass commented 1 year ago

Yes we found out and it works as expected. Thing is it involves a post-install manual configuration when it could we be managed during the Helm installation. Not a big issue on our side since we made a new Helm Chart (depending on twuni's) and use these settings in its values.yaml but what if not...