Closed Bug 1121187 Opened 8 years ago Closed 7 years ago

[raptor] develop b2g-restart test


(Firefox OS Graveyard :: Gaia::PerformanceTest, defect)

Gonk (Firefox OS)
Not set


(Not tracked)



(Reporter: rwood, Assigned: rwood)




(6 files, 1 obsolete file)

The existing raptor reboot test (boot_test.js) doesn't work on the b2g-emulator, because 'adb reboot' is not supported on the emulator. It is not possible to do a simulation of a hard reboot on the emulator.

I was considering on emulator doing a soft-reset (b2g restart) instead of a reboot. Not quite the same but I believe measuring the b2g process restart time would still provide significant value for an FXOS perf test (and just using it to establish a baseline to detect regressions; not as actual real reboot performance numbers).

Eli, I was thinking I would add a new method in /lib/device/index.js called "rebootEmulator", which would do the same as "reboot" with regards to markers etc. however insead of an 'adb reboot' it would just do a b2g restart (adb stop b2g && adb start b2g) and would do a 'waitForHomescreen" (instead of "adb wait-for-device since when just restarting b2g the emulator is always running).

Then in the /lib/suite/reboot.js itself, if options.emulator is set then invoke 'rebootEmulator' instead. 

What do you think?  Or should I create a /lib/emulator/index.js (although lots will be duplicate)? Or should I create a new perf test called 'reset_test', which would work on both device and emulator, and then we just won't run the existing 'boot_test' on the emulator? Thanks in advance for your thoughts!
Flags: needinfo?(eperelman)
I think adding a new suite phase for only restarting B2G is in order. It would function similar to the Reboot suite, but rather than doing the device reboot it would do the B2G restart. Most of the code for this to happen already exists spread through Reboot and ColdLaunch, so it may also be in order to push some of that into the base Phase for re-use. Then we can create a new test separate from reboot_test (e.g. b2g_restart_test) which would consume the new phase and be run by the simulator.

Does that sound reasonable?
Flags: needinfo?(eperelman)
Yes that sounds great. I will put a patch together for this, but will wait until the MozDevice work (Bug 1115134) has landed.
Summary: [raptor] reboot test doesn't work on b2g-emulator → [raptor] develop b2g-restart test
Blocks: 1129096
Add restart-b2g phase to raptor.
Attachment #8559811 - Flags: review?(eperelman)
Add restart-b2g raptor test.
Attachment #8559813 - Flags: review?(eperelman)
Comment on attachment 8559811 [details] [review]

My main concern is just getting the time from the device rather than overriding with the time from the Node process. Mozdevice will need to be modified to fetch the device time during the B2G restart process.

Other than that, just a couple nits to address.
Attachment #8559811 - Flags: review?(eperelman) → review-
Attachment #8559812 - Attachment is obsolete: true
Attachment #8559813 - Flags: review?(eperelman) → review+
Comment on attachment 8559811 [details] [review]

Thanks for the review Eli. PR updated, I'll squash the commits after review.
Attachment #8559811 - Flags: review- → review?(eperelman)
Corresponding change to mozdevice, to return device time before b2g restart.
Attachment #8560539 - Flags: review?(eperelman)
Comment on attachment 8560539 [details] [review]

Before merging this, could you also include a bump to the package.json to 1.2.2?
Attachment #8560539 - Flags: review?(eperelman) → review+
(In reply to :Eli Perelman from comment #9)
> Comment on attachment 8560539 [details] [review]
> Before merging this, could you also include a bump to the package.json to
> 1.2.2?

Right, I had that change locally but forgot to add it to my commit, d'oh, anyway thanks, done & merged:
Comment on attachment 8559811 [details] [review]

PR updated as per our conversation on IRC (and your github gist for re-organizing phase.format).
Comment on attachment 8559811 [details] [review]

Awesome job on this. R+ with nits fixed.
Attachment #8559811 - Flags: review?(eperelman) → review+
Thanks Eli. Rebased, nits fixed, squashed and merged:
Update gaia-node-modules.
Attachment #8561454 - Flags: review?(kgrandon)
Comment on attachment 8561454 [details] [review]

Sounds good. Feel free to update the gaia revision as well.
Attachment #8561454 - Flags: review?(kgrandon) → review+
(In reply to Autolander from comment #18)
> Created attachment 8561590 [details] [review]
> [PullReq] rwood-moz:bug1121187 to mozilla-b2g:master

Green try run. Carrying fw the R+ from Eli, and have R+ from Kevin. Merged:
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.