[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user mephi42 commented on the issue: https://github.com/apache/zeppelin/pull/2816 Hi @jongyoul, sorry, I'm afraid I won't have time to finish this anytime soon. That said, the first 7 patches, which improve stability and performance of the test suite, are ready as far as I'm concerned. Maybe you could give them a try and get them in in case they look good? ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2816 @mephi42 ping. Do you have any progress on it? ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2816 ping @mephi42 ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user mephi42 commented on the issue: https://github.com/apache/zeppelin/pull/2816 Another update. This is now more than just FF>=55 disablement, so maybe I should rename the pull request. I tried to split the work into meaningful chunks, so if you think some commits are worth integrating right away, I can send them as separate pull requests. ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2816 I understand Selenium, Gecko, WebExtensions are constantly evolving. However, with this (https://github.com/apache/zeppelin/pull/2826/files), I'm able to make all the Selenium test case pass on FF-58. This PR is still work in progress. Also, there is stuff that will get fixed with FF-59. ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2816 Thanks @mephi42 looking forward more progress. ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user mephi42 commented on the issue: https://github.com/apache/zeppelin/pull/2816 I just pushed my WIP branch. With those changes things look better locally, but there are still issues. ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user mephi42 commented on the issue: https://github.com/apache/zeppelin/pull/2816 @zjffdu I'm currently trying to stabilize them on my local machine. There are still a couple things that need debugging - the worst one being that sometimes click() on e.g. Login button produces no effect. @felixcheung, to me this is a useful safeguard when running locally and new Firefox is by accident in $PATH. ---
[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2816 @mephi42 It still fails, could you help take a look at it ? The selenium test is flaky, very appreciated you could help on that. ---