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.

Sign up for GitHub

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

Jump to bottom

adb screenrecord process can't be stopped on device #244

Closed
vdelendik opened this issue Sep 6, 2023 · 2 comments
Closed

adb screenrecord process can't be stopped on device #244

vdelendik opened this issue Sep 6, 2023 · 2 comments
Labels
bug Something isn't working
Milestone
2.0.4

Comments

@vdelendik
Copy link
Contributor

In some scenarios we see that screenrecord continue to record videos and occupy space on device. As result it occupy all space too fast.
As a hotfix we have to restore agai handler for below log message:

root@456174331aa9:/home/androidusr# cat /tmp/log/appium.log | grep "c826141c-3d34-4ff0-87db-9f65cad80ea2"
2023-09-04 14:12:37:309 [AndroidUiautomator2Driver@9897 (c826141c)] [AndroidUiautomator2Driver@9897 (c826141c)] Session created with session id: c826141c-3d34-4ff0-87db-9f65cad80ea2
2023-09-04 14:13:27:265 [AppiumDriver@64d4] [AppiumDriver@64d4] New AndroidUiautomator2Driver session created successfully, session c826141c-3d34-4ff0-87db-9f65cad80ea2 added to master session list
2023-09-04 14:13:27:308 [AndroidUiautomator2Driver@9897 (c826141c)] [AndroidUiautomator2Driver@9897 (c826141c)] Cached the protocol value 'W3C' for the new session c826141c-3d34-4ff0-87db-9f65cad80ea2
2023-09-04 14:16:27:404 [AppiumDriver@64d4] [AppiumDriver@64d4] Removing session 'c826141c-3d34-4ff0-87db-9f65cad80ea2' from our master session list

it guarantees we stop recording during negative scenario when appium don't send delete request.

more stable solutions are under investigation in scope of #235 and #195
@vdelendik vdelendik added the bug Something isn't working label Sep 6, 2023
@vdelendik vdelendik added this to the 2.0.4 milestone Sep 6, 2023
vdelendik pushed a commit that referenced this issue Sep 6, 2023
#244: improved finish sesison id detection for removed from master li…
b212054 
…st sessions
@vdelendik
Copy link
Contributor Author

improvement is available in appium:2.0.4-beta2. @dhreben, I'm going to test it on my own as a hotfix for problematic device

@vdelendik
Copy link
Contributor Author

everything is good, no complains

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
Device Farm (MCloud)
Archived in project
Milestone
2.0.4
Development

No branches or pull requests
1 participant
@vdelendik