* [PATCH RESEND] tools/kvm_stat: fix display of error when multiple processes are found
@ 2022-06-14 12:11 Dmitry Klochkov
2022-06-15 12:14 ` Paolo Bonzini
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Klochkov @ 2022-06-14 12:11 UTC (permalink / raw)
To: kvm, linux-kernel
Cc: Paolo Bonzini, Stefan Raspl, Christian Borntraeger, Dmitry Klochkov
Instead of printing an error message, kvm_stat script fails when we
restrict statistics to a guest by its name and there are multiple guests
with such name:
# kvm_stat -g my_vm
Traceback (most recent call last):
File "/usr/bin/kvm_stat", line 1819, in <module>
main()
File "/usr/bin/kvm_stat", line 1779, in main
options = get_options()
File "/usr/bin/kvm_stat", line 1718, in get_options
options = argparser.parse_args()
File "/usr/lib64/python3.10/argparse.py", line 1825, in parse_args
args, argv = self.parse_known_args(args, namespace)
File "/usr/lib64/python3.10/argparse.py", line 1858, in parse_known_args
namespace, args = self._parse_known_args(args, namespace)
File "/usr/lib64/python3.10/argparse.py", line 2067, in _parse_known_args
start_index = consume_optional(start_index)
File "/usr/lib64/python3.10/argparse.py", line 2007, in consume_optional
take_action(action, args, option_string)
File "/usr/lib64/python3.10/argparse.py", line 1935, in take_action
action(self, namespace, argument_values, option_string)
File "/usr/bin/kvm_stat", line 1649, in __call__
' to specify the desired pid'.format(" ".join(pids)))
TypeError: sequence item 0: expected str instance, int found
To avoid this, it's needed to convert pids int values to strings before
pass them to join().
Signed-off-by: Dmitry Klochkov <kdmitry556@gmail.com>
---
tools/kvm/kvm_stat/kvm_stat | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/kvm/kvm_stat/kvm_stat b/tools/kvm/kvm_stat/kvm_stat
index 5a5bd74f55bd..9c366b3a676d 100755
--- a/tools/kvm/kvm_stat/kvm_stat
+++ b/tools/kvm/kvm_stat/kvm_stat
@@ -1646,7 +1646,8 @@ Press any other key to refresh statistics immediately.
.format(values))
if len(pids) > 1:
sys.exit('Error: Multiple processes found (pids: {}). Use "-p"'
- ' to specify the desired pid'.format(" ".join(pids)))
+ ' to specify the desired pid'
+ .format(" ".join(map(str, pids))))
namespace.pid = pids[0]
argparser = argparse.ArgumentParser(description=description_text,
base-commit: 6cd88243c7e03845a450795e134b488fc2afb736
--
2.35.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH RESEND] tools/kvm_stat: fix display of error when multiple processes are found
2022-06-14 12:11 [PATCH RESEND] tools/kvm_stat: fix display of error when multiple processes are found Dmitry Klochkov
@ 2022-06-15 12:14 ` Paolo Bonzini
2022-06-15 17:30 ` Dmitry Klochkov
0 siblings, 1 reply; 3+ messages in thread
From: Paolo Bonzini @ 2022-06-15 12:14 UTC (permalink / raw)
To: Dmitry Klochkov; +Cc: kvm, linux-kernel, Stefan Raspl, Christian Borntraeger
Queued, thanks.
Paolo
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH RESEND] tools/kvm_stat: fix display of error when multiple processes are found
2022-06-15 12:14 ` Paolo Bonzini
@ 2022-06-15 17:30 ` Dmitry Klochkov
0 siblings, 0 replies; 3+ messages in thread
From: Dmitry Klochkov @ 2022-06-15 17:30 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: kvm, linux-kernel, Stefan Raspl, Christian Borntraeger
On Wed, Jun 15, 2022 at 08:14:27AM -0400, Paolo Bonzini wrote:
> Queued, thanks.
>
> Paolo
>
>
Thank you Paolo!
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-06-15 17:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-14 12:11 [PATCH RESEND] tools/kvm_stat: fix display of error when multiple processes are found Dmitry Klochkov
2022-06-15 12:14 ` Paolo Bonzini
2022-06-15 17:30 ` Dmitry Klochkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).