Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Docker fix #102

Merged
merged 2 commits into from
May 11, 2018
Merged

Docker fix #102

merged 2 commits into from
May 11, 2018

Conversation

stukalin
Copy link
Contributor

@stukalin stukalin commented May 11, 2018

Fixes #98, #87, #78.
Addresses #96.

Apparently docker didn't like this osprey update. I rolled the recent updates back. I rolled back some more package updates because I am not sure that they didn't cause any more issues.
I also fixed the elastic search.

@stukalin stukalin requested a review from jkleinsc May 11, 2018 14:36
Copy link
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for the PR @stukalin!

@jkleinsc jkleinsc merged commit f46b2c9 into master May 11, 2018
@jkleinsc jkleinsc deleted the docker-fix branch May 11, 2018 15:29
@drakemedic
Copy link
Contributor

I don't think this has fixed the problems.
The ssl certificate generation still fails - here's the log

Attaching to hospitalrunserver_nginx_1
nginx_1 | Bootstrapping dependencies for Debian-based OSes...
nginx_1 | Get:1 http://security.debian.org jessie/updates InRelease [94.4 kB]
nginx_1 | Hit http://nginx.org jessie InRelease
nginx_1 | Ign http://deb.debian.org jessie InRelease
nginx_1 | Get:2 http://deb.debian.org jessie-updates InRelease [145 kB]
nginx_1 | Hit http://deb.debian.org jessie Release.gpg
nginx_1 | Hit http://deb.debian.org jessie Release
nginx_1 | Get:3 http://security.debian.org jessie/updates/main amd64 Packages [650 kB]
nginx_1 | Get:4 http://nginx.org jessie/nginx amd64 Packages [43.9 kB]
nginx_1 | Get:5 http://deb.debian.org jessie-updates/main amd64 Packages [23.0 kB]
nginx_1 | Get:6 http://deb.debian.org jessie/main amd64 Packages [9064 kB]
nginx_1 | Fetched 10.0 MB in 48s (208 kB/s)
nginx_1 | Reading package lists...
nginx_1 | W: Size of file /var/lib/apt/lists/deb.debian.org_debian_dists_jessie-updates_main_binary-amd64_Packages.gz is not what the server reported 23046 23088
nginx_1 | Reading package lists...
nginx_1 | Building dependency tree...
nginx_1 | Reading state information...
nginx_1 | augeas-lenses is already the newest version.
nginx_1 | libaugeas0 is already the newest version.
nginx_1 | ca-certificates is already the newest version.
nginx_1 | gcc is already the newest version.
nginx_1 | libffi-dev is already the newest version.
nginx_1 | python is already the newest version.
nginx_1 | python-dev is already the newest version.
nginx_1 | python-virtualenv is already the newest version.
nginx_1 | virtualenv is already the newest version.
nginx_1 | libssl-dev is already the newest version.
nginx_1 | openssl is already the newest version.
nginx_1 | openssl set to manually installed.
nginx_1 | 0 upgraded, 0 newly installed, 0 to remove and 46 not upgraded.
nginx_1 | Upgrading certbot-auto 0.12.0 to 0.24.0...
nginx_1 | Replacing certbot-auto...
nginx_1 | Creating virtual environment...
nginx_1 | Installing Python packages...
nginx_1 | Installation succeeded.
nginx_1 | Saving debug log to /var/log/letsencrypt/letsencrypt.log
nginx_1 | Plugins selected: Authenticator standalone, Installer None
nginx_1 | Obtaining a new certificate
nginx_1 | Performing the following challenges:
nginx_1 | http-01 challenge for www.resurge-hms.org
nginx_1 | Cleaning up challenges
nginx_1 | Problem binding to port 80: Could not bind to IPv4 or IPv6.

To get it running, you need to run the certificate generation separately:
docker run -it --rm -p 443:443 -p 80:80 --name certbot -v "hospitalrun-server/data/nginx/letsencrypt:/etc/letsencrypt" certbot/certbot certonly --standalone -d your.domain.com

And change nginx/Dockerfile so that:
RUN envsubst < /etc/nginx/conf.d/default.conf.tmpl > /etc/nginx/conf.d/default.conf
&& envsubst < /etc/nginx/conf.d/defaultssl.conf.tmpl > /etc/nginx/conf.d/defaultssl

reads:
RUN envsubst < /etc/nginx/conf.d/default.conf.tmpl > /etc/nginx/conf.d/default.conf
&& envsubst < /etc/nginx/conf.d/defaultssl.conf.tmpl > /etc/nginx/conf.d/defaultssl.conf

@stukalin
Copy link
Contributor Author

@drakemedic There is an entrypoint ENTRYPOINT /etc/nginx/entrypoint.sh. And the script then says
mv /etc/nginx/conf.d/defaultssl /etc/nginx/conf.d/defaultssl.conf \
If that works correctly or not is the another question. But as I suggested previously this should've been done on purpose.

@ghost
Copy link

ghost commented Oct 15, 2019

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@ghost ghost added the released label Oct 15, 2019
@ghost
Copy link

ghost commented Oct 15, 2019

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@ghost
Copy link

ghost commented Oct 20, 2019

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Docker-compose installation fails on VPS
3 participants