January 2017 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: engine: raise the default timeout on engine setup to 1800s
by Code Review 11 Jan '17

11 Jan '17

>From Simone Tiraboschi <stirabos(a)redhat.com>: Simone Tiraboschi has submitted this change and it was merged. Change subject: engine: raise the default timeout on engine setup to 1800s ...................................................................... engine: raise the default timeout on engine setup to 1800s Raise the default timeout on engine setup to 1800 seconds. Change-Id: I8d9b56975d35111ca91f09b89c2283203dd3a4b0 Bug-Url: https://bugzilla.redhat.com/1352384 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/69973 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d9b56975d35111ca91f09b89c2283203dd3a4b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0.6.z]: build: ovirt-release40-4.0.6.1
by Code Review 11 Jan '17

11 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release40-4.0.6.1 ...................................................................... build: ovirt-release40-4.0.6.1 Change-Id: I1939af9fb574d264c4944df21bcbd2031a48d048 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-release40.spec.in 2 files changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/69977 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1939af9fb574d264c4944df21bcbd2031a48d048 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0.6.z Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0.6.z]: node: add sysstat
by Code Review 11 Jan '17

11 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: node: add sysstat ...................................................................... node: add sysstat Include sysstat so `sar` is available, and upstream stays in sync with downstream. Change-Id: Ie5885c086169af275ca7d7ee483def448f3b31c6 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1399124 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/69976 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie5885c086169af275ca7d7ee483def448f3b31c6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0.6.z Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0.6.z]: packaging: gpg key update
by Code Review 11 Jan '17

11 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: gpg key update ...................................................................... packaging: gpg key update The shipped key expired in April 2016 $ gpg --import RPM-GPG-ovirt $ gpg --list-keys ------------------------------------ pub 2048R/FE590CB7 2014-03-30 [expired: 2016-04-02] uid oVirt <infra(a)ovirt.org> the key validity has been extended already. updating the gpg key: $ gpg --recv-keys FE590CB7 $ gpg --list-keys ------------------------------------ pub 2048R/FE590CB7 2014-03-30 [expires: 2021-04-03] uid oVirt <infra(a)ovirt.org> sub 2048R/004BC303 2014-03-30 [expires: 2021-04-03] $ gpg --armor --export FE590CB7 >/tmp/RPM-GPG-ovirt-v2 Change-Id: I4ad10781743ea6baff2fb9193b9922b6e339c803 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 16eac17d44d02b9f634833c1adba196cc46b3c80) (cherry picked from commit 9f25e1b7a840200ba357b41a6c0a5ee9051c96a2) --- D RPM-GPG-ovirt A RPM-GPG-ovirt-v2 M ovirt-release40.spec.in 3 files changed, 43 insertions(+), 31 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/69975 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ad10781743ea6baff2fb9193b9922b6e339c803 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0.6.z Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[v2.1.z]: storage: add an helper method to disconnect the storage server
by Code Review 11 Jan '17

11 Jan '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: storage: add an helper method to disconnect the storage server ...................................................................... storage: add an helper method to disconnect the storage server Add an helper method to HAClient to disconnect the hosted-engine storage server. Change-Id: Ib595116ed922ab081486bcea2c9052e823c50f2c Related-To: https://gerrit.ovirt.org/69949/ Bug-Url: https://bugzilla.redhat.com/1001181 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> (cherry picked from commit 2020312815a2f83700b9f36e242330eb075d0e93) --- M ovirt_hosted_engine_ha/client/client.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/69931 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib595116ed922ab081486bcea2c9052e823c50f2c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.1.z Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: core: do not copy lease_sd_id from vm to template
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core: do not copy lease_sd_id from vm to template ...................................................................... core: do not copy lease_sd_id from vm to template When creating a template, do not copy the storage domain that the lease of the VM that the template is based on resides in to the template configuration. Change-Id: I59f86759c92e75887f6de2fc5ce476fc23f95b75 Bug-Url: https://bugzilla.redhat.com/1317429 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/69823 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59f86759c92e75887f6de2fc5ce476fc23f95b75 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: CreateAllSnapshotsFromVmCommandTest redundant mocking
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core: CreateAllSnapshotsFromVmCommandTest redundant mocking ...................................................................... core: CreateAllSnapshotsFromVmCommandTest redundant mocking Removed redundant mock objects. Change-Id: I69af4c351011d76b0d35da94b38123569deca66c Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/snapshots/CreateAllSnapshotsFromVmCommandTest.java 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/69953 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69af4c351011d76b0d35da94b38123569deca66c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Queries to get storagedomain dr and georepsession
by Code Review 11 Jan '17

11 Jan '17

>From Sahina Bose <sabose(a)redhat.com>: Sahina Bose has submitted this change and it was merged. Change subject: engine: Queries to get storagedomain dr and georepsession ...................................................................... engine: Queries to get storagedomain dr and georepsession Change-Id: Ie61c465b1fdff5288e268cb89f3189d5fb843e2f Bug-Url: https://bugzilla.redhat.com/1398593 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetGlusterVolumeGeoRepSessionByIdQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/GetStorageDomainDRQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 3 files changed, 42 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Verified; Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/69942 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie61c465b1fdff5288e268cb89f3189d5fb843e2f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: webadmin: Suggest the option to add an unplugged vnic (error...
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Suggest the option to add an unplugged vnic (error message) ...................................................................... webadmin: Suggest the option to add an unplugged vnic (error message) added the missing possibility to add an unplugged vnic when adding a vnic profile to a running vm with the network of the vnic missing on the host. Bug-Id: https://bugzilla.redhat.com/1115059 Signed-off-by: Leon Goldberg <lgoldber(a)redhat.com> Change-Id: I702cb8e42cbc53bde8114bd55482dc4e7e5d0d6b --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Leon Goldberg: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69901 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I702cb8e42cbc53bde8114bd55482dc4e7e5d0d6b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Leon Goldberg <lgoldber(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: core: acquireLock doesn't check for null lock list
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core: acquireLock doesn't check for null lock list ...................................................................... core: acquireLock doesn't check for null lock list Change-Id: Ic0d2fbd3742cc25dc35b6be5ce394290cb25a16e Bug-Url: https://bugzilla.redhat.com/1408599 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachUserToVmFromPoolAndRunCommand.java 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Shahar Havivi: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69962 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0d2fbd3742cc25dc35b6be5ce394290cb25a16e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0