- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Add ability to check if a VDS is hosting the HE VM
by rgolan＠redhat.com 09 May '16

09 May '16

Roy Golan has submitted this change and it was merged. Change subject: engine: Add ability to check if a VDS is hosting the HE VM ...................................................................... engine: Add ability to check if a VDS is hosting the HE VM Add a property to the VDS entity that tracks whether a given host is hosting the hosted engine VM. The property is calculated and populated in the VDS view and is not persisted. This approach provides the simplest way to get this information into the VDS entity since the Java 1.7 limitation on classes in the common package prevents using convenience methods available elsewhere in the code base. The change to the VDS view created no significant performance issues when tested locally and at scale. Change-Id: I7e2609f59e588c6ab9279b74312b68dc5af06d5a Bug-Url: https://bugzilla.redhat.com/1308861 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDaoTest.java M packaging/dbscripts/create_views.sql 4 files changed, 127 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/55669 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e2609f59e588c6ab9279b74312b68dc5af06d5a Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: fix typo in an error message string
by tjelinek＠redhat.com 09 May '16

09 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: fix typo in an error message string ...................................................................... webadmin: fix typo in an error message string Fix typo in an error message for message constant: windowsHostNameInfo Bug-Url: https://bugzilla.redhat.com/1331939 Change-Id: Ie656c0e4eaec47d1cdeac9dc6c04a056f0df51c1 Signed-off-by: Sharon Gratch <sgratch(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Sharon Gratch: Verified -- To view, visit https://gerrit.ovirt.org/57191 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie656c0e4eaec47d1cdeac9dc6c04a056f0df51c1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[master]: packaging: setup: Move DBTransaction to common
by Shirly Radco 09 May '16

09 May '16

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: Move DBTransaction to common ...................................................................... packaging: setup: Move DBTransaction to common so that it runs also in cleanup. Bug-Url: https://bugzilla.redhat.com/1328860 Change-Id: I9c620d4c5e5842249fa193e02c8fb76e5521ff78 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/engine_connection.py 2 files changed, 40 insertions(+), 41 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/56458 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c620d4c5e5842249fa193e02c8fb76e5521ff78 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ioprocess: add fc23 build
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ioprocess: add fc23 build ...................................................................... ioprocess: add fc23 build Change-Id: If37c46a96b7402ededac0b2c58c7207a37f5fb84 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ioprocess/ioprocess_create-rpms.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57232 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If37c46a96b7402ededac0b2c58c7207a37f5fb84 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: publish-rpms-nightly-3.6: add cockpit-ovirt
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: publish-rpms-nightly-3.6: add cockpit-ovirt ...................................................................... publish-rpms-nightly-3.6: add cockpit-ovirt cockpit-ovirt is needed by NGN, in tech preview for 3.6 Change-Id: I02b8891af6561a5e35d446619e3032e7e85fb66d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me Sandro Bonazzola: Verified; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57227 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02b8891af6561a5e35d446619e3032e7e85fb66d Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Fix typos in vdsm and vdsm-jsonrpc-java projects
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Fix typos in vdsm and vdsm-jsonrpc-java projects ...................................................................... Fix typos in vdsm and vdsm-jsonrpc-java projects stardard -> standard Change-Id: I57ec82408f7e7c53c785d3b2c8044c9953dc2b3d Signed-off-by: Nir Soffer <nsoffer(a)redhat.com> --- M jobs/confs/projects/vdsm-jsonrpc-java/vdsm-jsonrpc-java_standard.yaml M jobs/confs/projects/vdsm/vdsm_standard.yaml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Nir Soffer: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57220 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I57ec82408f7e7c53c785d3b2c8044c9953dc2b3d Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ioprocess: switch 3.6 to 0.15.1 branch
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ioprocess: switch 3.6 to 0.15.1 branch ...................................................................... ioprocess: switch 3.6 to 0.15.1 branch Change-Id: I1e82f9029019fad79f61db6087f75b25e67e605d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ioprocess/ioprocess_create-rpms.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml 2 files changed, 4 insertions(+), 12 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Gil Shinar: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57158 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e82f9029019fad79f61db6087f75b25e67e605d Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: setup: getDBConfig: Allow overriding host
by sbonazzo＠redhat.com 09 May '16

09 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: getDBConfig: Allow overriding host ...................................................................... packaging: setup: getDBConfig: Allow overriding host So that if db is at localhost, when generating conf for a remote machine, can put instead our fqdn. Change-Id: I83ab0f3140629b3f842842f15d8ccdf7910e6d3e Bug-Url: https://bugzilla.redhat.com/1318665 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine_common/database.py 1 file changed, 15 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/56462 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I83ab0f3140629b3f842842f15d8ccdf7910e6d3e Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: PermissionList - correct VdcObjectType for LunDisk
by tnisan＠redhat.com 08 May '16

08 May '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: PermissionList - correct VdcObjectType for LunDisk ...................................................................... webadmin: PermissionList - correct VdcObjectType for LunDisk PermissionListModel: Returning an appropriate VdcObjectType for LunDisk/CinderDisk. Needed for proper display/creation of permissions. Change-Id: I1f66ef6a98947420543686bd566ea3c26e06f1e6 Bug-Url: https://bugzilla.redhat.com/1325785 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/PermissionListModel.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Daniel Erez: Verified -- To view, visit https://gerrit.ovirt.org/57196 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f66ef6a98947420543686bd566ea3c26e06f1e6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix findbugs DLS issue in BondPanel class
by tnisan＠redhat.com 08 May '16

08 May '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix findbugs DLS issue in BondPanel class ...................................................................... core: Fix findbugs DLS issue in BondPanel class Commit eb86f8a219e5e99f71462d1478c1db5a6680f7db introduced a dead local store in BondPanel, this patch removes this DLS Change-Id: Iaed1beedaa9ca77a5304fadbb10229aa3404529e Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/host/BondPanel.java 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57209 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaed1beedaa9ca77a5304fadbb10229aa3404529e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0