Allow use of custom local and remote builds

RESOLVED FIXED

Status

Testing
Autophone
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mcote, Assigned: bc)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Autophone should be able to run tests against custom local and remote (at least http) builds, and specifying the location should be as easy as possible.
(Reporter)

Updated

3 years ago
No longer blocks: 1037059
(Reporter)

Updated

3 years ago
Blocks: 1037067
(Assignee)

Updated

3 years ago
Assignee: nobody → bclary
Status: NEW → ASSIGNED
(Assignee)

Comment 1

3 years ago
Created attachment 8499841 [details] [diff] [review]
bug-1037064-build-url.patch

Adds option to trigger_runs.py:
  --build-url=BUILD_URL
                        url of build to test; may be an http or file schema;
and the ability to specify try as a repo.

build_url can be an http(s) url pointing to a build directory such as 

python trigger_runs.py --repo=try --build-url=http://ftp.mozilla.org/pub/mozilla.org/mobile/try-builds/wlitwinczyk@mozilla.com-fde6088fe349/try-android/

or it can be a local path to a build directory as in

python trigger_runs.py --repo=mozilla-central --build-url=/mozilla/builds/nightly/mozilla/fennec-opt/dist/

Local paths require that we force the download and recreation of the metadata.json since the path doesn't change but the build does.

Note if pulse is enabled and try is an enabled repo, then autophone will automatically test all try builds.
Attachment #8499841 - Flags: review?(mcote)
(Reporter)

Comment 2

3 years ago
Comment on attachment 8499841 [details] [diff] [review]
bug-1037064-build-url.patch

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

::: trigger_runs.py
@@ +72,5 @@
> +        build_urls = builds.BuildCache(
> +            options.repos, options.buildtypes,
> +            product, build_platforms,
> +            buildfile_ext).find_builds_by_directory(
> +                options.build_url)

We build the same BuildCache object, with the same parameters, in three different 'if' clauses now.  We should construct it once and just do the find_* calls separately.
Attachment #8499841 - Flags: review?(mcote) → review+
(Assignee)

Comment 3

3 years ago
https://github.com/mozilla/autophone/commit/a11d7cb5e68ddfad0bc7545e1d8eaf652c0a2d3a
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.