Re: [libvirt] [PATCH] blockjob: use stable disk string in job event
On Sat, Jun 14, 2014 at 07:49:40AM -0600, Eric Blake wrote:
If you think backward compatibility of old clients that expected
and operated on an absolute name is too important to break, I'm
open to suggestions on alternatives how to preserve that. We don't
have a flag argument during event registration, or I would have
used it. Otherwise, I hope this change is okay as-is.
Hmm, I fear this is one of those cases where it seems fine
to change the semantics, but is liable to bite us later.
The most obvious alternative is to just introduce a new event
BLOCK_JOB_EVENT_2 or something less sucky. That still leaves
the question of how to generate the old event in the fact of
a disk with no local name - we could use empty string '' as
the "path" if the disk lacks one. Or we could simply not emit
the old style event. I'd probably go for empty string though.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|