gerrit-hooks has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 11:
* 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: migration added migrateChangeParams verb
..
migration added migrateChangeParams verb
Currently it contains only the bandwidth but it will be used also
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 10: Verified+1
--
To view, visit https://gerrit.ovirt.org/46934
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 10: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/46934
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 9: Code-Review+2
still looks good -> +2
--
To view, visit https://gerrit.ovirt.org/46934
To
gerrit-hooks has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 9:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Francesco Romani has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 8: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/46934
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 8:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/7/vdsm/rpc/vdsmapi-schema.json
File
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/7/vdsm/virt/migration.py
File vdsm/virt/migration.py:
Francesco Romani has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
(2 comments)
another note in schema, otherwise seems ok
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/5/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1557:
Francesco Romani has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7: Code-Review-1
(1 comment)
almost ok, one minor comment inside
Francesco Romani has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/7/vdsm/virt/migration.py
File
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
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: migration added migrateChangeParams verb
..
Patch Set 5: Code-Review+1
(1 comment)
preliminary review, minor comment inside
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1546:
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 4:
* 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: migration added migrateChangeParams verb
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/46934
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1546:
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 2:
(2 comments)
https://gerrit.ovirt.org/#/c/46934/2/vdsm/rpc/vdsmapi-schema.json
File
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
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: migration added migrateChangeParams verb
..
Patch Set 2: Code-Review-1
(6 comments)
few comments inside, but prtty much all of them are minor. -1 for
automat...@ovirt.org has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Tomas Jelinek has uploaded a new change for review.
Change subject: migration added migrateChangeParams verb
..
migration added migrateChangeParams verb
Currently it contains only the bandwidth but it will be used also for
28 matches
Mail list logo