On 13.01.2016 17:39, Michal Privoznik wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1250331 > > It all works like this. The change-media command dumps domain > XML, finds the corresponding cdrom device we want to change media > in and returns it in the xmlNodePtr form. This way we don't have > to bother with keeping all the subelements or attributes that we > don't care about in the XML that is fed back to libvirt for the > update API. > > Now, the problem is we try to be clever here and detect if disk > already has a source (indicated by <source/> subelement). > However, bare fact that the element is there does not mean disk > has source. The element has some attributes and only if @file or > @dev is within them disk has source. Any other attribute is > meaningless for our purpose now. Make our clever check better. > > Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> > --- > tools/virsh-domain.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) ping Michal -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list