Dylan Hardison [:dylan] (he/him) Assignee
	Description • 8 years ago

Per IRC discussion with dkl: to move forward on various infrastructure projects we need to be testing against centos6, and so the docker stuff should be based on that.

I've also split out the base-level requirements into a separate bmo-base docker image, available as dylanwh/bmo-base (also in github, in mozilla-bteam/docker-bmo-base)

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 2 • 8 years ago

Pull request updated. It's now using the latest bmo-base, and this updates the taskgraph file to refer to mozillabteam/bmo-ci
(which I have also uploaded)

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 4 • 8 years ago

Comment on attachment 8783007 [details] [review]
Small dockerfile tweak (v1)

lgtm. I had a reason for using ADD originally but I can't remember what it was, and so much has changed it is probably not important. :)

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 5 • 8 years ago

I removed the dependency on the local lib changes landing by copying the modules from the vendor tarball to /usr/local/* directories.

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 6 • 8 years ago

in hindsight, the separate git repo for the docker stuff was better for several reasons. We want the docker image updated when we make changes to it, and we want task cluster to run tests when we make changes to the bmo repo. There would be a race condition if both of these things happen at once.

mozillabteam/bmo-ci on dockerhub (and mozilla-bteam/docker-bmo-ci on github) are split off from this bug/patch
and this just changes the taskgraph.json file

	David Lawrence [:dkl]
	Comment 7 • 8 years ago

Comment on attachment 8776583 [details] [review]
Pull Request

r=dkl

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 8 • 8 years ago

To github.com:mozilla-bteam/bmo.git
   806f970..5801d06  master -> master