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

Prepare repo for automated build on Dockerhub #205

Merged
merged 2 commits into from
Oct 17, 2016
Merged

Prepare repo for automated build on Dockerhub #205

merged 2 commits into from
Oct 17, 2016

Conversation

bcorijn
Copy link

@bcorijn bcorijn commented Oct 17, 2016

Changes done:

  • Updated base image from Fedora to Alpine. Reduces the image size from 436MB to 130 MB.
  • Moved the docker file and startup script to the root directory
  • Removed the ENV variables out of the docker file and into the compose file
  • Pointed the compose file to the future image on docker hub
  • Adjusted the compose file to use Version 2 syntax, as recommended by Docker

This should make it doable to have an image on Dockerhub to use for people while keeping the functionality of the compose file to create a local stack.

@matejkramny matejkramny merged commit fd89143 into semaphoreui:master Oct 17, 2016
@bcorijn bcorijn deleted the docker branch October 24, 2016 12:30
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.

None yet

2 participants