Project

General

Profile

Actions

Bug #5016

closed

[SDKs] Should be able to use ^C to interrupt arv-ws.

Added by Peter Amstutz about 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Story points:
0.5

Subtasks 1 (0 open1 closed)

Task #5017: Review 5016-ctrl-c-arv-wsResolvedPeter Amstutz01/21/2015Actions

Related issues

Has duplicate Arvados - Bug #4507: [SDKs] arv ws should exit on ctrl-cDuplicateActions
Actions #1

Updated by Peter Amstutz about 9 years ago

  • Assigned To set to Peter Amstutz
Actions #2

Updated by Ward Vandewege about 9 years ago

  • Subject changed from Should be able to use ^C to interrupt arv-ws. to [SDKs] Should be able to use ^C to interrupt arv-ws.
Actions #3

Updated by Brett Smith about 9 years ago

691d642 does the job, but it can be surprisingly sluggish. Here's a transcript where I hit ^C as soon as I saw the fallback notices:

brinstar % arv-ws; date
2015-01-21 10:12:03 arvados.events[21214] WARNING: Got exception _ssl.c:331: No root certificates specified for verification of other-side certificates. trying to connect to websockets at wss://ws.qr1hi.arvadosapi.com/websocket
2015-01-21 10:12:03 arvados.events[21214] WARNING: Websockets not available, falling back to log table polling
^CWed Jan 21 10:12:16 EST 2015

Taking about 10-13 seconds is what I typically saw, independently of how I set the poll interval. If there's a way to make it snappier, that seems like it'd be nice, but if not, this is good to merge. Thanks.

Actions #4

Updated by Brett Smith about 9 years ago

Okay, I see now that what's going on in my last comment is that the thread is blocking on the first large API response of logs, and that ^C reacts more like expected after that. So that leaves me less concerned and I'm happy to see the branch merged as-is.

Actions #5

Updated by Peter Amstutz about 9 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:ffcfb73e6a5dfd626b1d523accc42e7a6af5e8ed.

Actions

Also available in: Atom PDF