Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-19 Thread fromani
Francesco Romani has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: (1 comment) Tomas: yes, your explanation about overloading this API and not adding a new one is

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-19 Thread mpolednik
Martin Polednik has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/57418/3//COMMIT_MSG Commit Message: PS3, Line 7: Lazy > What

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-17 Thread Tomas Golembiovsky
Tomas Golembiovsky has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: (3 comments) https://gerrit.ovirt.org/#/c/57418/3//COMMIT_MSG Commit Message: PS3, Line 7: Lazy >

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread mpolednik
Martin Polednik has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/57418/3//COMMIT_MSG Commit Message: PS3, Line 7: Lazy

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread mpolednik
Martin Polednik has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: (2 comments) https://gerrit.ovirt.org/#/c/57418/3/lib/api/vdsm-api.yml File lib/api/vdsm-api.yml:

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/57418 To unsubscribe, visit

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/57418 To unsubscribe, visit

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 3: * #1294629::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1294629::OK, public bug * Check

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 2: (1 comment) Just one PEP8 issue, otherwise fine for me now.

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 2: * #1294629::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1294629::OK, public bug * Check

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread Tomas Golembiovsky
Tomas Golembiovsky has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 1: (3 comments) Francesco: You mean why not add another API call? The name 'getExternalVMs' seems

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread fromani
Francesco Romani has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 1: (1 comment) initial review. Have you explored the possibility of adding a new verb instead of adding

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-16 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 1: (3 comments) https://gerrit.ovirt.org/#/c/57418/1/lib/vdsm/utils.py File lib/vdsm/utils.py: Line 413:

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-15 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 1: (2 comments) https://gerrit.ovirt.org/#/c/57418/1/lib/api/vdsm-api.yml File lib/api/vdsm-api.yml: Line

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-13 Thread Tomas Golembiovsky
Tomas Golembiovsky has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 1: Verified+1 (1 comment) https://gerrit.ovirt.org/#/c/57418/1/lib/api/vdsm-api.yml File

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-13 Thread Tomas Golembiovsky
Tomas Golembiovsky has uploaded a new change for review. Change subject: v2v: Lazy loading of external VMs info .. v2v: Lazy loading of external VMs info API changes allowing implementation of lazy loading of information about

Change in vdsm[master]: v2v: Lazy loading of external VMs info

2016-05-13 Thread automation
gerrit-hooks has posted comments on this change. Change subject: v2v: Lazy loading of external VMs info .. Patch Set 1: * #1294629::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1294629::OK, public bug * Check