1129143 - Upgrade docker images to use tc-vcs 2.0.2 or later

Status: RESOLVED FIXED

(Taskcluster :: General, defect)

Comment 1

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: bz://1129143/lightsofapollo

/r/3295 - Bug 1129143 - Upgrade tc-vcs to 2.0.2 for desktop/emulator builds r=wcosta

Pull down this commit:

hg pull review -r b78a535119edce341eb92f1815e0803068300e16

Comment 2

[James Lal [:lightsofapollo]]

Comment on attachment 8558751 [details]
MozReview Request: bz://1129143/lightsofapollo

/r/3295 - Bug 1129143 - Upgrade tc-vcs to 2.0.2 for desktop/emulator builds r=wcosta
/r/3315 - [WIP] Fix various emulator issues:

Pull down these commits:

hg pull review -r ac05c5ca69f7c6efe11bcd6692cd6000158f0e93

Comment 3

[James Lal [:lightsofapollo]]

Comment on attachment 8558751 [details]
MozReview Request: bz://1129143/lightsofapollo

/r/3295 - Bug 1129143 - Upgrade tc-vcs to 2.0.2 for desktop/emulator builds r=wcosta
/r/3315 - [WIP] Fix various emulator issues:
/r/3383 - Update docker images to use the correct packages and gcc versions
/r/3385 - Use lightsofapollo mozharness which includes emulator perf fixes
/r/3387 - Update to tc-vcs 2.1.0
/r/3389 - Add xz
/r/3391 - Update tc-vcs to 2.1.2 which has better support for catch fetching failure

Pull down these commits:

hg pull review -r b167c2861976faa7cf7309f9fa80ff4349fc030f

Comment 4

[James Lal [:lightsofapollo]]

Comment on attachment 8558751 [details]
MozReview Request: bz://1129143/lightsofapollo

Commits are not quite as clear as I would like but this is a collection of random fixes/optimizations anyway =p

Once again I did not apply much to the phone builder but the changes in builder should make it trivial to apply the changes here.

Comment 5

[James Lal [:lightsofapollo]]

https://reviewboard.mozilla.org/r/3293/#review2751

Comment 6

[James Lal [:lightsofapollo]]

Comment on attachment 8558751 [details]
MozReview Request: bz://1129143/lightsofapollo

https://reviewboard.mozilla.org/r/3293/#review2753

Comment 7

[James Lal [:lightsofapollo]]

Attachment: 1129143.patch

Comment 8

[Wander Lairson Costa]

Comment on attachment 8559984 [details] [diff] [review]
1129143.patch

Review of attachment 8559984 [details] [diff] [review]:
-----------------------------------------------------------------

A few nits, but lgtm

::: testing/docker/b2g-build/Dockerfile
@@ +84,5 @@
> +RUN mkdir -p /home/worker/bin
> +COPY bin/repository-url.py /home/worker/bin/repository-url.py
> +
> +# RPM Source urls...
> +ENV MOZILLA_RPM_URL http://mockbuild-repos.pub.build.mozilla.org/releng/public/CentOS/6/x86_64/

I think this is not needed anymore.

::: testing/docker/builder/emulator-ics.env
@@ +3,5 @@
> +MOZHARNESS_REPOSITORY=https://github.com/lightsofapollo/build-mozharness
> +MOZHARNESS_REV=emulator-perf
> +GECKO_HEAD_REV=tip
> +TARGET=emulator
> +B2G_CONFIG=emulator

B2G_CONFIG is not used anymore.

::: testing/docker/builder/emulator-kk.env
@@ +3,5 @@
> +MOZHARNESS_REPOSITORY=https://github.com/lightsofapollo/build-mozharness
> +MOZHARNESS_REV=emulator-perf
> +GECKO_HEAD_REV=tip
> +TARGET=emulator-kk
> +B2G_CONFIG=emulator-kk

B2G_CONFIG is not used anymore.

::: testing/docker/builder/emulator.env
@@ +2,4 @@
>  GECKO_HEAD_REPOSITORY=https://hg.mozilla.org/mozilla-central
>  GECKO_HEAD_REV=tip
>  TARGET=emulator
>  B2G_CONFIG=emulator-ics

B2G_CONFIG is not used anymore.

::: testing/taskcluster/mach_commands.py
@@ +136,5 @@
>      @CommandArgument('--base-repository',
>          default=os.environ.get('GECKO_BASE_REPOSITORY'),
>          help='URL for "base" repository to clone')
>      @CommandArgument('--mozharness-repository',
> +        default='https://github.com/lightsofapollo/build-mozharness',

hrm Shouldn't be mozilla.org repo?

Comment 9

[James Lal [:lightsofapollo]]

https://hg.mozilla.org/integration/b2g-inbound/rev/43259e1d2f2e

re-irc using my own mozharness as default for the moment (this should change shortly)

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/43259e1d2f2e

Comment 12

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Bug 1129143 - Upgrade tc-vcs and fix related emulator building issues r=wcosta

Comment 13

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: [WIP] Fix various emulator issues:

Comment 14

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Update docker images to use the correct packages and gcc versions

Comment 15

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Use lightsofapollo mozharness which includes emulator perf fixes

Comment 16

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Update to tc-vcs 2.1.0

Comment 17

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Add xz

Comment 18

[James Lal [:lightsofapollo]]

Attachment: MozReview Request: Update tc-vcs to 2.1.2 which has better support for catch fetching failure

Comment 19

[Pete Moore [:pmoore][:pete]]

Resetting Version and Target Milestone that accidentally got changed...