Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

Feature/docker advanced config #117

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

Conversation

lksv
Copy link

@lksv lksv commented Apr 30, 2015

This feature allows to use custom docker images.

@joshk
Copy link
Contributor

joshk commented Apr 30, 2015

thank you for the PR

we would certainly be interested in merging in some of these changes

is there a particular reason you have added a license for your company?

@joshk
Copy link
Contributor

joshk commented Apr 30, 2015

also, I will need to check with the team, but i think some of these changes would be best added to travis-ci/worker repo instead.

@lksv
Copy link
Author

lksv commented Apr 30, 2015

@joshk add license)
My contract requires put my work with the copyright holder of AVG. ...I understand your question, but I cannot do with it much.

You can specify image name, user name, default command and enable/disable
cpuset. See config/worker.example.docker.yml
@lksv lksv force-pushed the feature/docker_advanced_config branch from 6a47526 to 32122bd Compare June 5, 2015 20:08
@lksv
Copy link
Author

lksv commented Jun 5, 2015

Custom license removed.

@lksv
Copy link
Author

lksv commented Jun 5, 2015

It seems to me, that failed build happens by something else ...not related to this commit.

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