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

Fix Redis Unix socket support #944

Merged
merged 2 commits into from
Jan 10, 2020

Conversation

tortuetorche
Copy link
Contributor

Rebase and rewrite the @epma01 pull request, based on last Nextcloud Docker changes

See: #735

@J0WI
Copy link
Contributor

J0WI commented Jan 9, 2020

Please run update.sh and make sure that the CI passes.

@tortuetorche tortuetorche force-pushed the fix-redis-unix-socket branch from a503182 to 76747a9 Compare January 9, 2020 16:11
Rebase and rewrite the @epma01 pull request, based on last Nextcloud Docker changes
See: nextcloud#735

Signed-off-by: Tortue Torche <tortuetorche@users.noreply.github.com>
Signed-off-by: Tortue Torche <tortuetorche@users.noreply.github.com>
@tortuetorche tortuetorche force-pushed the fix-redis-unix-socket branch from 13080a6 to f57f293 Compare January 9, 2020 16:17
@tortuetorche
Copy link
Contributor Author

Hi @J0WI,

Please run update.sh and make sure that the CI passes.

I ran the update.sh script but there is still few Docker build images with ARCH=i386 who failed.
I don't think it's related to my pull request, see the Travis CI results for more details...

Have a good day,
Tortue Torche

@J0WI J0WI merged commit d9ab30a into nextcloud:master Jan 10, 2020
neffets pushed a commit to neffets/docker-nextcloud that referenced this pull request Jan 12, 2020
* nextcloud-docker/master:
  Fix Redis Unix socket support (nextcloud#944)
  Correct supervisord examples so that the container can start (nextcloud#925)
  Run update.sh (nextcloud#959)
  Run update.sh
  Upgrade Alpine to 3.11
  Run update.sh
  Run update.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants