qvm-* tools should use stderr for diagnostic output

[marmarek]

Reported by joanna on 17 Jul 2011 10:00 UTC
Tested with:

qvm-run --pass_io somevm "cat somelarge.jpg | md5sum"

qvm-run --pass_io somevm "cat somelarge.jpg" | md5sum

They do not return the same checksum! This has been tested on the old qrexec1, not the new qrexec2. I will test the new later today.

Migrated-From: https://wiki.qubes-os.org/ticket/276

[marmarek]

Comment by marmarek on 17 Jul 2011 10:08 UTC
Have you used '-q' switch? Without it there is 'Running command on VM ...' message from qvm-run...

[marmarek]

Comment by marmarek on 17 Jul 2011 10:08 UTC
Have you used '-q' switch? Without it there is 'Running command on VM ...' message from qvm-run...

[marmarek]

Comment by joanna on 17 Jul 2011 10:33 UTC
Ha! Indeed, the -q solves the problem. Has this changes since some previous version?

Anyway, I think we should implicitly assume '-q' when '--pass_io' is specified. This would confuse less users and also is pretty logical IMHO.

[marmarek]

Comment by joanna on 17 Jul 2011 10:33 UTC
Ha! Indeed, the -q solves the problem. Has this changes since some previous version?

Anyway, I think we should implicitly assume '-q' when '--pass_io' is specified. This would confuse less users and also is pretty logical IMHO.

[marmarek]

Comment by marmarek on 17 Jul 2011 11:40 UTC
IMHO this message should be printed to stderr, instead of stdout, which also solves problem.

[marmarek]

Comment by marmarek on 17 Jul 2011 11:40 UTC
IMHO this message should be printed to stderr, instead of stdout, which also solves problem.

[marmarek]

Comment by joanna on 17 Jul 2011 11:41 UTC
Yes, agree.

[marmarek]

Comment by joanna on 17 Jul 2011 11:41 UTC
Yes, agree.

[marmarek]

Comment by joanna on 17 Jul 2011 11:53 UTC
But then, we should really change all the prints in all the qvm-tools to use stderr -- currently they all are using stdout...

[marmarek]

Comment by joanna on 17 Jul 2011 11:53 UTC
But then, we should really change all the prints in all the qvm-tools to use stderr -- currently they all are using stdout...

[marmarek]

Comment by joanna on 17 Jul 2011 11:57 UTC
For now, I just disable verbose when --passio is passed:

http://git.qubes-os.org/?p=joanna/core.git;a=commitdiff;h=4044c2da8be2100f2d81bfa3de869735ada20994

and I will move this ticket to Beta 3 (renaming it)

[marmarek]

Comment by joanna on 17 Jul 2011 11:57 UTC
For now, I just disable verbose when --passio is passed:

http://git.qubes-os.org/?p=joanna/core.git;a=commitdiff;h=4044c2da8be2100f2d81bfa3de869735ada20994

and I will move this ticket to Beta 3 (renaming it)

[marmarek]

Comment by rafal on 17 Jul 2011 19:06 UTC
Another issue is with "-a" flag - when starting a domain, xl prints some "Daemon running with PID xxx" at stdout.

[marmarek]

Comment by rafal on 17 Jul 2011 19:06 UTC
Another issue is with "-a" flag - when starting a domain, xl prints some "Daemon running with PID xxx" at stdout.

[marmarek]

Comment by marmarek on 10 Oct 2011 23:55 UTC
Fixed in qubes core, xl still uses stdout.

http://git.qubes-os.org/gitweb/?p=marmarek/core.git;a=commit;h=98827c7020987290a97a164c4c7c24b8c82e4e05
http://git.qubes-os.org/gitweb/?p=marmarek/core.git;a=commit;h=ed23b0d6a2f6bb8a846577364c14f9fa9a82448c

[marmarek]

Comment by marmarek on 10 Oct 2011 23:55 UTC
Fixed in qubes core, xl still uses stdout.

http://git.qubes-os.org/gitweb/?p=marmarek/core.git;a=commit;h=98827c7020987290a97a164c4c7c24b8c82e4e05
http://git.qubes-os.org/gitweb/?p=marmarek/core.git;a=commit;h=ed23b0d6a2f6bb8a846577364c14f9fa9a82448c

[marmarek]

Comment by marmarek on 13 Oct 2011 10:23 UTC
http://git.qubes-os.org/gitweb/?p=marmarek/xen.git;a=commit;h=6e6d71d4e29d46ad5a1251e16940e098bc2957b3

[marmarek]

Comment by marmarek on 13 Oct 2011 10:23 UTC
http://git.qubes-os.org/gitweb/?p=marmarek/xen.git;a=commit;h=6e6d71d4e29d46ad5a1251e16940e098bc2957b3