Skip to content

Faster CI#206

Merged
vdemeester merged 1 commit into
docker:masterfrom
dnephin:why-cross-so-slow
Jul 13, 2017
Merged

Faster CI#206
vdemeester merged 1 commit into
docker:masterfrom
dnephin:why-cross-so-slow

Conversation

@dnephin
Copy link
Copy Markdown
Contributor

@dnephin dnephin commented Jun 19, 2017

Runtime is now under 3min (from ~9min)

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jun 19, 2017

Codecov Report

Merging #206 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #206   +/-   ##
=======================================
  Coverage   48.93%   48.93%           
=======================================
  Files         177      177           
  Lines       11846    11846           
=======================================
  Hits         5797     5797           
  Misses       5687     5687           
  Partials      362      362

Copy link
Copy Markdown
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐸

@dnephin dnephin force-pushed the why-cross-so-slow branch 2 times, most recently from 882d719 to 7ec790c Compare June 23, 2017 17:18
@dnephin dnephin force-pushed the why-cross-so-slow branch from 7ec790c to 0718d63 Compare June 23, 2017 17:44
@dnephin
Copy link
Copy Markdown
Contributor Author

dnephin commented Jun 23, 2017

I cleaned up the cross script to remove the duplication of jobs (that was in previous versions of this PR)

Comment thread dockerfiles/Dockerfile.cross Outdated
RUN /tmp/osx-cross.sh
ENV PATH /osxcross/target/bin:$PATH

FROM dnephin/golang-cross@sha256:3a23c1791a8f9bb4a79e2708b2a87b51502ef24315170ed0fc27baa056c1a862
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we try to get this in the dockercore org? I think we used that for other images used during build/ci; https://hub.docker.com/u/dockercore/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can still decide to keep the dockerfile for building it in this repository

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be happy to move the image to dockercore, what org should receive the github repo?

I don't think we should keep a copy here if it's not used by CI. Keeping around files that aren't actually used just causes confusion. If someone needs to make a change they can go to the source repo.

We pin an exact digest here, so upgrades are easy.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tiborvass do you know if we already have a general purpose git repo for base images used in CI?

We could move the golang-cross Dockerfile to that repo, if one exists.

Reusable docker engine

Signed-off-by: Daniel Nephin <dnephin@docker.com>
@dnephin dnephin force-pushed the why-cross-so-slow branch from 0718d63 to 1eb66d6 Compare July 5, 2017 19:23
@dnephin
Copy link
Copy Markdown
Contributor Author

dnephin commented Jul 5, 2017

Ok, this is now using the docker org on github, and the dockercore org on hub.

@dnephin
Copy link
Copy Markdown
Contributor Author

dnephin commented Jul 11, 2017

ping for another review

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐮

@vdemeester vdemeester merged commit e3746d3 into docker:master Jul 13, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.07.0 milestone Jul 13, 2017
@dnephin dnephin deleted the why-cross-so-slow branch July 13, 2017 14:10
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
[17.06] bump version to 17.06.2-ce-rc1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants