Skip to content
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

limit number of retries for wireless connect and do exit 0 if unsuccessful #360

Closed
vdelendik opened this issue Mar 15, 2024 · 2 comments
Closed
Labels
enhancement New feature or request
Milestone

Comments

@vdelendik
Copy link
Contributor

as of now we try to connect for hours. let's make 3-5min timeout and do exit 0 if no way to do. 3-5m should cover avg reboot cycle for phone/tablet/tv

Connecting to: 10.151.206.23:5555
03-15 13:46:34.966   101  3845 W adb     : network.cpp:149 failed to connect to '10.151.206.23:5555': No route to host
failed to connect to '10.151.206.23:5555': No route to host
Connecting to: 10.151.206.23:5555
03-15 13:46:42.976   101  3859 W adb     : network.cpp:149 failed to connect to '10.151.206.23:5555': No route to host
failed to connect to '10.151.206.23:5555': No route to host
Connecting to: 10.151.206.23:5555
03-15 13:46:51.029   101  3873 W adb     : network.cpp:149 failed to connect to '10.151.206.23:5555': No route to host
failed to connect to '10.151.206.23:5555': No route to host

@vdelendik vdelendik added the enhancement New feature or request label Mar 15, 2024
@vdelendik vdelendik added this to the 2.0.14 milestone Mar 15, 2024
vdelendik pushed a commit that referenced this issue Mar 15, 2024
vdelendik added a commit that referenced this issue Mar 15, 2024
#361, #360, #359, #327: improved android devices exit and decommissio…
@vdelendik
Copy link
Contributor Author

number of retries limited b y10 with 5 sec interval by default:

while [[ $index -lt 10 ]]; do

changes are available for testing in appium:2.0.14 because I already applied changes on one of the prod as a hot fix

vdelendik added a commit that referenced this issue Mar 15, 2024
#360: fixed isConnected verification for the wireless device
@vdelendik
Copy link
Contributor Author

successfully verified on inaccessible wireless device:

03-15 16:52:24.292    84   208 W adb     : network.cpp:149 failed to connect to '10.146.84.44:5555': No route to host
failed to connect to '10.146.84.44:5555': No route to host
Device 10.146.84.44:5555 is not connected!
Exiting without restarting...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

No branches or pull requests

1 participant