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

docs: Document that `s3.regionEndpoint` requires a transport scheme #98

Closed 0xDEC0DE closed 4 months ago

0xDEC0DE commented 1 year ago

The regionEndpoint value does not work at all without one, so provide a nice hint that it is required in the vendor-provided materials.

Fixes Issue: #97

0xDEC0DE commented 1 year ago

As you can see, I consider this a documentation issue, not an issue with the chart itself. A simple hint like this would have steered me towards the correct configuration immediately.

0xDEC0DE commented 4 months ago

@vyas-n Apologies, my commit signing setup was a bit of a mess. That's all rectified now, and you ought be able to re-enable auto-merge for this PR.

vyas-n commented 4 months ago

Thanks @0xDEC0DE!