Closed
Bug 829099
Opened 12 years ago
Closed 12 years ago
Gaia UI tests are triggered twice
Categories
(Release Engineering :: General, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: armenzg, Assigned: armenzg)
References
Details
Attachments
(4 files, 1 obsolete file)
(deleted),
patch
|
mozilla
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mozilla
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mozilla
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mozilla
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
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?
Comment 1•12 years ago
|
||
If we change the buildername to correspond to the build name, TBPL will put them on the correct row.
Assignee | ||
Comment 2•12 years ago
|
||
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
Assignee | ||
Comment 3•12 years ago
|
||
I will take this bug first since it has buildbot required changes (which I expect less people comfortable fixing).
Assignee: nobody → armenzg
Assignee | ||
Updated•12 years ago
|
Priority: -- → P1
Assignee | ||
Comment 4•12 years ago
|
||
I need to fix the sendchanges as well.
Attachment #702829 -
Flags: review?(aki)
Assignee | ||
Comment 5•12 years ago
|
||
Attachment #702829 -
Attachment is obsolete: true
Attachment #702829 -
Flags: review?(aki)
Attachment #702835 -
Flags: review?(aki)
Assignee | ||
Comment 6•12 years ago
|
||
Attachment #702852 -
Flags: review?(aki)
Comment 7•12 years ago
|
||
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+
Comment 8•12 years ago
|
||
(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.
Comment 9•12 years ago
|
||
(In reply to Ed Morley [:edmorley UTC+0] from comment #8)
> the builders with that name.
the builds with that buildername.
Comment 10•12 years ago
|
||
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+
Assignee | ||
Comment 11•12 years ago
|
||
The sendchanges differ like this:
--branch ash-panda_gaia_central-opt-unittest
--branch ash-panda-opt-unittest
Attachment #703345 -
Flags: review?(aki)
Comment 12•12 years ago
|
||
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+
Assignee | ||
Updated•12 years ago
|
Attachment #702835 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Attachment #702852 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Attachment #703345 -
Flags: checked-in+
Assignee | ||
Comment 13•12 years ago
|
||
Attachment #703900 -
Flags: review?(aki)
Updated•12 years ago
|
Attachment #703900 -
Flags: review?(aki) → review+
Comment 14•12 years ago
|
||
This is in production, minus the tools patch which landed after the most recent reconfig.
Assignee | ||
Updated•12 years ago
|
Attachment #703900 -
Flags: checked-in+
Assignee | ||
Comment 15•12 years ago
|
||
It is working now:
https://tbpl.mozilla.org/?tree=Cedar&jobname=b2g_panda
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•