Closed Bug 829099 Opened 12 years ago Closed 12 years ago

Gaia UI tests are triggered twice

Categories

(Release Engineering :: General, defect, P1)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: armenzg, Assigned: armenzg)

References

Details

Attachments

(4 files, 1 obsolete file)

We currently have 2 b2g panda builds: one is based of gaia nightly and the other based of gaia central. We trigger gaia ui tests for each but both report as if they were triggered by the same build. I think we separate the two and I think it would be nice to point on the tbpl summary which changesets where used for each build/tests. That way there is no need to load logs. Thoughts/comments?
If we change the buildername to correspond to the build name, TBPL will put them on the correct row.
This requires duplicating the b2g panda builders and add "central" in the name: http://hg.mozilla.org/build/buildbot-configs/file/default/mozilla-tests/b2g_config.py We will also have to modify the sendchanges: http://hg.mozilla.org/build/mozharness/file/4fe45d726b2b/scripts/b2g_build.py#l889
Blocks: 802317
I will take this bug first since it has buildbot required changes (which I expect less people comfortable fixing).
Assignee: nobody → armenzg
Priority: -- → P1
Attached patch add b2g_panda_gaia_central platform (obsolete) (deleted) — Splinter Review
I need to fix the sendchanges as well.
Attachment #702829 - Flags: review?(aki)
Attachment #702829 - Attachment is obsolete: true
Attachment #702829 - Flags: review?(aki)
Attachment #702835 - Flags: review?(aki)
Attachment #702852 - Flags: review?(aki)
Comment on attachment 702852 [details] [diff] [review] add b2g_panda_gaia_central platform to puppet Hm, maybe b2g_panda-gaia_central ? I don't have a strong opinion here.
Attachment #702852 - Flags: review?(aki) → review+
(In reply to Aki Sasaki [:aki] from comment #7) > Hm, maybe b2g_panda-gaia_central ? The test buildernames need to match "b2g.*panda_gaia_central" so they get placed on the same row as the builders with that name.
(In reply to Ed Morley [:edmorley UTC+0] from comment #8) > the builders with that name. the builds with that buildername.
Comment on attachment 702835 [details] [diff] [review] add b2g_panda_gaia_central platform Same. We seem to have variants end in -variant (tegra_android-armv6, leopard-o) so b2g_panda-gaia_central might make sense. Splitting by dashes is a pretty big hack, though, and I'm not sure if it's required. Do we need a build patch to fix the sendchange/upload location?
Attachment #702835 - Flags: review?(aki) → review+
The sendchanges differ like this: --branch ash-panda_gaia_central-opt-unittest --branch ash-panda-opt-unittest
Attachment #703345 - Flags: review?(aki)
Comment on attachment 703345 [details] [diff] [review] modify sendchanges for each build I like this. The pgo_build:False might need to be configurable at some point, but this is good for now.
Attachment #703345 - Flags: review?(aki) → review+
Attachment #702835 - Flags: checked-in+
Attachment #702852 - Flags: checked-in+
Attachment #703345 - Flags: checked-in+
Attachment #703900 - Flags: review?(aki)
Attachment #703900 - Flags: review?(aki) → review+
This is in production, minus the tools patch which landed after the most recent reconfig.
Attachment #703900 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
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: