Stop building geckodriver as part of the build job
Categories
(Testing :: geckodriver, defect, P3)
Tracking
(firefox78 fixed)
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: whimboo, Assigned: catlee)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Once bug 1534533 is fixed, we can stop building geckodriver as part of the build job. Given that test jobs rely on the geckodriver binary as part of the common tests package, those will have to be updated first to be dependent on the geckodriver-repack
job instead.
https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&searchStr=geckodriver-repack
Note that right now this job only runs for Nightly builds of Firefox, which will also change with bug 1534533. By then we will built it for every commit, but only sign the binaries for Nightly builds of Firefox.
Comment 1•5 years ago
|
||
Does geckodriver-repack
also build? In that case it’s somewhat
misleadingly named.
Reporter | ||
Comment 2•5 years ago
|
||
No, the repack job doesn't build, but I don't see the naming as problematic right now.
Comment 3•5 years ago
|
||
I’m confused, where will geckodriver be built then?
Reporter | ||
Comment 4•5 years ago
|
||
Sorry the summary from bug 1534533 was a bit misleading. The build would happen in the toolchain task, if we agree on that. For a while we will run this job in parallel, and then we can remove this repack job.
Updated•5 years ago
|
Comment 5•5 years ago
|
||
How would this affect local developers who may well need geckodriver for running tests? Or is this just about CI?
Note that building geckodriver is relatively quick since most of the deps are also used for gecko, so it's not going to be a huge win on time/cost. But not nothing of course.
Reporter | ||
Comment 6•5 years ago
|
||
(In reply to James Graham [:jgraham] from comment #5)
How would this affect local developers who may well need geckodriver for running tests? Or is this just about CI?
I think that it would affect them when they are doing an artifact build. In such a case we should get the binary from the toolchain job and not the build job.
Assignee | ||
Comment 7•5 years ago
|
||
One issue is that the version built as part of the build in automation is huge (>50MB), which bloats the test archives and makes them slower to compress, transfer, and unpack.
Assignee | ||
Comment 8•5 years ago
|
||
Use fetches in tests to pull in the geckodriver binaries from the toolchain
builds.
Adjust paths in web-platform tests to refer to fetches
Reporter | ||
Comment 9•5 years ago
|
||
Chris, we are still blocked on bug 1577110. Maybe you can have a look at this? I'm more than happy to get rid of the build job and only have the toolchain jobs, but signing must work for us to be able to ship geckodriver to our users.
Reporter | ||
Comment 10•5 years ago
|
||
In regards of the notarization problem (bug 1588081), is there anything which is different between the normal builds and toolchain builds? If yes, and it would introduce a regression we won't actually be able to release from toolchain builds.
Assignee | ||
Updated•5 years ago
|
Reporter | ||
Comment 11•5 years ago
|
||
Now with bug 1577110 fixed we should still wait a bit of time to allow consumers to adjust their code for the new location of the geckodriver binary.
James, where is the work for wptrunner tracked?
Comment 12•5 years ago
|
||
Assignee | ||
Comment 14•5 years ago
|
||
Updated•5 years ago
|
Reporter | ||
Comment 15•5 years ago
|
||
Chris, is the old patch on this bug still relevant or made obsolete by your new patch or other patches?
Assignee | ||
Comment 16•5 years ago
|
||
(In reply to Henrik Skupin (:whimboo) [⌚️UTC+2] from comment #15)
Chris, is the old patch on this bug still relevant or made obsolete by your new patch or other patches?
The part around fetches is obsolete. The part that disables building geckodriver in CI is still valid. I'll update the patch.
Updated•5 years ago
|
Comment 17•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment 18•5 years ago
|
||
bugherder |
Comment 19•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment 20•5 years ago
|
||
bugherder |
Description
•