Dan Kenigsberg has submitted this change and it was merged.
Change subject: v2v: Import VM from OVA file
..
v2v: Import VM from OVA file
OVA is a tar file which contain a VM with its Ovf file and its disks as
well.
Change-Id:
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Shahar Havivi has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 9: Verified+1
--
To view, visit https://gerrit.ovirt.org/43367
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Shahar Havivi has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 6:
(2 comments)
https://gerrit.ovirt.org/#/c/43367/6/vdsm/v2v.py
File vdsm/v2v.py:
Line 164:
Line 165:
Line 166:
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Francesco Romani has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 8: Code-Review+2
thanks, looks better now.
I believe that _run_command is not the best name, but not good enough
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Francesco Romani has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 6: Code-Review-1
(2 comments)
On overall, looks good. But a couple of questions inside. -1 for visibility
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Shahar Havivi has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/43367
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 4:
(4 comments)
big question(s) inside, -1 for visibility.
Francesco Romani has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 4: Code-Review-1
furthermore: can you please point me where you addressed comments from Arik in
V2?
--
To view,
Shahar Havivi has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 4:
(4 comments)
https://gerrit.ovirt.org/#/c/43367/4/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Shahar Havivi has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/43367
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Shahar Havivi has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 4: Verified+1
--
To view, visit https://gerrit.ovirt.org/43367
To unsubscribe, visit
Arik Hadas has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 2: Verified-1
(1 comment)
https://gerrit.ovirt.org/#/c/43367/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 349: def
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: Import VM from OVA file
..
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Shahar Havivi has uploaded a new change for review.
Change subject: v2v: Import VM from OVA file
..
v2v: Import VM from OVA file
OVA is a tar file which contain a VM with its Ovf file and its disks as
well.
Change-Id:
25 matches
Mail list logo