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

Support setting database port #2300

Closed
jessebot opened this issue Sep 20, 2024 · 2 comments
Closed

Support setting database port #2300

jessebot opened this issue Sep 20, 2024 · 2 comments
Labels
enhancement feature: auto config (environment variables) Auto configuring via environment variables needs info Additional info needed to triage

Comments

@jessebot
Copy link
Contributor

Hoi nextcloud/docker friends!

I wanted to inquiry about adding some env vars for database ports? Happy to submit the PR for setting postgresql/mysql ports via env vars, but I'm not entirely sure if it's supported upstream in nextcloud/server. Either way, I wanted to kick off the conversation here, so we can then maybe support it upstream in nextcloud/helm.

Please also see nextcloud/helm#632 for more info.

Thanks and kind regards!

@joshtrichards
Copy link
Member

The port is just specified as part of the dbhost parameter in Nextcloud. I don't believe there's anything specific we need to do to support it in the image.

I haven't used an alternative port admittedly that I can recall, but I do use UNIX sockets a fair bit in my deploys. They're specified in a similar manner and I know MYSQL_HOST and POSTGRES_HOST already work well with the syntax for that.

@joshtrichards joshtrichards added enhancement needs info Additional info needed to triage feature: auto config (environment variables) Auto configuring via environment variables labels Sep 22, 2024
@jessebot
Copy link
Contributor Author

ok, great, than we can just document that upstream! There's no need to put any additional effort in here. Thanks for your prompt response! :) I'll close this now since it doesn't look like there's anything else needed, but feel free to reopen if you see it differently, and I'm happy to help out 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature: auto config (environment variables) Auto configuring via environment variables needs info Additional info needed to triage
Projects
None yet
Development

No branches or pull requests

2 participants