automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view,
Dan Kenigsberg has submitted this change and it was merged.
Change subject: v2v: get VM information from OVA file
..
v2v: get VM information from OVA file
In order to import a VM that exists in an OVA file the oVirt engine
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 10:
(1 comment)
https://gerrit.ovirt.org/#/c/43271/10/vdsm/v2v.py
File vdsm/v2v.py:
Line 185:
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 11: Code-Review+2
raising score
--
To view, visit https://gerrit.ovirt.org/43271
To unsubscribe, visit
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 10:
(1 comment)
https://gerrit.ovirt.org/#/c/43271/10/vdsm/v2v.py
File vdsm/v2v.py:
Line 185:
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 10: Code-Review+1
(2 comments)
partial ACK.
Since we're cleaning up schema and return value, fuether
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 9:
(2 comments)
https://gerrit.ovirt.org/#/c/43271/9/vdsm/rpc/vdsmapi-schema.json
File
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 10: Verified+1
--
To view, visit https://gerrit.ovirt.org/43271
To unsubscribe, visit
Adam Litke has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 9: Code-Review-1
(2 comments)
https://gerrit.ovirt.org/#/c/43271/9/vdsm/rpc/vdsmapi-schema.json
File
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 9: Code-Review+2
thanks for the updates, looks good to me now.
--
To view, visit
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 9: Verified+1
--
To view, visit https://gerrit.ovirt.org/43271
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 8: Code-Review+1
(2 comments)
minor doc comments inside. I see patch 44471 adds tests, hence preliminar
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 6:
(3 comments)
testing patch: https://gerrit.ovirt.org/#/c/44471/
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 6:
(4 comments)
minor comments. Looks OK so far.
I of course trust the verification, but this code begs
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 6: Code-Review-1
minor nits, -1 for visibility
--
To view, visit https://gerrit.ovirt.org/43271
To
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/43271
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 4: Code-Review-1
(10 comments)
mostly questions, but quite some of them. -1 for visibility.
Arik Hadas has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/43271/4/vdsm/v2v.py
File vdsm/v2v.py:
Line 175: root =
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 4:
(9 comments)
https://gerrit.ovirt.org/#/c/43271/4/vdsm/rpc/vdsmapi-schema.json
File
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 4: Verified+1
--
To view, visit https://gerrit.ovirt.org/43271
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Francesco Romani has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/43271/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 49:
Line 50:
Shahar Havivi has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/43271/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 49:
Line 50:
Shahar Havivi has uploaded a new change for review.
Change subject: v2v: get VM information from OVA file
..
v2v: get VM information from OVA file
In order to import a VM that exists in an OVA file the oVirt engine
needs to get
automat...@ovirt.org has posted comments on this change.
Change subject: v2v: get VM information from OVA file
..
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
34 matches
Mail list logo
