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 Dockerfile and deployment instructions #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ffevotte
Copy link

@ffevotte ffevotte commented Dec 8, 2020

I've been using a docker + docker-compose setup to deploy LocalPackageServer instances in my local network, and thought it could be of relatively general use (which I think is confirmed by #1).

This PR implements a deployment procedure based on docker-compose and customizable via a .env configuration file (a setup which is largely inspired by PkgServer.jl). Everything should be documented in the deploy/README.md file, but please feel free to ask for more documentation if you feel like some details are missing.

Of course, comments & criticism are welcome!

If this PR is merged, deploying a LocalPackageServer instance should be as easy as cloning the git repo, editing the .env file and running make. As a further step, it should be possible to integrate this github repo with docker hub, which IIUC would allow docker hub to always provide an up-to-date image for LocalPackageServer (in much the same way as one is available for PkgServer). If we decide to go down that route, deploying a LocalPackageServer instance could become as simple as running one docker (or docker-compose) command, without having to clone or install anything on the host machine.

@codecov-io
Copy link

codecov-io commented Dec 8, 2020

Codecov Report

Merging #2 (a9e1fc0) into master (251470b) will increase coverage by 2.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   63.51%   65.54%   +2.03%     
==========================================
  Files           5        5              
  Lines         222      238      +16     
==========================================
+ Hits          141      156      +15     
- Misses         81       82       +1     
Impacted Files Coverage Δ
src/meta.jl 65.90% <0.00%> (+0.05%) ⬆️
src/gitserver.jl 77.58% <0.00%> (+0.80%) ⬆️
src/config.jl 74.19% <0.00%> (+0.86%) ⬆️
src/resource.jl 67.46% <0.00%> (+3.95%) ⬆️
src/LocalPackageServer.jl 13.63% <0.00%> (+4.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 251470b...a9e1fc0. Read the comment docs.

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