Skip to content

Conversation

@aanand
Copy link

@aanand aanand commented Mar 27, 2015

I've moved the DCO validation step out of script/test-versions, where it really doesn't belong, and added script/ci, which:

  1. Performs DCO validation
  2. Runs tests against all Docker versions
  3. Builds the Linux binary

Currently the last step is failing with an odd Permission denied error, which I'm still investigating.

@dnephin
Copy link

dnephin commented Mar 27, 2015

sounds like a good idea to me

@aanand
Copy link
Author

aanand commented Mar 27, 2015

OK, should be working now.

@jfrazelle - could you try updating the Jenkins command to:

$ TAG="docker-compose:$(git rev-parse --short HEAD)"
$ docker build -t "$TAG" .
$ docker run --rm --volume="/var/run/docker.sock:/var/run/docker.sock" --volume="$(pwd):/code" -e "TAG=$TAG" --entrypoint="script/ci" "$TAG"

@jessfraz
Copy link

will dpo!

On Fri, Mar 27, 2015 at 11:48 AM, Aanand Prasad notifications@github.com
wrote:

OK, should be working now.

@jfrazelle https://github.com/jfrazelle - could you try updating the
Jenkins command to:

$ TAG="docker-compose:$(git rev-parse --short HEAD)"
$ docker build -t "$TAG" .
$ docker run --rm --volume="/var/run/docker.sock:/var/run/docker.sock" --volume="$(pwd):/code" -e "TAG=$TAG" --entrypoint="script/ci" "$TAG"


Reply to this email directly or view it on GitHub
#1202 (comment).

@jessfraz
Copy link

done!

On Fri, Mar 27, 2015 at 11:51 AM, Jessica Frazelle jess@docker.com wrote:

will dpo!

On Fri, Mar 27, 2015 at 11:48 AM, Aanand Prasad notifications@github.com
wrote:

OK, should be working now.

@jfrazelle https://github.com/jfrazelle - could you try updating the
Jenkins command to:

$ TAG="docker-compose:$(git rev-parse --short HEAD)"
$ docker build -t "$TAG" .
$ docker run --rm --volume="/var/run/docker.sock:/var/run/docker.sock" --volume="$(pwd):/code" -e "TAG=$TAG" --entrypoint="script/ci" "$TAG"


Reply to this email directly or view it on GitHub
#1202 (comment).

Signed-off-by: Aanand Prasad <aanand.prasad@gmail.com>
aanand added a commit that referenced this pull request Mar 27, 2015
@aanand aanand merged commit 853ce25 into docker:master Mar 27, 2015
@aanand aanand deleted the jenkins-script branch March 27, 2015 21:59
aanand added a commit to aanand/fig that referenced this pull request Apr 1, 2015
WIP: Jenkins script
(cherry picked from commit 853ce25)
aanand added a commit to aanand/fig that referenced this pull request Apr 1, 2015
WIP: Jenkins script
(cherry picked from commit 853ce25)
Signed-off-by: Aanand Prasad <aanand.prasad@gmail.com>
yuval-k pushed a commit to yuval-k/compose that referenced this pull request Apr 10, 2015
WIP: Jenkins script
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
yuval-k pushed a commit to yuval-k/compose that referenced this pull request Apr 10, 2015
WIP: Jenkins script
Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
StefanScherer added a commit to hypriot/compose that referenced this pull request Apr 19, 2015
* tag '1.2.0': (46 commits)
  Bump 1.2.0
  Merge pull request docker#1278 from albers/completion-run-user
  Merge pull request docker#1261 from aanand/fix-vars-in-volume-paths
  Merge pull request docker#1202 from aanand/jenkins-script
  Merge pull request docker#1213 from moysesb/relative_build
  Merge pull request docker#1226 from aanand/merge-multi-value-options
  Merge pull request docker#1225 from aanand/fix-1222
  Fix regression in Dns and DnsSearch settings
  Fix volume merging when there's no explicit host path
  Fix service dict merging when only one dict has a volumes key
  Update docker-py and requests version ranges
  Revert "Remove restriction for requests version, update docker-py requirement"
  Revert "Use dev version of Docker"
  Revert "Add 'labels:' config option"
  Add 'labels:' config option
  Use dev version of Docker
  Make volume host paths relative to file, merge volumes when extending
  Implement `extends`
  Specify all HostConfig at create time
  Add build status \o/
  ...

Conflicts:
	Dockerfile
	README.md
	script/build-linux
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