December 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: do not attempt to start websocket proxy at...
by iheim＠redhat.com 09 Dec '13

09 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: do not attempt to start websocket proxy at development mode ...................................................................... packaging: setup: do not attempt to start websocket proxy at development mode Change-Id: I40f061ec6cf658e1d8ec5c3663698ceca7569b38 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/config/websocket_proxy.py 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22183 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40f061ec6cf658e1d8ec5c3663698ceca7569b38 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Change gluster brick status to Unknown
by kmayilsa＠redhat.com 09 Dec '13

09 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Change gluster brick status to Unknown ...................................................................... engine: Change gluster brick status to Unknown When a host goes to non responsive state, change the gluster brick's status on the host to Unknown Change-Id: I80f0e581b86a34b649343a931eaccf8e41b8e275 Bug-Url: https://bugzilla.redhat.com/1021441 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterStatus.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/GlusterState.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GlusterBrickMapper.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/BrickStatusCell.java 5 files changed, 28 insertions(+), 4 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/21951 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80f0e581b86a34b649343a931eaccf8e41b8e275 Gerrit-PatchSet: 4 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: fix to remove multiple host when hosts are in maint...
by ofrenkel＠redhat.com 09 Dec '13

09 Dec '13

Omer Frenkel has submitted this change and it was merged. Change subject: gluster: fix to remove multiple host when hosts are in maintenance ...................................................................... gluster: fix to remove multiple host when hosts are in maintenance Add a lock to wait for complete gluster command execution and release Change-Id: I0f74af59dc0723eda46ac1b7c53b2a980c49fda4 Signed-off-by: Timothy Asir <tjeyasin(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1024997 Signed-off-by: Timothy Asir <tjeyasin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterJob.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJobTest.java 4 files changed, 30 insertions(+), 29 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21865 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f74af59dc0723eda46ac1b7c53b2a980c49fda4 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: frontend: fixed findbugs issue
by tjelinek＠redhat.com 09 Dec '13

09 Dec '13

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: fixed findbugs issue ...................................................................... frontend: fixed findbugs issue renamed method to lowercase Change-Id: I042faba1ae2ff6640fd221e920f2bc8251867668 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I042faba1ae2ff6640fd221e920f2bc8251867668 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: do not attempt to start websocket proxy at...
by sbonazzo＠redhat.com 09 Dec '13

09 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: do not attempt to start websocket proxy at development mode ...................................................................... packaging: setup: do not attempt to start websocket proxy at development mode Change-Id: I40f061ec6cf658e1d8ec5c3663698ceca7569b38 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/config/websocket_proxy.py 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40f061ec6cf658e1d8ec5c3663698ceca7569b38 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin : Retain bricks refresh
by kmayilsa＠redhat.com 09 Dec '13

09 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : Retain bricks refresh ...................................................................... webadmin : Retain bricks refresh On clicking retain brick button from remove-brick status pop up,after succesfull execution of the command the pop up was not getting refreshed and hence the stopped at time was not getting displayed.But,on closing the pop up and reopening it again,with the new pop up the stopped at time was getting displayed. Hence,this patch fixes it by refreshing the pop up after performing retain-brick option. Change-Id: Iddc56eec6f9e456c6fdcf6dd45152b58fb0878ed Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1037583 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeRebalanceStatusModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumeRebalanceStatusPopupView.java 3 files changed, 8 insertions(+), 13 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21966 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddc56eec6f9e456c6fdcf6dd45152b58fb0878ed Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: update firewall for all services
by sbonazzo＠redhat.com 09 Dec '13

09 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: update firewall for all services ...................................................................... packaging: setup: update firewall for all services Update the firewall for websocket_proxy and postgresql not only during setup. Always ask whether to update the firewall and do not keep the answer in the postinstall answer file, to allow users to keep their manual changes on upgrades. Output manual configuration instructions only for supported firewall managers. Bug-Url: https://bugzilla.redhat.com/1024707 Bug-Url: https://bugzilla.redhat.com/1029020 Bug-Url: https://bugzilla.redhat.com/1023316 Change-Id: If3c1a634b2e8539ebd604205b5487290c8d8a1a9 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py A packaging/setup/ovirt_engine_setup/firewall_manager_base.py A packaging/setup/plugins/ovirt-engine-common/network/__init__.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager_firewalld.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager_human.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager_iptables.py A packaging/setup/plugins/ovirt-engine-common/network/process_firewalld_services.py M packaging/setup/plugins/ovirt-engine-remove/network/__init__.py A packaging/setup/plugins/ovirt-engine-remove/network/firewall_manager.py D packaging/setup/plugins/ovirt-engine-remove/network/firewalld.py M packaging/setup/plugins/ovirt-engine-setup/config/websocket_proxy.py M packaging/setup/plugins/ovirt-engine-setup/legacy/__init__.py D packaging/setup/plugins/ovirt-engine-setup/legacy/firewall_manager.py M packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py M packaging/setup/plugins/ovirt-engine-setup/provisioning/postgres.py 16 files changed, 954 insertions(+), 393 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20737 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If3c1a634b2e8539ebd604205b5487290c8d8a1a9 Gerrit-PatchSet: 38 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: fix typo in gluster hooks event option names
by kmayilsa＠redhat.com 09 Dec '13

09 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: fix typo in gluster hooks event option names ...................................................................... gluster: fix typo in gluster hooks event option names Replace "GLuster" with "Gluster" in LocalizedEnums.properties file for gluster hook add, remove and update options. Change-Id: I5577938b32f26507e6044b1f4d273588a8e36ce9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1036035 Signed-off-by: Timothy Asir <tjeyasin(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Timothy Asir: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21950 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5577938b32f26507e6044b1f4d273588a8e36ce9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: db: fix dal test config file
by emesika＠redhat.com 09 Dec '13

09 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: db: fix dal test config file ...................................................................... db: fix dal test config file Match the conf file to the common settings. (http://www.ovirt.org/OVirt_Engine_Development_Environment#Enable_DAO_Tests) Change-Id: I5ffc320576dbce8709b7e90afa1d08445011cc75 Signed-off-by: Kobi Ianko <kobi(a)redhat.com> --- M backend/manager/modules/dal/src/test/filters/pgsql.properties 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22172 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ffc320576dbce8709b7e90afa1d08445011cc75 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: Fix typo
by iheim＠redhat.com 09 Dec '13

09 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Fix typo ...................................................................... engine: Fix typo Thiss patch fixes a minor spelling issue in MoM policy synchronization; Cluster--> Host --> "Synch Mom Policy" should be "Sync MoM Policy". Change-Id: I325c14e8b5c3fc97d6293d80a7f05d68fe89a1b9 Bug-Url: https://bugzilla.redhat.com/1039275 Signed-off-by: Doron Fediuck <dfediuck(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22171 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I325c14e8b5c3fc97d6293d80a7f05d68fe89a1b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0