Closed Bug 1467827 Opened Last year Closed Last year

Port raptor firefox tp6 to linux

Categories

(Testing :: Raptor, enhancement)

Version 3
enhancement
Not set

Tracking

(firefox62 fixed)

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: rwood, Assigned: rwood)

References

Details

Attachments

(1 file)

No description provided.
Depends on: 1467720
Assignee: nobody → rwood
Status: NEW → ASSIGNED
Note: The TC configs to add Raptor tp6 to linux64 and win10 on try as tier3 are included in Bug 1467823
Depends on: 1467823
Try run to see latest status of Raptor tests on all platforms (only OSX is green so far):

https://treeherder.mozilla.org/#/jobs?repo=try&revision=54dde805af6faeac619af64144b0310ce0f8ea75&filter-tier=1&filter-tier=2&filter-tier=3
Making this bug for porting raptor tp6 firefox to linux64 (filed Bug 1468865 for porting to Win10).
Summary: Port raptor firefox tp6 to linux and win10 → Port raptor firefox tp6 to linux
Comment on attachment 8985234 [details]
Bug 1467827 - Port raptor firefox tp6 to linux;

https://reviewboard.mozilla.org/r/250862/#review257704


Code analysis found 2 defects in this patch:
 - 2 defects found by mozlint

You can run this analysis locally with:
 - `./mach lint path/to/file` (JS/Python)


If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx


::: testing/raptor/raptor/playback/mitmproxy.py:133
(Diff revision 5)
> +    def transform_platform(str_to_transform):
> +        # transform platform name i.e. 'mitmproxy-rel-bin-{platform}.manifest'
> +        # transforms to 'mitmproxy-rel-bin-osx.manifest'
> +        if '{platform}' not in str_to_transform:
> +            return str_to_transform
> +        if 'win' in self.config['platform']:

Error: Undefined name 'self' [flake8: F821]

::: testing/raptor/raptor/playback/mitmproxy.py:135
(Diff revision 5)
> +        # transforms to 'mitmproxy-rel-bin-osx.manifest'
> +        if '{platform}' not in str_to_transform:
> +            return str_to_transform
> +        if 'win' in self.config['platform']:
> +            platform_id = 'win'
> +        elif self.config['platform'] == 'mac':

Error: Undefined name 'self' [flake8: F821]
Comment on attachment 8985234 [details]
Bug 1467827 - Port raptor firefox tp6 to linux;

https://reviewboard.mozilla.org/r/250862/#review257730

a few nits/optional things to consider.

::: taskcluster/ci/test/raptor.yml:35
(Diff revision 8)
>      try-name: raptor-firefox-tp6
>      treeherder-symbol: Rap(tp6)
>      run-on-projects:
>          by-test-platform:
>              macosx.*: ['try', 'mozilla-central']
> +            linux64.*: ['try', 'mozilla-central']

can we just make this:
win.*: []

::: taskcluster/taskgraph/transforms/tests.py:412
(Diff revision 8)
>              test['mozharness']['set-moz-node-path'] = True
>              test.setdefault('e10s', True)
>  
>          # software-gl-layers is only meaningful on linux unittests, where it defaults to True
> -        if test['test-platform'].startswith('linux') and test['suite'] != 'talos':
> +        if test['test-platform'].startswith('linux') and test['suite'] != 'talos' and \
> +                test['suite'] != 'raptor':

maybe:
and test['suite'] not in ['talos', 'raptor']

::: testing/raptor/raptor/playback/mitmproxy.py:139
(Diff revision 8)
> +            platform_id = 'win'
> +        elif self.config['platform'] == 'mac':
> +            platform_id = 'osx'
> +        else:
> +            platform_id = 'linux64'
> +        return str_to_transform.replace('{platform}', platform_id)

this might be useful in other parts of raptor- consider refactoring to a 'utils' section
Attachment #8985234 - Flags: review?(jmaher) → review+
Comment on attachment 8985234 [details]
Bug 1467827 - Port raptor firefox tp6 to linux;

https://reviewboard.mozilla.org/r/250862/#review257730

> can we just make this:
> win.*: []

Not sure what you mean here. Remove mac and linux64 lines and just have win to []? Then have default on try and central? But then any new or other platforms will auto be added to central... ?

> maybe:
> and test['suite'] not in ['talos', 'raptor']

Ah yes thanks

> this might be useful in other parts of raptor- consider refactoring to a 'utils' section

Good idea thanks will do
all platforms is:
linux
win
osx

and soon to be android, we can solve this later.
Pushed by rwood@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7f424be1fb4a
Port raptor firefox tp6 to linux; r=jmaher
https://hg.mozilla.org/mozilla-central/rev/7f424be1fb4a
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Blocks: 1449180
You need to log in before you can comment on or make changes to this bug.