- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[v2.0.z]: Add a timeout to the notification sender
by msivak＠redhat.com 23 Aug '16

23 Aug '16

Martin Sivák has submitted this change and it was merged. Change subject: Add a timeout to the notification sender ...................................................................... Add a timeout to the notification sender This configures a timeout for SMTP connection attempts to avoid broker lockup in case the SMTP server is not responding properly. Change-Id: I9e19aedcbdbfd99c16d971452ef31b7909b507a7 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1359059 Signed-off-by: Martin Sivak <msivak(a)redhat.com> (cherry picked from commit b5091cff74bdf267912d223ac5c4b5e2c51d6ac7) --- M ovirt_hosted_engine_ha/broker/broker.conf M ovirt_hosted_engine_ha/broker/notifications.py 2 files changed, 9 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62717 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e19aedcbdbfd99c16d971452ef31b7909b507a7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.0.z Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix validation message replacement when attaching boot...
by tnisan＠redhat.com 23 Aug '16

23 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix validation message replacement when attaching boot disk to VM ...................................................................... core: Fix validation message replacement when attaching boot disk to VM Change-Id: I4cfb8586a5b5cb0997fdbc7f93d8367a325902e8 Bug-Url: https://bugzilla.redhat.com/1368954 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/DiskValidator.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62716 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4cfb8586a5b5cb0997fdbc7f93d8367a325902e8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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-hosted-engine-ha[master]: Add a timeout to the notification sender
by msivak＠redhat.com 23 Aug '16

23 Aug '16

Martin Sivák has submitted this change and it was merged. Change subject: Add a timeout to the notification sender ...................................................................... Add a timeout to the notification sender This configures a timeout for SMTP connection attempts to avoid broker lockup in case the SMTP server is not responding properly. Change-Id: I9e19aedcbdbfd99c16d971452ef31b7909b507a7 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1359059 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M ovirt_hosted_engine_ha/broker/broker.conf M ovirt_hosted_engine_ha/broker/notifications.py 2 files changed, 9 insertions(+), 3 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Andrej Krejcir: Verified -- To view, visit https://gerrit.ovirt.org/61948 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e19aedcbdbfd99c16d971452ef31b7909b507a7 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: post ovirt-hosted-engine-setup-1.3.7.3
by sbonazzo＠redhat.com 23 Aug '16

23 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-1.3.7.3 ...................................................................... build: post ovirt-hosted-engine-setup-1.3.7.3 Change-Id: Ie372412d18996ee908e5a45a26a0ae4634a0679c Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 11 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62712 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie372412d18996ee908e5a45a26a0ae4634a0679c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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-sdk[sdk_4.0]: Fault is provided in Action reponse
by omachace＠redhat.com 23 Aug '16

23 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Fault is provided in Action reponse ...................................................................... Fault is provided in Action reponse Action reponse provides fault of the action, in case the fault occures. This patch changes the flow, so we chechk the fault inside the action. Change-Id: Iba8fc809365d26473d84086a332aa4b169041459 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1367020 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit b1f69b0a2500a9b6a6f7783d0a18eb48766e0335) --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java M sdk/lib/ovirtsdk4/service.py 2 files changed, 20 insertions(+), 21 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62714 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba8fc809365d26473d84086a332aa4b169041459 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: ovirt-hosted-engine-setup-1.3.7.3
by sbonazzo＠redhat.com 23 Aug '16

23 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-1.3.7.3 ...................................................................... build: ovirt-hosted-engine-setup-1.3.7.3 Change-Id: I40ef2e6428ccac01b80acba57ce03f28e8ba160c Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 12 insertions(+), 12 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62707 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40ef2e6428ccac01b80acba57ce03f28e8ba160c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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-sdk[sdk_4.0]: Add generic reader
by omachace＠redhat.com 23 Aug '16

23 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add generic reader ...................................................................... Add generic reader Currently the SDK doesn't provide any mechanism to read an XML object without explicitly specifying its type. For example, to read a virtual machine the only method available is "VmReader.read_one". But there are situations where the SDK can't know in advance what type of object is going to be received from the server, for example "action" or "fault". To support that, this patch adds a new "Reader.read" method that decides the type of object based on the name of the first tag, and invokes the corresponding specific reader to read it. Change-Id: Ibe55914453ea6c6c7e46406bdcbad7acaa813721 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 4f8e4c5ba72b7b0777b372cc12d8cb1e4691a52f) --- M generator/src/main/java/org/ovirt/sdk/python/ReadersGenerator.java M sdk/lib/ovirtsdk4/reader.py M sdk/tests/test_reader.py 3 files changed, 215 insertions(+), 30 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62715 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe55914453ea6c6c7e46406bdcbad7acaa813721 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Fault is provided in Action reponse
by omachace＠redhat.com 23 Aug '16

23 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Fault is provided in Action reponse ...................................................................... Fault is provided in Action reponse Action reponse provides fault of the action, in case the fault occures. This patch changes the flow, so we chechk the fault inside the action. Change-Id: Iba8fc809365d26473d84086a332aa4b169041459 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1367020 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java M sdk/lib/ovirtsdk4/service.py 2 files changed, 20 insertions(+), 21 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61976 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba8fc809365d26473d84086a332aa4b169041459 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Add generic reader
by omachace＠redhat.com 23 Aug '16

23 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add generic reader ...................................................................... Add generic reader Currently the SDK doesn't provide any mechanism to read an XML object without explicitly specifying its type. For example, to read a virtual machine the only method available is "VmReader.read_one". But there are situations where the SDK can't know in advance what type of object is going to be received from the server, for example "action" or "fault". To support that, this patch adds a new "Reader.read" method that decides the type of object based on the name of the first tag, and invokes the corresponding specific reader to read it. Change-Id: Ibe55914453ea6c6c7e46406bdcbad7acaa813721 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ReadersGenerator.java M sdk/lib/ovirtsdk4/reader.py M sdk/tests/test_reader.py 3 files changed, 215 insertions(+), 30 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62609 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe55914453ea6c6c7e46406bdcbad7acaa813721 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: Adding build-artifacts-manual.sh script
by sbonazzo＠redhat.com 23 Aug '16

23 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Adding build-artifacts-manual.sh script ...................................................................... Adding build-artifacts-manual.sh script Adding build-artifacts-manual.sh script and related packages and repos files. build-artifacts-manual.sh is used for creating rpms from an existing tarball. It assumes that the tarball already exists in the projects directory and builds srpms and rpms, and moves the files to exported_artifacts directory. Change-Id: I761bbb5d5414c1b3d5d5e82637d34d5737e6cbb9 Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A automation/build-artifacts-manual.packages A automation/build-artifacts-manual.repos A automation/build-artifacts-manual.sh 3 files changed, 25 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Sharon Naftaly: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62458 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I761bbb5d5414c1b3d5d5e82637d34d5737e6cbb9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0