Bug #11519

arv-get should abort on ctrl/C

Added by Tom Morris 6 months ago. Updated 9 days ago.

Status:NewStart date:04/19/2017
Priority:NormalDue date:
Assignee:Lucas Di Pentima% Done:

0%

Category:-
Target version:2017-10-25 Sprint
Story points-Remaining (hours)0.00 hour
Velocity based estimate-

Description

1423 MiB / 43967 MiB 3.2%^CTraceback (most recent call last):
File "/home/tfmorris/venv/local/lib/python2.7/site-packages/arvados/keep.py", line 480, in _headerfunction
def _headerfunction(self, header_line):
KeyboardInterrupt
2255 MiB / 43967 MiB 5.1%^CTraceback (most recent call last):
File "/home/tfmorris/venv/local/lib/python2.7/site-packages/arvados/keep.py", line 480, in _headerfunction
def _headerfunction(self, header_line):
KeyboardInterrupt
2319 MiB / 43967 MiB 5.3%^C^C^CTraceback (most recent call last):
File "/home/tfmorris/venv/local/lib/python2.7/site-packages/arvados/keep.py", line 480, in _headerfunction
def _headerfunction(self, header_line):
KeyboardInterrupt
^C^C^C^C^C^C^CTraceback (most recent call last):
File "/home/tfmorris/venv/local/lib/python2.7/site-packages/arvados/keep.py", line 480, in _headerfunction
def _headerfunction(self, header_line):
KeyboardInterrupt
^CTraceback (most recent call last):
File "/home/tfmorris/venv/local/lib/python2.7/site-packages/arvados/keep.py", line 480, in _headerfunction
def _headerfunction(self, header_line):
KeyboardInterrupt
^CTraceback (most recent call last):
File "/home/tfmorris/venv/local/lib/python2.7/site-packages/arvados/keep.py", line 480, in _headerfunction
def _headerfunction(self, header_line):
KeyboardInterrupt
^C


Subtasks

Task #11571: ReviewNewTom Clegg

History

#1 Updated by Lucas Di Pentima 6 months ago

  • Assignee set to Lucas Di Pentima

#2 Updated by Lucas Di Pentima 5 months ago

  • Status changed from New to In Progress

#3 Updated by Lucas Di Pentima 5 months ago

I haven't been able to reproduce the problem using a local virtualenv and PySDK at master.

Nevertheless, I've found a pycurl bug report that seem to be similar to this issue: https://github.com/pycurl/pycurl/issues/413

The odd thing is that the code listed to reproduce the problem seem to work (that is, fails as reported) on my dev environment.

#4 Updated by Lucas Di Pentima 5 months ago

  • Target version changed from 2017-05-10 sprint to 2017-05-24 sprint

#5 Updated by Lucas Di Pentima 5 months ago

  • Target version changed from 2017-05-24 sprint to 2017-06-07 sprint

#6 Updated by Lucas Di Pentima 4 months ago

  • Target version changed from 2017-06-07 sprint to 2017-06-21 sprint

#7 Updated by Tom Morris 4 months ago

  • Assignee changed from Lucas Di Pentima to Tom Morris

#8 Updated by Tom Morris 4 months ago

  • Target version changed from 2017-06-21 sprint to 2017-07-05 sprint

#9 Updated by Tom Morris 4 months ago

This sounds not just similar, but identical, to me (including the fact that it occurs with HEADERFUNCTION): https://github.com/pycurl/pycurl/issues/413#issuecomment-276954718

The workaround: https://github.com/lorien/grab/commit/f26fd3052aaa9d9b24f1098759d73dcc2c00d7fa#diff-50f780fd1dcf1fa66656abf741a55c36L489

seems a little convoluted to me, so I'm not 100% sure we want to go down that path, but perhaps you could discuss with Tom and/or Peter to brainstorm ideas for a possible solution.

#10 Updated by Tom Morris 4 months ago

  • Assignee changed from Tom Morris to Lucas Di Pentima

#11 Updated by Lucas Di Pentima 4 months ago

  • Target version changed from 2017-07-05 sprint to 2017-07-19 sprint

#12 Updated by Lucas Di Pentima 3 months ago

  • Status changed from In Progress to New
  • Target version changed from 2017-07-19 sprint to 2017-08-02 sprint

#13 Updated by Tom Morris 3 months ago

  • Target version changed from 2017-08-02 sprint to 2017-08-16 sprint

#14 Updated by Lucas Di Pentima 2 months ago

  • Target version changed from 2017-08-16 sprint to 2017-08-30 Sprint

#15 Updated by Lucas Di Pentima about 1 month ago

  • Target version changed from 2017-08-30 Sprint to 2017-09-13 Sprint

#16 Updated by Lucas Di Pentima about 1 month ago

  • Target version changed from 2017-09-13 Sprint to 2017-09-27 Sprint

#17 Updated by Peter Amstutz 24 days ago

It sounds like we need to do something like this in order to detect ^C properly.

#18 Updated by Lucas Di Pentima 23 days ago

  • Target version changed from 2017-09-27 Sprint to 2017-10-11 Sprint

#19 Updated by Lucas Di Pentima 9 days ago

  • Target version changed from 2017-10-11 Sprint to 2017-10-25 Sprint

Also available in: Atom PDF