Public Project Meeting - May 06, 2021
Continuing the series of bi-weekly public project meetings, here is the timeline of the meeting held on May 06, 2021,5:30 PM CET. (Below times are on IST)
Meetings are held on the #selenium-tlc channel on Selenium Slack.
Simon Stewart 9:01 PM
Alright then. It’s time
9:02
@AutomatedTester, @titusfortner @p0deje, @jimevans, @harsha509, @barancev, @Puja Jagani, and @diemol (plus anyone else who’s interested)
Let’s get the show on the road
9:02 PM
I think we can wait for @diemol to be back for the stats for the past couple of weeks
Any agenda items?
9:03 PM
- Stats
- What do we want in Beta 4?
- When do we ship Beta 4?
Diego Molina 9:03 PM
Here, quick meeting with my manager done
David Burns 9:04 PM
Managers are the worst…
Simon Stewart 9:04 PM
hands over meeting reigns to @diemol
Diego Molina 9:04 PM
Stats from previous meeting:
- 157 open issues,
- 25 open pull requests
- Current: 163 open issues, 30 open pull requests
Data taken from this link[https://www.diemol.com/github-repo-stats/seleniumhq_selenium/index.html]
David Burns 9:05 PM
A lot of those are from @Puja Jagani
9:05 PM
she has been busy
Simon Stewart 9:06 PM
She’s been rocking it
Diego Molina 9:06 PM
the issues or the PRs?
9:07
Right, work that we want for the next beta (4)
(I could not contribute that much during the last two weeks :confused:)
9:08 PM
These are items from the last meeting:
- Pipe VNC connectors through the WebSocket plumbing for live video
- Investigate: Grid gets stuck when screenshot is taken in a terminated session
- Investigate: Grid gets stuck when the browser driver process cannot be killed
- Investigate: Router request timeouts when forwarding command to Node
- Unify new Session Queue
- Stress test the new Grid on AWS or similar with >50 simultaneous tests
- Check that the Redis/JDBC backed services actually function
- More than 1 IE session per Node? https://github.com/SeleniumHQ/selenium/issues/9388
Maybe only this one got done? “Unify new Session Queue”
Simon Stewart 9:08 PM
I’d like to add:
- Distributor follows spec when allocating new sessions
David Burns 9:08 PM
Do we have issues/docs on the investigate stuff?
Simon Stewart 9:08 PM
Yeah. We unified the new session queue
9:09 PM
I’ve a PR in the works for the “follow the spec” bit
David Burns 9:09 PM
for the last item about IE… we can allow that to happen but it seems like a major footgun
Diego Molina 9:10 PM
There are issues for the investigation items, and maybe there is a PR from @Puja Jagani solving the “browser driver process cannot be killed” one
I will double check that all items have a GitHub issue, and if not, I will create one
David Burns 9:12 PM
thank you
Diego Molina 9:14 PM
For language bindings we had this:
Language bindings
- “se:cdpVersion” is missing in C#, Java and JS
- Can all bindings do CDP over Grid
- Document features and double check all bindings implement all features
Saksham Gupta 9:14 PM
Hello all, been a year since I started my internship. Thought I’ll drop by to say hi.
Simon Stewart 9:15 PM
Java should be doing se:cdpVersion now
David Burns 9:15 PM
@harsha509 do you need a hand with the cdpVersion stuff for JS?
Simon Stewart 9:16 PM
I think we need to review what’s in the python bindings that’s not elsewhere: my impression is that there’s been a few things added
David Burns 9:17 PM
@simonstewart only 1 thing which is mobile options
Simon Stewart 9:17 PM
kk
Sri Harsha 9:17 PM
Yes @AutomatedTester, i have been busy with paid work, didnt got a chance to look into it
Diego Molina 9:17 PM
what are mobile options?
David Burns 9:17 PM
@diemol the ability to set android* config better
you could do it before but I made it a little nicer after a request from whimboo
9:18 PM
I actually want to change it to support iOS better
Diego Molina 9:18 PM
Nice!
David Burns 9:19 PM
but it means you can create a normal chrome/firefox options and pass it through to the driver to connect to said browser on android :+1: 1
9:20 PM
you could before if you knew the magic for it
Diego Molina 9:20 PM
Something else we need to figure out is how to release the IDE, given that @Todd Tarsi is helping us and contributing to it
9:20 PM
does anyone know about it?
Simon Stewart 9:20 PM
Who did the last release? @corevo, I thought
9:20 PM
?
David Burns 9:21 PM
I think it was him. I will email him and Dave for guidance here
Diego Molina 9:21 PM
great, thank you
and I have one last thing from my side
I have spoken to @Puja Jagani and asked her if she is interested in becoming a committer Therefore, I have created this PR https://github.com/SeleniumHQ/seleniumhq.github.io/pull/679 Folks, please go an check it out
I think she has done so much great work and without her we would not have moved at the speed we are currently moving :slightly_smiling_face:
Puja Jagani
Thank you David, Diego and Simon for the constant support and help :relaxed: Very happy working with the Selenium TLC folks!
Simon Stewart 9:25 PM
I’m onboard with this
Diego Molina 9:25 PM
(there is probably a shorter way to say what I said, but my English is no bueno sometimes)
David Burns 9:26 PM
+1 but I am biased :smile:
Diego Molina 9:26 PM
We simply need PR approvals :slightly_smiling_face:
Titus Fortner 9:27 PM
Sorry, I’m showing up late. :slightly_smiling_face:
Did we get the blockers for beta 4?
Have a couple bug fixes that would be nice to get released
Simon Stewart 9:27 PM
@diemol’s in charge here
Diego Molina 9:28 PM
No @titusfortner, sorry. I think we need to improve with PR reviews in the next days so we can get closer to beta 4
9:28 PM
I believe you are talking about the JUnit issue?
Sri Harsha 9:29 PM
i will update few things in ide stuff, electron version bump and updating rules_nodejs
Titus Fortner 9:29 PM
No, that’s a different issue :slightly_smiling_face:
Diego Molina 9:30 PM
Which issues did you have in time?
9:30 PM
in mind*
Titus Fortner 9:32 PM
some fixes in Ruby code
9:32 PM
just wanted to get stuff out there sooner if that’s an option
Simon Stewart 9:33 PM
We’ve yet to agree when we want to ship beta4, so there’s probably time
9:34 PM
Unless you want the fixes to be blockers
Titus Fortner 9:35 PM
I mean, Alex changed something that was blocking someone and I’m wondering when we can get it released to him :smile:
Simon Stewart 9:36 PM
Ah! So you want the release sooner :slightly_smiling_face:
Titus Fortner 9:36 PM
9:36 PM
apparently there is a project that refuses to look at firstMatch values
Sri Harsha 9:36 PM
For JS i want to:
- Implement “se:cdpVersion”
- API documentation for beta 4
Simon Stewart 9:37 PM
@titusfortner I have no idea how a project not following the spec is a reason for us to ship sooner, but I assume there’s context I’m missing….
David Burns 9:37 PM
is this appium?
Titus Fortner 9:40 PM
A Watir user is using aerokube/ggr
Ruby code defaults to F/M, but the project doesn’t accept it (https://github.com/aerokube/ggr/issues/314)
He asked the devs and their private reply: Ggr does not support “sophisticated” selenium “match” rules and we do not plan to implement that, because it is ambiguous
So Ruby changed default implementation to be A/M since it *shouldn’t matter but apparently does
just trying to help the user
Simon Stewart 9:41 PM
It’s the exact opposite of ambiguous
Titus Fortner 9:41 PM
oh, I know
9:41 PM
if this had been a public response i’d have been all over it
Simon Stewart 9:41 PM
WE should probably have a chat with the Aerokube folks
9:42 PM
Because Se4 will break them a lot
Titus Fortner 9:42 PM
The suggestion I got is either to try their new product: aerokube.com/moon or do not use ggr and go with selenium.
Simon Stewart 9:43 PM
So moon supports the w3c payload but not ggr?
Titus Fortner 9:44 PM
that’s what it sounds like
David Burns 9:44 PM
Aerokube… the group that bad mouths selenium… that aerokube?
Titus Fortner 9:44 PM
I don’t know what ggr vs moon is, didn’t get too deep into it
Simon Stewart 9:45 PM
Moon is their paid-for product.
Titus Fortner 9:45 PM
ah of course it is
you want fancy, you pay the money!
s/fancy/sophisticated
Simon Stewart 9:45 PM
A perfectly valid business model
David Burns 9:45 PM
In which they they tell everyone that selenium grid is useless
9:46 PM
I’m happy to help those that don’t bad mouth us
Difference of opinion is also fine
9:47 PM
But… *channels his inner @jimevans *
Titus Fortner 9:47 PM
I don’t remember off hand which bindings default to A/M and which to F/M
9:47 PM
I was going to run some tests
Diego Molina 9:50 PM
I think we should not hurry for a project that does not support standards
Titus Fortner 9:51 PM
I don’t care about them
9:51 PM
I care about a user who needs to change because of them
Diego Molina 9:51 PM
Anyway, I think those were all the topics we had :slightly_smiling_face:
