Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-02-02 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 4: (4 comments) https://gerrit.ovirt.org/#/c/52875/4/tests/functional/supervdsmFuncTests.py File tests/f

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-02-01 Thread edwardh
Edward Haas has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/52875 To unsubscribe, visit https://gerrit.ovir

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-02-01 Thread edwardh
Edward Haas has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/52875/3/vdsm/supervdsmServer File vdsm/supervdsmServer: Line 1

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-02-01 Thread edwardh
Edward Haas has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: (3 comments) https://gerrit.ovirt.org/#/c/52875/3/vdsm/supervdsmServer File vdsm/supervdsmServer: Line

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-02-01 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: (3 comments) https://gerrit.ovirt.org/#/c/52875/3/vdsm/supervdsmServer File vdsm/supervdsmServer: Li

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-02-01 Thread automation
gerrit-hooks has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-31 Thread edwardh
Edward Haas has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: (4 comments) I like++ https://gerrit.ovirt.org/#/c/52875/3/vdsm/supervdsmServer File vdsm/supervdsmServ

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-31 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: Yaniv, I think this change is not needed. -- To view, visit https://gerrit.ovirt.org/52875 To unsubscrib

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: please don't leave your code-review -1 if you don't have any code comments that deserve -1... lets di

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 3: * #1182092::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1182092::OK, public bug * Check

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 2: its RFE for 4.0 - https://bugzilla.redhat.com/show_bug.cgi?id=1182092 -- To view, visit https://gerr

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 2: Code-Review-1 Why do we need this? -- To view, visit https://gerrit.ovirt.org/52875 To unsubscribe, visi

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 2: there are still some gaps, and I want to move the rest of the functions to separate modules.. feel fr

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread ybronhei
Yaniv Bronhaim has uploaded a new change for review. Change subject: [wip] Dynamic load of supervdsm_api funcs .. [wip] Dynamic load of supervdsm_api funcs The patch adds supervdsm_api folder (which will move later once all inte

Change in vdsm[master]: [wip] Dynamic load of supervdsm_api funcs

2016-01-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: [wip] Dynamic load of supervdsm_api funcs .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat