Dan Kenigsberg has submitted this change and it was merged.
Change subject: v2v: add try/except to get_external_vms
..
v2v: add try/except to get_external_vms
Single domain error causes get_external_vms to fail.
Change-Id:
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 7:
* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Shahar Havivi has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 6: Verified+1
--
To view, visit https://gerrit.ovirt.org/49507
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 6: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/49507
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 6:
looks good, please verify
--
To view, visit https://gerrit.ovirt.org/49507
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 6:
* 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: v2v: add try/except to get_external_vms
..
Patch Set 5: Code-Review+1
(1 comment)
+1 and not +2 because of a question. Looks good to me, however.
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 3:
* 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: add try/except to get_external_vms
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/49507/3/vdsm/v2v.py
File vdsm/v2v.py:
Line 154:
Shahar Havivi has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 3:
(2 comments)
https://gerrit.ovirt.org/#/c/49507/3/vdsm/v2v.py
File vdsm/v2v.py:
Line 154:
Francesco Romani has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 4: Code-Review-1
(1 comment)
thanks, almost there, but exception() is not yet used correctly.
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 5:
* 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: add try/except to get_external_vms
..
Patch Set 4: Verified+1
--
To view, visit https://gerrit.ovirt.org/49507
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 4:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Milan Zamazal has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 3: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/49507/3/vdsm/v2v.py
File vdsm/v2v.py:
Line 154:
Francesco Romani has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 3:
(2 comments)
part of Milan's comment was unanswered, and I'm reposting it :)
Besides that, looks
Milan Zamazal has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 3:
(2 comments)
https://gerrit.ovirt.org/#/c/49507/3/vdsm/v2v.py
File vdsm/v2v.py:
Line 154:
Shahar Havivi has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/49507/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 175:
Milan Zamazal has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 2:
(2 comments)
https://gerrit.ovirt.org/#/c/49507/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 175:
Shahar Havivi has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/49507/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 175:
Francesco Romani has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/49507/1/vdsm/v2v.py
File vdsm/v2v.py:
Line 172:
Milan Zamazal has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/49507/2/vdsm/v2v.py
File vdsm/v2v.py:
Line 175:
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Shahar Havivi has uploaded a new change for review.
Change subject: v2v: add try/except to get_external_vms
..
v2v: add try/except to get_external_vms
Single domain error causes get_external_vms to fail.
Change-Id:
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/49557
To
Shahar Havivi has abandoned this change.
Change subject: v2v: add try/except to get_external_vms
..
Abandoned
--
To view, visit https://gerrit.ovirt.org/49557
To unsubscribe, visit https://gerrit.ovirt.org/settings
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 2:
* 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: v2v: add try/except to get_external_vms
..
Patch Set 1: Code-Review-1
(1 comment)
I'm totally fine with the concept, not sure how this patch achieves that.
Shahar Havivi has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 1:
(1 comment)
> (1 comment)
>
> I'm totally fine with the concept, not sure how this patch
>
Shahar Havivi has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/49507
To unsubscribe, visit
Shahar Havivi has uploaded a new change for review.
Change subject: v2v: add try/except to get_external_vms
..
v2v: add try/except to get_external_vms
Single domain error causes get_external_vms to fail.
Bug-Url:
gerrit-hooks has posted comments on this change.
Change subject: v2v: add try/except to get_external_vms
..
Patch Set 1:
* #1284412::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1284412::OK, public bug
* Check
32 matches
Mail list logo