"cit logdog cat" does not timeout on invalid log path |
||||
Issue description
cit logdog cat chromium/bb/chromium.gpu.fyi/Linux_Release__AMD_R7_240_/1928/+/recipes/steps/webgl2_conformance_tests/0/logs/json.output/0
[I2017-04-25T00:49:07.292307-04:00 124799 0 fetcher.go:366] No logs returned. Sleeping... {"delay":"5s", "index":0}
[I2017-04-25T00:49:12.417524-04:00 124799 0 fetcher.go:366] No logs returned. Sleeping... {"delay":"5s", "index":0}
[I2017-04-25T00:49:17.580121-04:00 124799 0 fetcher.go:366] No logs returned. Sleeping... {"delay":"5s", "index":0}
[I2017-04-25T00:49:22.708707-04:00 124799 0 fetcher.go:366] No logs returned. Sleeping... {"delay":"5s", "index":0}
...
It also goes into an infinite loop when I try a definitely non-exist path.
If the log doesn't exist or there's some reason why I can't get it, there should be an error message.
I'm trying to write a script to download a series of logs, e.g.
for i in 1928 .. 2006
chromium/bb/chromium.gpu.fyi/Linux_Release__AMD_R7_240_/$i/+/recipes/steps/webgl2_conformance_tests/0/logs/json.output/0
and it seems I have to start all the downloads, wait a while, then kill the ones that are hung.
,
Jul 11 2017
->P3, FR, Available. That said, as Dan said, it's by design so I'm tempted to mark as WontFix. Albeit, it could be interesting to add basic validation that at least a prefix of the path is valid but I don't know if it is possible at all.
,
Jul 12
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 19
Closing as WontFix as per #1 and since it's gone for quite a while without activity. If there's renewed interest in having a built-in feature for validating a stream is preexisting, we can reopen or file a new feature request. Thanks! |
||||
►
Sign in to add a comment |
||||
Comment 1 by d...@chromium.org
, Apr 25 2017