[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-28 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fix notifier parameters .. core: Fix notifier parameters The default parameters of the notifier service are not adjusted when the engine is confi

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-28 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 6: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7187 To unsubscribe, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-28 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 6: Verified Rebased and verified building and installing new packages. -- To view, visit http://gerrit.ovirt.org/718

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-28 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7187 To unsubscribe, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-23 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 4: Looks good to me, approved Following Itamar's comment - approving this patch. -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-23 Thread iheim
Itamar Heim has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 4: no point on working on something there isn't a clear demand for. notification can be assumed to be on same server -

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-23 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 4: Good question Yair. In fact the notifier can't be installed without the engine, and (to some degree) the engine c

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-23 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 4: Juan, this looks good. General question - I understood why you took off all server URL (i.e - localhost:80) - do

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-21 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 4: Build Failed http://jenkins.ovirt.info/job/patch_engine_gwt_admin/527/ : FAILURE http://jenkins.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-21 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 4: Rebased. -- To view, visit http://gerrit.ovirt.org/7187 To unsubscribe, visit http://gerrit.ovirt.org/settings G

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 3: Build Successful http://jenkins.ovirt.info/job/patch_engine_gwt_admin/434/ : SUCCESS http://jenkins.ov

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-20 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Fix notifier parameters .. Patch Set 3: Verified Rebased and verified with new build and installation of a new set of RPMs. -- To view, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: Fix notifier parameters

2012-08-14 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: core: Fix notifier parameters .. core: Fix notifier parameters The default parameters of the notifier service are not adjusted when the engine is configured. T