Closed Bug 868092 Opened 11 years ago Closed 11 years ago

Turn on all tests on UX project branch

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: MattN, Assigned: catlee)

References

Details

(Whiteboard: [simple])

Attachments

(1 file)

Bug 779419 turned off all tests on UX branch to reduce infra load.

The purpose of the UX branch has temporarily changed to become the integration branch for Australis so the tests are much more useful now. Please turn on the tests (including talos) for the UX branch.

Thanks
This is either it, or it's the easy part and someone will also need to enable Android (easy) and b2g (less easy, and with at least one trap for the unwary).

MattN: are you going to avoid touching anything that could even possibly affect Android and b2g? So far, it looks like browser/ and widget/cocoa/, which would make desktop-only fine.
Attachment #745670 - Flags: review?(catlee)
Flags: needinfo?(mnoorenberghe+bmo)
And of course there's the other trap for the unwary, the fact that doing this is going to schedule tests on every UX commit since the time they were turned off, but luckily this week's |buildduty is pretty canny, and he'll deal with that.
(In reply to Phil Ringnalda (:philor) from comment #1)
> MattN: are you going to avoid touching anything that could even possibly
> affect Android and b2g? So far, it looks like browser/ and widget/cocoa/,
> which would make desktop-only fine.

It's unlikely that Australis will break b2g or Android without also breaking desktop. At the same time, we only make a few pushes per day to UX on average so the benefit of knowing immediately when we break android/b2g may outweigh the cost of infra load. I don't feel strongly either way as I haven't been watching the infrastructure load closely, and that was the reason these tests were shut off.
Flags: needinfo?(mnoorenberghe+bmo)
Attachment #745670 - Flags: review?(catlee) → review+
in production
Assignee: nobody → catlee
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
(In reply to Matthew N. [:MattN] from comment #0)
> The purpose of the UX branch has temporarily changed to become the
> integration branch for Australis so the tests are much more useful now.

Do you mean Australis testing, or "the UX tree will be merging directly into inbound"? The latter will bring all the UX tree cruft with it, so I'm very much against it, unless we reset the UX tree before work continues for real.
Blocks: 779419
Flags: needinfo?(mnoorenberghe+bmo)
s/inbound/mozilla-central/
Ah, UX was reset a month ago deliberately for it (in bug 862671), in which case that's fine :-)
Flags: needinfo?(mnoorenberghe+bmo)
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: