Closed Bug 725703 Opened 9 years ago Closed 8 years ago

tests should figure out the app name for fennec instead of hardcoding everywhere

Categories

(Release Engineering :: General, defect, P4)

ARM
Android
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aki, Assigned: Callek)

References

Details

(Whiteboard: [automation][try][tegras])

Attachments

(2 files, 1 obsolete file)

As of bug 703080, we now have a way to figure out the appname from an apk.
It's debatable if it's useful to fail when the appname is incorrect on mozilla-{central,aurora,beta,release}, since nothing else is testing branding.

However, on Try we should definitely try to grab package-name.txt from the apk.  If that succeeds, then the package name should be in there; otherwise we can assume org.mozilla.fennec.
Duplicate of this bug: 725659
Bug 703080 only fixed Android native. We need the equivalent for android-xul.
This will be useful for Autolanding on branches since we'll want proper try of mozilla-{esr10,beta,aurora} before pushing to those branches with autoland.
Blocks: 657828
(In reply to Aki Sasaki [:aki] from comment #2)
> Bug 703080 only fixed Android native. We need the equivalent for android-xul.

Not anymore, so we should now always have |package-name.txt| available for this. That said I'm not currently working on this.
Hey Alex, this came up today in a different context, but I want to know if you consider the:

"Merge Day Missed branding changes for mobile" a use-case we need to continue watching for in the automation.

If we don't it allows us to remove a whole bunch of hardcodes, and potential releng automation issues, but then we have nothing, other than QA, doing branding sanity checks. (And QA doesn't check all the builds we generate, e.g. Android NoIonmonkey, Android Debug)

Either way I expect releng to implement some subset of this work item, the direction we take however, will rely on your answer. ("Keep the check" will be a bit harder, believe it or not)

-- FWIW I'm not giving an ETA on this being worked on, so no rush on answer.
Flags: needinfo?(akeybl)
(In reply to Justin Wood (:Callek) from comment #5)
> "Merge Day Missed branding changes for mobile" a use-case we need to
> continue watching for in the automation.

Where are these automation results currently presented? I know philor has previously filed a bug on branding mistakes on x86 Android builds (which is helpful), so perhaps these tests were what tipped him off.
Flags: needinfo?(akeybl)
In the current world, if you don't update the branding as part of a merge, then every single test fails to start up, and tbpl shows a solid row of orange/red, and I either file a bug or more commonly just push the fix.

With this bug fixed, the tests would still run, and your only clue that you forgot about a branding change would be when someone, perhaps QA, noticed that the app had the wrong branding.
Blocks: 842846
Summary: tegras should figure out the app name instead of hardcoding everywhere → tests should figure out the app name for fennec instead of hardcoding everywhere
For posterity after an e-mail thread with akeybl (for relman) and kbronsnan for QA they were ok with taking this in terms of the more-manual-work and loss of safety net in branding.
Attached patch [custom] v1 (obsolete) — Splinter Review
This is untested so far, but I'll test before I deploy one way or another (and won't deploy on a friday either)

All branches we care about have this file (including b2g18)
Assignee: nobody → bugspam.Callek
Status: NEW → ASSIGNED
Attachment #733804 - Flags: review?(aki)
Attached patch [configs] v1Splinter Review
Attachment #733807 - Flags: review?(aki)
Attachment #733807 - Attachment is patch: true
Attachment #733807 - Flags: review?(aki) → review+
Comment on attachment 733804 [details] [diff] [review]
[custom] v1

r=me with the promised testing.
Attachment #733804 - Flags: review?(aki) → review+
Attached patch [custom] v1.1Splinter Review
Fixes issues uncovered in testing, including reconfig busting regression from http://hg.mozilla.org/build/buildbotcustom/rev/e854634ca5bb
Attachment #733804 - Attachment is obsolete: true
Attachment #735953 - Flags: review?(aki)
Comment on attachment 735953 [details] [diff] [review]
[custom] v1.1

The interdiff makes sense.
Attachment #735953 - Flags: review?(aki) → review+
This is now pushed, just waiting on a reconf
in production
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.