Re: [libvirt] [PATCH 03/11] vsh: Add @silent to vshConnectionHook

On Tue, Nov 07, 2017 at 01:22:51PM +0100, Michal Privoznik wrote: I guess debug messages probably need to be filtered too. Actually allocation errors should be covered as well. And you missed some. It's fine as it is, since no auto-completion is perfect, I use lot of them extensively and I must say I don't care if it outputs something when some error happens. The nice thing about auto-completion is that if it does not work at all or works differently (outputs something it "shouldn't") nothing happens. There is no data loss, broken migrations or whatever. Anyway looking through all the things that you are modifying here and to those that could still be modified, I think this could be approached a bit better. As I said it's kind of fine if we keep it like this for now, but it could be even better. Consider this: - we do not modify what messages are reported - we leave the function that manages log output to decide - when the completer is called we set it up so that there is no log output Guess what, we have first two things in place and for the third one you can just call vshCloseLogFile(ctl) and you're golden ;) If you want to make it even better, you can already do that before you get to any completer. For example if you specify '-q' multiple times on the command line it could switch to super quiet mode, e.g.: diff --git i/tools/virsh.c w/tools/virsh.c index f830331f6bbe..6b7eafeba0be 100644 --- i/tools/virsh.c +++ w/tools/virsh.c @@ -782,6 +782,8 @@ virshParseArgv(vshControl *ctl, int argc, char **argv) vshOpenLogFile(ctl); break; case 'q': + if (ctl->quiet) + vshCloseLogFile(ctl); ctl->quiet = true; break; case 't': -- And just call virsh from the bash completion with '-qq' And the same for virt-admin of course. And maybe document it :)