Skip to content
This repository has been archived by the owner on May 3, 2020. It is now read-only.

Update docker image #513

Merged
merged 3 commits into from
Nov 23, 2018
Merged

Update docker image #513

merged 3 commits into from
Nov 23, 2018

Conversation

alxbl
Copy link
Contributor

@alxbl alxbl commented Oct 5, 2018

This pull request completely revamps the Dockerfile to use ruby 2.3.5 as requested and simplifies the image building process. It also includes the documentation for building Serpico containers and initial support for decoupling the database file from the container itself.

Currently, it's still missing a way to expose the certificates and configuration outside of the container so that a containerized instance of Serpico can run from an existing database and configuration. If there's enough people interested, I will work on that next.

Cheers,

@BuffaloWill
Copy link
Contributor

@alxbl Thank you!! My only blocker to merging this is the bump in Ruby version in the Gemfile. I would like to keep it at 2.3.3. We typically do not change Ruby versions unless there is a significant need (i.e. security issue, critical compatibility issue). Thoughts?

@alxbl
Copy link
Contributor Author

alxbl commented Nov 23, 2018

I don't mind leaving it at 2.3.3, I updated to 2.3.5 because of #388. If you'd like to handle that in a future version, I can update this PR to stick to 2.3.3.

Let me know.

@BuffaloWill
Copy link
Contributor

Ok, great. Let's stick with 2.3.3 and get it bumped later.

That was my bad in #388. IIRC originally we were on 2.3.5, but due to incompatibility in the Windows build we had to drop down to 2.3.3.

@alxbl
Copy link
Contributor Author

alxbl commented Nov 23, 2018

I see, good to know. I'll make the modification in a few minutes.

@alxbl
Copy link
Contributor Author

alxbl commented Nov 23, 2018

I've reverted both PRs to 2.3.3.

Cheers,

@BuffaloWill BuffaloWill merged commit bc258c5 into SerpicoProject:master Nov 23, 2018
@BuffaloWill
Copy link
Contributor

Thanks @alxbl!

@BuffaloWill
Copy link
Contributor

Side note, I updated the wiki page to reflect the new documentation.

https://github.com/SerpicoProject/Serpico/wiki/Running-Serpico-From-Docker

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

Successfully merging this pull request may close these issues.

2 participants