Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
FredNass opened this issue Jan 10, 2023 · 2 comments
Open

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

FredNass opened this issue Jan 10, 2023 · 2 comments

Comments

@FredNass
Copy link

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
Copy link

0xDEC0DE commented Feb 2, 2023

Workaround

In your local config:

docker-registry:
  configData:
    storage:
      redirect:
        disable: true

@FredNass
Copy link
Author

FredNass commented Feb 2, 2023

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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants