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

Unable to chose POSTGRES_USERNAME and POSTGRES_DB with manual installation #5612

Open
arthurpers86 opened this issue Nov 19, 2024 · 5 comments
Labels
1. to develop Accepted and waiting to be taken care of documentation Improvements or additions to documentation help wanted Extra attention is needed

Comments

@arthurpers86
Copy link

I am deploying Nextcloud using the manual-install and configuring it to use my own PostgreSQL database hosted on a separate VM.

I have created a database user in PostgreSQL and set the corresponding environment variable POSTGRES_USER with this username. However, this setup does not work as expected because the prefix "oc_" is added to the username here and here.

I also see that POSTGRES_USER and POSTGRES_DB is hardcoded here.

What is the purpose of adding this prefix?
Why are environment variables for POSTGRES_USER and POSTGRES_DB defined if their values cannot be changed?

@arthurpers86 arthurpers86 added the 0. Needs triage Pending approval or rejection. This issue is pending approval. label Nov 19, 2024
@szaimen
Copy link
Collaborator

szaimen commented Nov 19, 2024

Hi, this has technical and legacy reasons. I guess this should be documented though. Can you create a PR?

@szaimen szaimen added 1. to develop Accepted and waiting to be taken care of documentation Improvements or additions to documentation help wanted Extra attention is needed and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Nov 19, 2024
@arthurpers86
Copy link
Author

Hi, thanks for your answer. Do you expect only documentation for this issue?
Is it not possible to have a better separation between manual-installation and mastercontainer installation ?

@szaimen
Copy link
Collaborator

szaimen commented Nov 20, 2024

Do you expect only documentation for this issue?

Yes, this would be great.

Is it not possible to have a better separation between manual-installation and mastercontainer installation ?

I fear this is indeed not possible because the same containers are used in both installation methods

@arthurpers86
Copy link
Author

I rebuilt the nextcloud/aio-nextcloud and nextcloud/aio-notify-push images after removing those lines, and it works fine.

Would it not be possible to define these variables with the required prefix directly here ? I assume this would only affect mastercontainer.
And set it as default in the manual-install conf for the ones using nextcloud/aio-postgresql ?

@szaimen
Copy link
Collaborator

szaimen commented Nov 20, 2024

No, this is not possible because it breaks existing instances

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of documentation Improvements or additions to documentation help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants