- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: moving environment plugin to common
by oschreib＠redhat.com 22 May '13

22 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: moving environment plugin to common ...................................................................... packaging: setup: moving environment plugin to common moving environment plugin to common since it will be needed also during cleanup. Change-Id: Ia48762913833335d22e823dd68d87901b014b839 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/system/__init__.py R packaging/setup/plugins/ovirt-engine-common/system/environment.py M packaging/setup/plugins/ovirt-engine-setup/system/__init__.py 3 files changed, 2 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14937 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia48762913833335d22e823dd68d87901b014b839 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: allow interactive NFS exports cleanup
by oschreib＠redhat.com 22 May '13

22 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: allow interactive NFS exports cleanup ...................................................................... packaging: setup: allow interactive NFS exports cleanup allow interactive NFS exports cleanup with engine-cleanup-2 - Refactored uninstall for allowing file grouping - Added group for exportfs files - Cleanup optional uninstall groups like exportfs only if requested. Change-Id: I16b5920f0cfc9f564aa92f7e0d5a6d33af5e0b44 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-remove/files/simple.py M packaging/setup/plugins/ovirt-engine-setup/core/uninstall.py M packaging/setup/plugins/ovirt-engine-setup/system/exportfs.py 4 files changed, 201 insertions(+), 24 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14702 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I16b5920f0cfc9f564aa92f7e0d5a6d33af5e0b44 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine:VDS command to update hook content
by sanjal＠redhat.com 22 May '13

22 May '13

Shireesh Anjal has submitted this change and it was merged. Change subject: engine:VDS command to update hook content ...................................................................... engine:VDS command to update hook content VDS command to update content of gluster hook on a server from the engine. Gluster hook is identified by gluster command, stage and name. Change-Id: I4dc8e12c7d7760114c392db19a56c9cefd4b5763 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/gluster/GlusterHookVDSParameters.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/AbstractGlusterBrokerCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/UpdateGlusterHookVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/VdsmErrors.java 10 files changed, 81 insertions(+), 2 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14638 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4dc8e12c7d7760114c392db19a56c9cefd4b5763 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Add rename logging for providers
by mkolesni＠redhat.com 22 May '13

22 May '13

Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Add rename logging for providers ...................................................................... engine: Add rename logging for providers Added ability to log to the audit log the renaming of a provider. Change-Id: I3a9206e8da761a88832507a3342d8526514f292f Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/UpdateProviderCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcObjectType.java 2 files changed, 25 insertions(+), 2 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14883 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a9206e8da761a88832507a3342d8526514f292f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: fixed memory requirements validation
by oschreib＠redhat.com 22 May '13

22 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: fixed memory requirements validation ...................................................................... packaging: setup: fixed memory requirements validation aligned memory requirements validation with legacy setup Change-Id: I46e125e418ec0f2951850825dbeca9a5f9eadfbf Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-setup/system/memcheck.py 2 files changed, 69 insertions(+), 40 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14916 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I46e125e418ec0f2951850825dbeca9a5f9eadfbf Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: cleanup RedirectServlet (6/N)
by lhornyak＠redhat.com 22 May '13

22 May '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cleanup RedirectServlet (6/N) ...................................................................... engine: cleanup RedirectServlet (6/N) - removed misleading log message (may be copy-paste from health status servlet) Change-Id: I9006cd0337b490199c0518837f21ef7caeb55fbf Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/redirect/RedirectServlet.java 1 file changed, 2 insertions(+), 5 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14555 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9006cd0337b490199c0518837f21ef7caeb55fbf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: cleanup RedirectServlet (5/N)
by lhornyak＠redhat.com 22 May '13

22 May '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cleanup RedirectServlet (5/N) ...................................................................... engine: cleanup RedirectServlet (5/N) - removed declaration of variables before they are used Change-Id: I2a9193e02ac219f1a281e017aa2b69d57211b53e Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/redirect/RedirectServlet.java 1 file changed, 4 insertions(+), 6 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14554 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a9193e02ac219f1a281e017aa2b69d57211b53e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: cleanup RedirectServlet (3/N)
by lhornyak＠redhat.com 22 May '13

22 May '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cleanup RedirectServlet (3/N) ...................................................................... engine: cleanup RedirectServlet (3/N) - use commons-lang StringUtil to check returned value Change-Id: Ib51ed8abbbbca374a75ad94c79ef4aad31fac9f7 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/redirect/RedirectServlet.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14552 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib51ed8abbbbca374a75ad94c79ef4aad31fac9f7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: cleanup RedirectServlet (2/N)
by lhornyak＠redhat.com 22 May '13

22 May '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cleanup RedirectServlet (2/N) ...................................................................... engine: cleanup RedirectServlet (2/N) - typo fixed Change-Id: Ife2aea13fcd0f1d2c88caf7b5f68d3439e155535 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/redirect/RedirectServlet.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14551 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife2aea13fcd0f1d2c88caf7b5f68d3439e155535 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: cleanup RedirectServlet (1/N)
by lhornyak＠redhat.com 22 May '13

22 May '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cleanup RedirectServlet (1/N) ...................................................................... engine: cleanup RedirectServlet (1/N) - catch identation fix Change-Id: I4f82a0adb6980fa06a0480cb132db0f2e4a34ba2 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/redirect/RedirectServlet.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14550 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f82a0adb6980fa06a0480cb132db0f2e4a34ba2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0