Re: [libvirt] [PATCH 4/4] examples: Fix event detail printing in python test
On Thu, Sep 06, 2012 at 17:50:25 -0600, Eric Blake wrote: On 09/06/2012 05:31 PM, Doug Goldstein wrote: On Thu, Sep 6, 2012 at 10:09 AM, Jiri Denemark jdene...@redhat.com wrote: If there is only one detail string for a particular event, we need to pu s/pu/put a/ Oh my and I just pushed the series without fixing this typo :-/ At least it's not the first one and certainly not the last one either. Thanks for the reviews. Jirka -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 4/4] examples: Fix event detail printing in python test
On Fri, Sep 7, 2012 at 3:26 AM, Jiri Denemark jdene...@redhat.com wrote: On Thu, Sep 06, 2012 at 17:50:25 -0600, Eric Blake wrote: On 09/06/2012 05:31 PM, Doug Goldstein wrote: On Thu, Sep 6, 2012 at 10:09 AM, Jiri Denemark jdene...@redhat.com wrote: If there is only one detail string for a particular event, we need to pu s/pu/put a/ Oh my and I just pushed the series without fixing this typo :-/ At least it's not the first one and certainly not the last one either. Thanks for the reviews. Jirka Ugh, sorry. I guess my brain has auto-correct built in because I didn't see that. Eric has spotted numerous typos that others have missed which is a good thing. -- Doug Goldstein -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH 4/4] examples: Fix event detail printing in python test
If there is only one detail string for a particular event, we need to pu comma after the string otherwise the string itself will be taken as a list and only its first character will be printed out. For example, myDomainEventCallback1 EVENT: Domain fedora17(12) Shutdown F instead of the desired myDomainEventCallback1 EVENT: Domain fedora17(12) Shutdown Finished --- examples/domain-events/events-python/event-test.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/domain-events/events-python/event-test.py b/examples/domain-events/events-python/event-test.py index fd71acd..3e314bf 100644 --- a/examples/domain-events/events-python/event-test.py +++ b/examples/domain-events/events-python/event-test.py @@ -443,12 +443,12 @@ def eventToString(event): def detailToString(event, detail): eventStrings = ( ( Added, Updated ), -( Removed ), +( Removed, ), ( Booted, Migrated, Restored, Snapshot, Wakeup ), ( Paused, Migrated, IOError, Watchdog, Restored, Snapshot ), ( Unpaused, Migrated, Snapshot ), ( Shutdown, Destroyed, Crashed, Migrated, Saved, Failed, Snapshot), -( Finished ) +( Finished, ), ( Memory, ) ) return eventStrings[event][detail] -- 1.7.12 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 4/4] examples: Fix event detail printing in python test
On Thu, Sep 6, 2012 at 10:09 AM, Jiri Denemark jdene...@redhat.com wrote: If there is only one detail string for a particular event, we need to pu comma after the string otherwise the string itself will be taken as a list and only its first character will be printed out. For example, myDomainEventCallback1 EVENT: Domain fedora17(12) Shutdown F instead of the desired myDomainEventCallback1 EVENT: Domain fedora17(12) Shutdown Finished --- examples/domain-events/events-python/event-test.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/domain-events/events-python/event-test.py b/examples/domain-events/events-python/event-test.py index fd71acd..3e314bf 100644 --- a/examples/domain-events/events-python/event-test.py +++ b/examples/domain-events/events-python/event-test.py @@ -443,12 +443,12 @@ def eventToString(event): def detailToString(event, detail): eventStrings = ( ( Added, Updated ), -( Removed ), +( Removed, ), ( Booted, Migrated, Restored, Snapshot, Wakeup ), ( Paused, Migrated, IOError, Watchdog, Restored, Snapshot ), ( Unpaused, Migrated, Snapshot ), ( Shutdown, Destroyed, Crashed, Migrated, Saved, Failed, Snapshot), -( Finished ) +( Finished, ), ( Memory, ) ) return eventStrings[event][detail] -- 1.7.12 ACK. -- Doug Goldstein -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 4/4] examples: Fix event detail printing in python test
On 09/06/2012 05:31 PM, Doug Goldstein wrote: On Thu, Sep 6, 2012 at 10:09 AM, Jiri Denemark jdene...@redhat.com wrote: If there is only one detail string for a particular event, we need to pu s/pu/put a/ comma after the string otherwise the string itself will be taken as a list and only its first character will be printed out. For example, -- Eric Blake ebl...@redhat.com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list