For the execution of our tests through mozharness we need a specific version of all the mozbase modules to reduce the risk of version conflicts in our CI and releng systems. As discussed with Armen the best approach here is to add this file to our test repository. That way we can also have branch specific versions of modules corresponding to the Marionette version.

Not sure if Chris is around or not. If he isn't maybe David can review this patch?

Comment on attachment 8658889 [details] github_pull_request.txt I would need more context to do this review. I can't tell exactly the problem this is intending to address.

This is still necessary because of the broken version dependency handling of mozbase packages. There is no-one who supports defining upper limits for API compatibility. So whenever a change happens which breaks API it will bust CI systems. And this is very bad! In-tree framework might not be affected but please keep in mind that we are still outside of the tree and using the pypi packages. So conflicts will not be noticed via try or whatever. See bug 1196447 for details. With pinning the mozbase versions we can ensure that exactly those packages are installed which are known to work. Once we bump versions in setup.py or a new version of a dependency gets released which has a feature we need, we can easily bump.

Comment on attachment 8658889 [details] github_pull_request.txt Thank you David. I think that I can remove the request from Chris now.

Merged PR to master as https://github.com/mozilla/firefox-ui-tests/commit/f91ab2f28da4e62b0fbf79b076e94999dcb7b10e. We need this patch across all branches down to 38esr. Good thing is that we use the same versino of Marionette across all branches. I will do the backport later today.

https://github.com/mozilla/firefox-ui-tests/commit/9d3aba7239cb57f7142f2f11aadf65de20953711 (aurora) https://github.com/mozilla/firefox-ui-tests/commit/87d8cb4d805b16465c8700bee3ffbedd91e771f6 (beta) https://github.com/mozilla/firefox-ui-tests/commit/a99bb21b2570bf0e65b769e490d4d2823cba8df9 (release) https://github.com/mozilla/firefox-ui-tests/commit/a8aaa07cdc3d9203ea5c0f641addaf6d2a7c7044 (esr38) For esr38 there was a merge conflict with .travis.yml which doesn't exist on that branch. I simply removed the file from the merge.