1118201 - Update gaia imports to check if it has marionette_driver package, if not use marionette still

Status: RESOLVED FIXED

(Testing :: Marionette Client and Harness, defect, P1)

Description

[David Burns :automatedtester]

This allows us to break things up into modular parts and makes things easier to ship

Comment 1

[David Burns :automatedtester]

Attachment: PR 27288: handle new marionette import

Comment 2

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 8546942 [details] [review]
PR 27288: handle new marionette import

Sorry for the delay in getting to this. It's quite an extensive change, and I'm guessing that it will be temporary - ultimately moving over to the marionette_driver package once it's available. I wonder if there's a way to do this in one go without the try/except? Could we release a new version of marionette-client with marionette_driver available? Then a future release would remove this in favour of the separately installable package?

Comment 3

[David Burns :automatedtester]

Currently I can't land marionette_driver, therefore release it, because it is failing gaiatests on treeherder. I think having the try/except allows us to upgrade everything everywhere and then release and if people don't update then it uses one or other and we can notify that in the next release they MUST update. 

At some point we are going to have to update the gaiatests to use marionette_driver package, I think this way does it nicely and people won't be aware of it when it first lands.

Comment 4

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Couldn't the current marionette client offer a marionette_driver package that essentially just masquerades as the marionette package? That way you could just switch to marionette_driver before the separate driver package is ready to ship. I just think a try/except for every file is a little ugly, but if you don't feel like we can avoid it then let's continue with this approach.

I think a test maintainer would be better to review this patch though, as it will affect imports for any patches in review or to be written. Also, this will rot quickly so will need some coordination to land it.

Comment 5

[Florin Strugariu [:Bebe]]

Comment on attachment 8546942 [details] [review]
PR 27288: handle new marionette import

Except for the 2 non-blocking nits all looks OK to me.

Comment 6

[Florin Strugariu [:Bebe]]

https://github.com/mozilla-b2g/gaia/commit/8087bb27e3d10700b69413438045fd6ac6eab312

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Moving bugs for Marionette client due to component changes.