Keith Robertson has posted comments on this change.

Change subject: tools: New tool to upload OVF archives (#xxxxxx)
......................................................................


Patch Set 9: (2 inline comments)

In response to Barak's comment's...
"Need to test it also with: 1 - VM with multiple disks and multiple snapshots 
per disk 2 - #1 but each disk with a different number of snapshots"

I can confirm that I tested a VM with 2 disks and 1 snap per-disk, and that the 
VM was successfully imported.

In response to Doron's comments, my mistake.  I do; however, have a BZ that I 
can attach to this patch.  I'll update the commit message and push again.

....................................................
Commit Message
Line 7: tools: New tool to upload OVF archives (#xxxxxx)
I think I just missed the comment.  I do have a BZ that I can attach to the 
patch now.  
https://bugzilla.redhat.com/show_bug.cgi?id=782182 

I will fix this in the next patch.

Line 9: https://bugzilla.redhat.com/XXXXXX
I think I just missed the comment. I do have a BZ that I can attach to the 
patch now. https://bugzilla.redhat.com/show_bug.cgi?id=782182

I will fix this in the next patch.

--
To view, visit http://gerrit.ovirt.org/574
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3679b44720919aecef457536cc9e19370531c06c
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Keith Robertson <[email protected]>
Gerrit-Reviewer: Andrew Cathrow <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Geert Jansen <[email protected]>
Gerrit-Reviewer: Keith Robertson <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Ronen Angluster <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to