Bug #22401
open
crunch-run --list (with no additional args) should not read from stdin
Added by Tom Clegg 4 months ago.
Updated 4 months ago.
Description
crunch-run --list can be a helpful tool for an admin during troubleshooting. However, that currently hangs waiting to read an instance price list from stdin. crunch-run --list </dev/null works, but an admin should not need to remember that.
Propose changing the default mode to "don't read from stdin" (admin-friendly), and update arvados-dispatch-cloud to pass an additional argument (--stdin=prices?) to enable the price list update behavior.
Tom Clegg wrote:
Propose changing the default mode to "don't read from stdin" (admin-friendly), and update arvados-dispatch-cloud to pass an additional argument (--stdin=prices?) to enable the price list update behavior.
Maybe decide based on stdin isatty?
- Related to Bug #22400: broken paging logic causes arvados-dispatch-cloud to hang at startup while looking up spot prices added
Brett Smith wrote in #note-1:
Maybe decide based on stdin isatty?
Hm. That would address the interactive use case without changing the a-d-c end, although it could still leave surprising behavior for scripts to discover later. Although I don't have a specific problem in mind, I'm also a little worried that ssh/pty stuff could cause this to make the wrong choice eventually.
Given the self-deploy version sync situation, I think an explicit command line argument would be more friendly/predictable.
Also available in: Atom
PDF
Updated by 
Updated by