1143565 - Release marionette-driver 0.3.1 and marionette-client 0.9.2

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

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

We need new packages released to pypi for both driver and client.

With the new versions and all hard-blockers fixed we sould be able get the Firefox UI Tests running in the former Mozmill CI system by simply depending on packages on pypi and not the latest state in mozilla-central for the environments.

CC'ing a bunch of people who also might have a list of blockers they need to be included.

Comment 1

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

David and Dave, do you both know of anything else we would need for the upcoming releases of those two packages?

Comment 2

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

Nothing that I'm aware of.

Comment 3

[David Burns :automatedtester]

Attachment: MozReview Request: bz://1143565/AutomatedTester

/r/5907 - Bug 1143565: Bump marionette driver to v 0.3; r=chmanchester
/r/5909 - Bug 1143565: Bump marionette client to v 0.9.1; r=chmanchester

Pull down these commits:

hg pull review -r 31f3ed16c624797c42eba1a9a7a291ce9474173f

Comment 4

[Chris Manchester (limited bugmail, email directly)]

Comment on attachment 8582059 [details]
MozReview Request: bz://1143565/AutomatedTester

https://reviewboard.mozilla.org/r/5905/#review4895

Ship It!

Comment 5

[David Burns :automatedtester]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3f4f37b1b910
https://hg.mozilla.org/integration/mozilla-inbound/rev/8673899b8229

Comment 6

[David Burns :automatedtester]

pushed to pypi, a new version may need to be released depending on Wr results

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Backed out due to the pypi issues (retriggers were still red):
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/fb3ccc24e257
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/f443567a8237

Comment 8

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

The patch for marionette-client is broken because it does not bump the version of the compatible marionette driver in the requirements.txt file. IMO using that file makes it very easy to miss those dependencies. Why don't we include those deps directly in the setup.py?

Comment 9

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

Also I thought we wanted to release driver version 0.2.1, but now we have 0.3. David please update the summary with the new versions you want to push for the next releases to stop breakage.

I will pull of those broken releases from pypi for the time being.

Comment 10

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

I removed both already uploaded packages from pypi to stop breakage for systems using those packages.

Comment 11

[David Burns :automatedtester]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7ff2810678d6

Comment 12

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

David, I still think your latest push is not correct:

> -marionette-transport == 0.4
> -marionette-driver == 0.2
> +marionette-transport >= 0.4
> +marionette-driver >= 0.2

This would still allow the new client to use a driver of version 0.2. Aren't those versions incompatible to each other due to at least one of the landed changes since the last release?

Comment 13

[David Burns :automatedtester]

Where is the runner not compatible with the driver? I can't find any code that suggests that. The whole point of splitting them out is that they could have their own release cycles.

As for the reason it was backed out, the limit on 0.2 (shown below) meant that it was not found when installing (since it was not in the tree)
> -marionette-driver == 0.2

Comment 14

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

Maybe I was wrong then that there was such a change. Sorry.

Regarding the versions, you might have problems uploading the files to pypi because it will not be allowed. Dave told me today a workaround. So if uploading them as tar.gz doesn't work, you can use .zip. Otherwise we need new versions for both.

Comment 15

[David Burns :automatedtester]

I have bumped the minor number to prevent us doing any hacks, if the try push works will update the summary

Comment 16

[David Burns :automatedtester]

Comment on attachment 8582059 [details]
MozReview Request: bz://1143565/AutomatedTester

/r/5907 - Bug 1143565: Bump marionette driver to v 0.3.1; r=chmanchester
/r/5909 - Bug 1143565: Bump marionette client to v 0.9.2; r=chmanchester
/r/5995 - Bug 1143565: Change requirements to not be fixed to a specific marionette transport and driver; r=chmanchester

Pull down these commits:

hg pull review -r 84d13c1f9481caaa6d0fdf6c704198bf472d71c6

Comment 17

[Chris Manchester (limited bugmail, email directly)]

Comment on attachment 8582059 [details]
MozReview Request: bz://1143565/AutomatedTester

https://reviewboard.mozilla.org/r/5905/#review4955

Ship It!

Comment 18

[David Burns :automatedtester]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6b4626d41fa3
https://hg.mozilla.org/integration/mozilla-inbound/rev/6edfb587504f
https://hg.mozilla.org/integration/mozilla-inbound/rev/0c7cb48aed86

Comment 19

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6b4626d41fa3
https://hg.mozilla.org/mozilla-central/rev/6edfb587504f
https://hg.mozilla.org/mozilla-central/rev/0c7cb48aed86

Comment 20

[David Burns :automatedtester]

Changelog for marionette-driver

Bug 1129702 - Add support for double clicks to marionette's action chains.;r=automatedtester
Bug 1097705 - Add ability to right and middle click to marionette's action chains.;r=automatedtester
Bug 1108590: Update key that is used to return elements from marionette to the client; r=chmanchester
Bug 1137388 - Add a facility to restart firefox from marionette from within the browser for update tests.;r=automatedtester
Bug 1141679 - Ignore the server's response when quitting the application with marionette.;r=jgriffin
Bug 1142404 - Fix marionette's restarts to keep from attempting to reconnect to the browser as it's shutting down.;r=dburns
Bug 1143565: Bump marionette driver to v 0.3.1; r=chmanchester

Comment 21

[David Burns :automatedtester]

Changelog for marionette-client

Bug 1129702 - Add support for double clicks to marionette's action chains.;r=automatedtester
Bug 1097705 - Add ability to right and middle click to marionette's action chains.;r=automatedtester
Bug 1136567 - Marionette test for selection carets' positions after changing orientation of device. r=Automatedtester
Bug 1109183 - Fix imports of the marionette client and remove spurious entry from sys.path provided by mach. r=ahal
Bug 1138839 - Part1 - Marionette test for selecting text inside an iframe. r=automatedtester
Bug 1138839 - Part2  - Fix naming issue in marionette test for selection carets. r=automatedtester
Bug 1137653: If there is a crash don't try any more marionette commands as they will just cause unnecessary errors; r=jgriffin
Bug 1134872 - Fixing getElementValueOfCssProperty to support chrome elements. r=dburns
Bug 1137388 - Add a facility to restart firefox from marionette from within the browser for update tests.;r=automatedtester
Bug 1025125 - If enabled, enter pdb session when test fails. r=ahal
Bug 1096488 - Unskip marionette test navigating to non-remote pages.;r=automatedtester
Bug 1096488 - Test that switching browser remoteness leaves marionette in a usable state.;r=automatedtester
NO BUG: correct import in marionette about_pages tests r=me DONTBUILD
Bug 1141129 - BaseMarionetteTestRunner.__init__ should initialize self.prefs as dictionary. r=dburns
Bug 1141679 - Ignore the server's response when quitting the application with marionette.;r=jgriffin
Bug 1142344 - Remove a racy assertion added in marionette test for frequently failing on a pgo build. r=jgriffin
Bug 1144240 - Fix generated Marionette documentation. r=ato
Bug 1144907 - Restore previous context after calling marionette's gather_debug. r=davehunt
Bug 1135846 - Expose marionette's actions code to chrome scope where applicable. r=dburns
Bug 1143565: Bump marionette client to v 0.9.2; r=chmanchester
Bug 1143565: Change requirements to not be fixed to a specific marionette transport and driver; r=chmanchester

Comment 22

[David Burns :automatedtester]

changelog generated with hg log -M -r dcc86f78d75e:: --template "{desc|firstline}\n" . in driver and client directory

Comment 23

[David Burns :automatedtester]

Changelog

Bug 1154691: Align Marionette with WebDriver errors
Bug 1083131: Always remove a profile created by marionette when the runner shuts down.
Bug 1154681: Use static lookups in errors.py
Bug 1154525: Make HTMLElement#location and #size use #rect internally
Bug 1157665: Add mozrunner dependency to marionette_driver
Bug 1158219: Don't set `id' field if undefined when switching frame

Comment 25

[David Burns :automatedtester]

Attachment: MozReview Request: Bug 1143565: Bump marionette driver to v 0.3.1; r=chmanchester

Comment 26

[David Burns :automatedtester]

Attachment: MozReview Request: Bug 1143565: Change requirements to not be fixed to a specific marionette transport and driver; r=chmanchester

Comment 27

[David Burns :automatedtester]

Attachment: MozReview Request: Bug 1143565: Bump marionette client to v 0.9.2; r=chmanchester

Comment 28

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

The fix for bug 1141519 was not part of this release. So removing.