1299869 - Android Gradle deps build broken with /home/worker/workspace/build/src/taskcluster/scripts/builder/build-linux.sh: line 7: /home/worker/scripts/xvfb.sh: No such file or directory

Status: RESOLVED FIXED

(Taskcluster :: General, defect)

Description

[Nick Alexander :nalexander [he/him]]

/home/worker/workspace/build/src/taskcluster/scripts/builder/build-linux.sh: line 7: /home/worker/scripts/xvfb.sh: No such file or directory

https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=2922384

I think the underlying Docker image has moved underneath the task.  It's not needed right now, but we need to fix this so that we can move the Android dependencies forward easily in the future.

Comment 2

[Nick Alexander :nalexander [he/him]]

Try is green: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0289aa620862&selectedJob=26776068.

Comment 3

[Nick Alexander :nalexander [he/him]]

Attachment: Bug 1299869 - Track desktop-build xfvb changes.

When I wrote this initially, I was told it was not possible to inherit
from desktop-build.  That may have changed, but I still don't know how
to inherit, so we duplicate the logic here, inviting bugs of this
class.  The underlying logic moves slowly so this is an acceptable
approach.

Review commit: https://reviewboard.mozilla.org/r/76126/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/76126/

Comment 4

[Nick Alexander :nalexander [he/him]]

(In reply to Nick Alexander :nalexander from comment #3)
> Created attachment 8787316 [details]
> Bug 1299869 - Track desktop-build xfvb changes.
> 
> When I wrote this initially, I was told it was not possible to inherit
> from desktop-build.  That may have changed, but I still don't know how
> to inherit, so we duplicate the logic here, inviting bugs of this
> class.  The underlying logic moves slowly so this is an acceptable
> approach.
> 
> Review commit: https://reviewboard.mozilla.org/r/76126/diff/#index_header
> See other reviews: https://reviewboard.mozilla.org/r/76126/

garndt: I see a typo in the commit message (should be "xvfb"); I'll update before landing.

Comment 5

[Greg Arndt [:garndt]]

Comment on attachment 8787316 [details]
Bug 1299869 - Track desktop-build xfvb changes.

https://reviewboard.mozilla.org/r/76126/#review74106

::: testing/docker/android-gradle-build/Dockerfile:7
(Diff revision 1)
> +FROM          taskcluster/centos6-build-upd:0.1.6.20160329195300
>  MAINTAINER    Nick Alexander <[e-mail]>
>  
>  # BEGIN ../desktop-build/Dockerfile
>  
> +# TODO remove when base image is updated

Is there a reason why we're adding this volume stuff in this patch? Maybe it's needed for something I'm overlooking.

Comment 6

[Greg Arndt [:garndt]]

Comment on attachment 8787316 [details]
Bug 1299869 - Track desktop-build xfvb changes.

https://reviewboard.mozilla.org/r/76126/#review74108

Thanks for taking this by the way.  I think it looks good, easy enough.  I was just unsure about the addition of the volume stuff.

Comment 7

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8787316 [details]
Bug 1299869 - Track desktop-build xfvb changes.

https://reviewboard.mozilla.org/r/76126/#review74106

> Is there a reason why we're adding this volume stuff in this patch? Maybe it's needed for something I'm overlooking.

No -- I was just tracking the desktop-build changes as closely as possible.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2677e17bd4d8
Track desktop-build xfvb changes. r=garndt

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2677e17bd4d8

Comment 10

[Intermittent Failures Robot]

5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* autoland: 4
* mozilla-central: 1

Platform breakdown:
* android-4-0-armv7-api15: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1299869&startday=2016-08-29&endday=2016-09-04&tree=all