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 Docker instructions to Readme #267

Closed
cdevroe opened this issue Jun 24, 2020 · 6 comments · Fixed by #268
Closed

Add Docker instructions to Readme #267

cdevroe opened this issue Jun 24, 2020 · 6 comments · Fixed by #268
Assignees
Milestone

Comments

@cdevroe
Copy link
Owner

cdevroe commented Jun 24, 2020

The appropriate docker-compose files have been part of the repository and release build for months. After having moved between several systems and several platforms; Mac and Windows mostly, I've worked out most of the kinks I've personally come across.

So I'll be adding the instructions to the readme and will consider Docker fully supported as a way to run Unmark locally. In fact, it is the preferred way!

@cdevroe cdevroe added this to the 2020.3 milestone Jun 24, 2020
@cdevroe cdevroe self-assigned this Jun 24, 2020
@cdevroe cdevroe linked a pull request Jun 24, 2020 that will close this issue
@runlevel-six
Copy link

This is likely because of something on my end, but with a fresh clone of unmark, using either the trunk or proposals/v2020.3 branch, with no existing docker volume, when I run docker-compose up -d and then navigate to http://localhost/setup (or just http://localhost) I get the following errors:

Warning: mysqli::real_connect(): (HY000/2002): No such file or directory in /var/www/html/system/database/drivers/mysqli/mysqli_driver.php on line 203

Warning: Cannot modify header information - headers already sent by (output started at /var/www/html/system/database/drivers/mysqli/mysqli_driver.php:203) in /var/www/html/system/core/Common.php on line 570

A Database Error Occurred
Unable to connect to your database server using the provided settings.

Filename: controllers/Install.php

Line Number: 8

That hasn't been an error I've seen when not running it containerized - figured I'd drop this here in case it was something outside of my experience whose fix is just an additional step in the instructions.

@cdevroe
Copy link
Owner Author

cdevroe commented Oct 5, 2020

@runlevel-six Thanks for reporting the issue. I just added Unmark to a new computer and I followed the Readme instructions and didn't get this issue. I wonder if you may have an old config (which isn't replaced by downloading or upgrading Unmark)?

@runlevel-six
Copy link

runlevel-six commented Oct 5, 2020

@cdevroe thanks for the response.

It was a fresh install on a new Ubuntu 20.04 server (I am planning to migrate from an existing install on an older version of Ubuntu). I just cloned the repository, ran the docker-compose up -d command, and then got this error. I then stopped the stack, removed the containers and volume, checked out the proposals/v2020.3 branch, and tried it again with the same result.

Honestly, it sounds like I did the same thing as you. I'll dig a little deeper. It does sound like in general the instructions are fine as is from your testing, so I don't want to hijack the thread - I can create a new issue if I don't find a fix. If I do find a fix I'll update this just to tie up the loose end (especially if it does turn out relevant to the process).

@cdevroe
Copy link
Owner Author

cdevroe commented Oct 5, 2020

@runlevel-six I can also quickly destroy my local set up and try again as well. I'll see if I can do this very soon. Sorry you're having issues. The Docker container should be the easiest way to use Unmark locally.

@cdevroe
Copy link
Owner Author

cdevroe commented Oct 6, 2020

@runlevel-six I've found the issue, I believe. The database-sample.php file does not have the proper credentials for the included database using Docker Compose. So the error you're receiving is very misleading. I'm unsure how to correct that error - but I have been able to fix the issue which will be included in 2020.3 which will be released publicly today.

cdevroe added a commit that referenced this issue Oct 6, 2020
@runlevel-six
Copy link

@cdevroe that did the trick. Everything worked fine from a fresh install following the instructions. Thanks!

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

Successfully merging a pull request may close this issue.

2 participants