-
Notifications
You must be signed in to change notification settings - Fork 994
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #36978 - Add possibility to use remote webdriver #9952
Fixes #36978 - Add possibility to use remote webdriver #9952
Conversation
Can one of the admins verify this patch? |
2 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
ok to test |
@MariaAga would you mind looking at this? Looks totally reasonable to me, but my knowledge around the integration tests is rather low. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've opened #9962 which should provide a cleaner base for this PR.
9529ef0
to
f89c5ac
Compare
I have no idea why the labeler fails maybe someone can help? |
that's just broken, ignore it for now :) |
826ba53
to
40b0f2a
Compare
c369a67
to
ac3b724
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small nitpicks.
A bigger question here is how we're testing in GitHub Actions today and if this would actually be better. I don't know the current chromedriver & chrome are installed.
ac3b724
to
d44b14f
Compare
59b757a
to
f871fff
Compare
bb361dc
to
72a9c95
Compare
Any news on this? |
@ekohl what do we need to do to get this forward? |
ping |
@ekohl Do you need any additional information on the answers from above? |
72a9c95
to
faf5a31
Compare
faf5a31
to
8577c35
Compare
8577c35
to
76f1da7
Compare
76f1da7
to
62554a2
Compare
62554a2
to
ba6bed9
Compare
Looks like, this does not work @dosas . Use the previous implementation?
` |
ba6bed9
to
6d3bc85
Compare
Failed tests are not related. |
In order to easily execute the selenium tests it would be good to have the possibility to use selenium remote webdriver images
To access the driver via vagrant or a CI docker image the IP address needs to be set using an evironment variable:
There are certain environments that require different chrome options to work so it is possible to add them via semicolon separated string as environment variable: