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

Allow users of self-contained builder to provide alternate location for assemble scripts #36

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mfojtik
Copy link
Contributor

@mfojtik mfojtik commented May 9, 2014

This is following the logic @bparees used for https://github.com/bparees/docker-image-examples/tree/remote_scripts/centos-wildfly-noscript

You can overide the default builder scripts by adding -s URL argument to the builder. In that case, builder image will fetch prepare,run,save-artifacts and restore-artifacts scripts from a remote HTTP directory and replace the original scripts.

$ docker run -v $(pwd):/tmp/src openshift/centos-ruby-builder -s http://URL/scripts

alternatively, users can place their customized scripts into .sti/ directory in the root of their GIT repository. The '-s' parameter takes precedense here.

@smarterclayton
Copy link
Contributor

Is this ready to deliver?

@bparees
Copy link
Contributor

bparees commented May 22, 2014

pretty sure this is dead code since we moved the ruby builder to openshift/ruby-19-centos.

@smarterclayton
Copy link
Contributor

Let's nuke this then - and remove the builder from docker index.

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.

3 participants