September 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: bookmarks first try
by vszocs＠redhat.com 19 Sep '14

19 Sep '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: bookmarks first try ...................................................................... webadmin: bookmarks first try - Bookmarks are not working the very first time you try, if you hadn't selected anything in the system tree yet. This was due to a system tree reset in bookmarkListModel_Navigated. This patch removes the reset and moves it into event handler that shows the system tree when moving from bookmarks or tags. Change-Id: I323869918c0affe15281526983e936db18ad6506 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110444 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/CommonModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/MainSectionView.java 2 files changed, 3 insertions(+), 10 deletions(-) Approvals: Alexander Wels: Verified Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32434 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I323869918c0affe15281526983e936db18ad6506 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.5]: build: ovirt-log-collector-3.5.0
by sbonazzo＠redhat.com 19 Sep '14

19 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-log-collector-3.5.0 ...................................................................... build: ovirt-log-collector-3.5.0 Change-Id: I9214e477b0584f314b627be8944106b624290c79 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33082 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9214e477b0584f314b627be8944106b624290c79 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.5]: build: ovirt-image-uploader-3.5.0
by sbonazzo＠redhat.com 19 Sep '14

19 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-image-uploader-3.5.0 ...................................................................... build: ovirt-image-uploader-3.5.0 Change-Id: I8cbd34131e4a5a6fda351f45a714b26af5026dd4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-image-uploader.spec.in 2 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33080 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8cbd34131e4a5a6fda351f45a714b26af5026dd4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.5]: build: ovirt-iso-uploader-3.5.0
by sbonazzo＠redhat.com 19 Sep '14

19 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-iso-uploader-3.5.0 ...................................................................... build: ovirt-iso-uploader-3.5.0 Change-Id: I7c90cd39545bfbd17127e85504b168d1bc80cc24 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-iso-uploader.spec.in 2 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33079 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c90cd39545bfbd17127e85504b168d1bc80cc24 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: fix late setup stages ordering
by sbonazzo＠redhat.com 19 Sep '14

19 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix late setup stages ordering ...................................................................... packaging: setup: fix late setup stages ordering Require vdsm-tool configure --force to be executed before trying to generate PKI certificates. Change-Id: Ic74b12397b89b449c03029d9e280536256ad0f72 Bug-Url: https://bugzilla.redhat.com/1139020 Bug-Url: https://bugzilla.redhat.com/1142696 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 73f2550ae821cc83f5362deaba723c3f2bc3003d) --- M src/plugins/ovirt-hosted-engine-setup/pki/vdsmpki.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33074 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic74b12397b89b449c03029d9e280536256ad0f72 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: fix late setup stages ordering
by sbonazzo＠redhat.com 19 Sep '14

19 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix late setup stages ordering ...................................................................... packaging: setup: fix late setup stages ordering Require vdsm-tool configure --force to be executed before trying to generate PKI certificates. Change-Id: Ic74b12397b89b449c03029d9e280536256ad0f72 Bug-Url: https://bugzilla.redhat.com/1139020 Bug-Url: https://bugzilla.redhat.com/1142696 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/pki/vdsmpki.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32592 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic74b12397b89b449c03029d9e280536256ad0f72 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in samples-uiplugins[master]: gluster-nagios-monitoring : Trends Actionable icons
by vszocs＠redhat.com 19 Sep '14

19 Sep '14

Vojtech Szocs has submitted this change and it was merged. Change subject: gluster-nagios-monitoring : Trends Actionable icons ...................................................................... gluster-nagios-monitoring : Trends Actionable icons With this patch, the action icons such as the print and the refresh appear to be clickable when a user hovers over it. Change-Id: I22b072f8478b58a6dedd7ea693df6f203cbd701d Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M gluster-nagios-monitoring/src/css/dashboard.css M gluster-nagios-monitoring/src/trendsTab.html 2 files changed, 26 insertions(+), 5 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, but someone else must approve Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29319 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I22b072f8478b58a6dedd7ea693df6f203cbd701d Gerrit-PatchSet: 7 Gerrit-Project: samples-uiplugins 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: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging: build: fix jboss deploy location
by piotr.kliczewski＠gmail.com 19 Sep '14

19 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: build: fix jboss deploy location ...................................................................... packaging: build: fix jboss deploy location broken by I88563c8c0 Change-Id: Id578f163f355b15743be1685038530047c39bfcc Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> (cherry picked from commit b87125e7a3fbf09ab2f32216a617296f383f7be0) --- M Makefile 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32971 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id578f163f355b15743be1685038530047c39bfcc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging: setup: add getHostname
by piotr.kliczewski＠gmail.com 19 Sep '14

19 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: setup: add getHostname ...................................................................... packaging: setup: add getHostname Move the code asking for the FQDN to a common place, to allow asking for other host names. Copy the FQDN to ENGINE_FQDN to be used by other plugins needing the engine fqdn. Change-Id: I8747002a14de6d96d5a7e185258d8ee0955b8a24 Related-To: https://bugzilla.redhat.com/1118330 Related-To: https://bugzilla.redhat.com/1080992 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 8b84424185bea6dd9f99940b9b4aa2637eb1504f) --- M packaging/setup/ovirt_engine_setup/engine/constants.py M packaging/setup/ovirt_engine_setup/engine_common/constants.py A packaging/setup/ovirt_engine_setup/hostname.py M packaging/setup/plugins/ovirt-engine-setup/base/network/hostname.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/dialog/titles.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/protocols.py 6 files changed, 390 insertions(+), 281 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/33062 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8747002a14de6d96d5a7e185258d8ee0955b8a24 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: remove unneeded call to lock vm in db on import vm
by ahadas＠redhat.com 19 Sep '14

19 Sep '14

Arik Hadas has submitted this change and it was merged. Change subject: core: remove unneeded call to lock vm in db on import vm ...................................................................... core: remove unneeded call to lock vm in db on import vm On import VM, we add the VM to the DB in ImageLocked status, so we don't need to call VmHandler.lockVm afterwards. Change-Id: I331f7d350bf7cd044a7b4ff64e9376963fd5ec9b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/32378 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I331f7d350bf7cd044a7b4ff64e9376963fd5ec9b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0