886288 - deploy new tsvgx and tscrollx talos tests to m-c only

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

we are turning on additional tests as a staging run on m-c only.  This will run in parallel with the other tests.  Once we observe these tests we will be ready to transition to the new tests as needed.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: new configs for tsvgx and tscrollx

Callek, I am not sure if the mobile stuff will work, but I took my best guess at it.

Comment 2

[Justin Wood (:Callek)]

Comment on attachment 766654 [details] [diff] [review] [diff] [review]
new configs for tsvgx and tscrollx

---------AUTOMATIC COMMENT---------
--  filter pep8-callek-june-2013 --
---------AUTOMATIC COMMENT---------

$pep8 <dir> --diff --max-line-length=159 --show-source < attachment.diff

/tmp/tmp.zbaY9yF2nI/mozilla-tests/config.py:189:160: E501 line too long (169 > 159 characters)
        'suites': GRAPH_CONFIG + ['--activeTests', 'tspaint_places_generated_med:tspaint_places_generated_max', '--setPref', 'hangmonitor.timeout=0', '--mozAfterPaint'],
                                                                                                                                                               ^

Comment 3

[Justin Wood (:Callek)]

(In reply to Joel Maher (:jmaher) from comment #0)
> we are turning on additional tests as a staging run on m-c only.  This will
> run in parallel with the other tests.  Once we observe these tests we will
> be ready to transition to the new tests as needed.

Can you explain why m-c only is wanted/needed/useful, as opposed to try or cedar instead?

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

we want to run these side by side for a while.  m-c is a lower volume tree and we can gain a few runs/day to get enough data.

Comment 5

[Justin Wood (:Callek)]

Comment on attachment 766654 [details] [diff] [review]
new configs for tsvgx and tscrollx

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

I'm r+ing provisionally, if the file I'm about to attach and request feedback from you looks as you intended.

::: mozilla-tests/mobile_config.py
@@ +163,5 @@
> +        ('mochitest-gl', (
> +            {'suite': 'mochitest-plain',
> +             'testPath': 'content/canvas/test/webgl/test_webgl_conformance_test_suite.html',
> +             },
> +        )),

as discussed on IRC, please drop this hunk

Comment 6

[Justin Wood (:Callek)]

Attachment: dump-master diff

This is a dump-master diff (clean vs patched) of attachment 766654 [details] [diff] [review], when it omits the mentioned hunk.

I used a high -U value so that its easier to see what each line references

Comment 7

[Justin Wood (:Callek)]

Attachment: intended diff

last attachment only listed one master, this is more like it.

(its a list of steps, and jobs, as buildbot sees it)

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 768861 [details] [diff] [review]
dump-master diff

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

lots of data here, but the job names and commands look as expected.

Comment 9

[Joel Maher ( :jmaher ) (UTC -8)]

waiting for reconfig:
https://hg.mozilla.org/build/buildbot-configs/rev/25db1e1006e7

Comment 10

[Kim Moir [:kmoir] ET]

In production