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

Aborted sessions don't kill requests to grid #5

Closed
VitaliLiavitski opened this issue Dec 6, 2019 · 3 comments
Closed

Aborted sessions don't kill requests to grid #5

VitaliLiavitski opened this issue Dec 6, 2019 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@VitaliLiavitski
Copy link

If we aborte Jenkins job, requests have status in progress for using phones and simulators

@vdelendik vdelendik added this to the 1.0 milestone Nov 23, 2020
@vdelendik vdelendik added the bug Something isn't working label Nov 23, 2020
@vdelendik
Copy link
Collaborator

it might be not a bug but let's investigate if we can handle somehow abort use-case on hub layer. By default aborted sessions should be disconnected by timeout.
Pay attention to the adb connection which might occupy device for longer time than timeout

@vdelendik vdelendik removed this from the 1.0 milestone Dec 10, 2020
@vdelendik vdelendik added this to the 2.0 milestone Nov 28, 2021
@vdelendik vdelendik modified the milestones: 2.0, 2.1 Jan 23, 2022
@vdelendik
Copy link
Collaborator

fixed by #54.
testing scenario should be:

  1. start iOS test or job via single device (ask @vdelendik to prepare explicit branch with rather slow steps)
  2. kill iOS jobs in the middle and start timer to track delay
    -> iOS device is occupied either in mcloud/grid/console and in stf
  3. start again iOS job vs the same device
    -> job is started, request added to queue as device still occupied in grid
  4. tail appium device log to detect:
2022-01-24 23:02:25:519 - [HTTP] --> DELETE /wd/hub/session/60cd4cef-c063-4ecb-8beb-bf422598784c
2022-01-24 23:02:25:519 - [HTTP] {}
2022-01-24 23:02:25:521 - [debug] [W3C (60cd4cef)] Encountered internal error running command: NoSuchDriverError: A sessio                                                                                                                   n is either terminated or not started

-> as only previous session removed in appium (1-4 min) new iOS job should occupy device and start testing

So goal to identify that pause in occupation is upto 4 minutes instead of 60m as in v1.0

@vdelendik
Copy link
Collaborator

tested on my own

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants