1013691 - Clean up TBPL symbols

Status: RESOLVED INCOMPLETE

(Tree Management Graveyard :: TBPL, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

reftest and reftest-sanity are currently sharing symbols. I'm going to turn reftest-sanity into Rs.

Also, the Gaia OOP tests are a pile of G-oop at the moment and need to look better. Also, it's been suggested that the symbols should get some changing around to better reflect what they are, especially now that Gaia-Try is working to replace Travis.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Especially with the Gaia Try work progressing, I think the time to make these symbol changes for people transitioning from Travis to TBPL is now.

Unless we don't think we need them after all and want to stick with the current ones for the various Gaia test suites.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Had an IRC chat with various Gaia test stakeholders today and I think we reached a basic consensus on the changes to make:

G -> Gu (description will remain Gaia Unit Tests)
Gi -> Gij (description will change to Gaia JS Integration Tests)
Gu -> Gip (description will change to Gaia Python Integration Tests)

Gb and Li will stay as-is.

If the above sounds good to everyone CCed here, I think that's a workable way forward!

Comment 3

[Ryan VanderMeulen [:RyanVM]]

BTW, it's unlikely I'll be able to change the buildbot names for the jobs (creates pain for questionable gain), so we'll have to live with those anyway.

Comment 4

[Julien Wajsberg [:julienw]]

Good for me :)

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Attachment: give reftest-sanity its own symbol

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Attachment: change Gaia unit test naming and symbols

Per comment 2

Comment 7

[Ed Morley [:emorley]]

Comment on attachment 8450226 [details] [diff] [review]
change Gaia unit test naming and symbols

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

r=me with using end of line anchors ($) where possible, to make these more robust when moved to treeherder + double checking with &debug=1 after this patchset is applied.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

First two parts pushed:
https://hg.mozilla.org/webtools/tbpl/rev/be6dc427bf80
https://hg.mozilla.org/webtools/tbpl/rev/42a2fa73a557

And a follow-up to add some end of line anchors:
https://hg.mozilla.org/webtools/tbpl/rev/dd91d8ad1d37

Leaving this bug open for more to come.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #8)

In production.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Attachment: use a consistent H symbol for Hazard Analysis jobs

I special-cased the shell version to show up in the SM() group along with the other shell jobs. I've confirmed that debug=1 comes up clean and existing jobs on TBPL show up correctly (including a Try run of the shell version).

I'll hold off on landing this until after sfink gets back from PTO and confirms that he's happy with this.

Comment 11

[Ed Morley [:emorley]]

Comment on attachment 8465474 [details] [diff] [review]
use a consistent H symbol for Hazard Analysis jobs

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

lgtm

Comment 12

[Steve Fink [:sfink] [:s:]]

Comment on attachment 8465474 [details] [diff] [review]
use a consistent H symbol for Hazard Analysis jobs

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

This is great, ship it!

Thanks.

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/webtools/tbpl/rev/06af9dad7e20

So looking back at comment 0, I think all that's left for here is the OOP test situation.

Comment 14

[Ryan VanderMeulen [:RyanVM]]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #13)
> https://hg.mozilla.org/webtools/tbpl/rev/06af9dad7e20

In production.

Comment 15

[Ed Morley [:emorley]]

TBPL has been EOLed (bug 1054977).

Comment 16

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.