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

use a dummy database name for the default postgres database name #11262

Merged
merged 1 commit into from
Sep 18, 2018

Conversation

icewind1991
Copy link
Member

@icewind1991 icewind1991 commented Sep 17, 2018

For some reason the docker image does not setup the permissions correctly,
by using a different name the nextcloud installer will create the database instead
with the correct permissions

This fixes the drone postgres tests

For some reason the docker image does not setup the permissions correctly,
by using a different name the nextcloud installer will create the database instead
with the correct permissions

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Sep 17, 2018
@icewind1991 icewind1991 added this to the Nextcloud 15 milestone Sep 17, 2018
Copy link
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@icewind1991 thanks for looking into it. those failures were really annoying.

@blizzz blizzz merged commit 9bbb88a into master Sep 18, 2018
@blizzz blizzz deleted the drone-postgres-db-name branch September 18, 2018 12:00
@danxuliu
Copy link
Member

For some reason the docker image does not setup the permissions correctly

You can find an explanation in nextcloud/spreed#1205 ;-)

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

Successfully merging this pull request may close these issues.

4 participants