June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[master]: Add scripts for the jenkins automation framework
by fabiand＠redhat.com 22 Jun '15

22 Jun '15

Fabian Deutsch has submitted this change and it was merged. Change subject: Add scripts for the jenkins automation framework ...................................................................... Add scripts for the jenkins automation framework Add scripts to automatically build and export RPMs. Check patches when submitted and when merged Change-Id: I97d67f7881fff50ef3100a01151bff7d55d98dc4 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/42508 Continuous-Integration: Jenkins CI Tested-by: Anatoly Litovsky <tlitovsk(a)redhat.com> Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> --- A automation/build-artifacts.req A automation/build-artifacts.sh A automation/check-merged.req A automation/check-merged.sh A automation/check-patch.req A automation/check-patch.sh 6 files changed, 49 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me, approved Jenkins CI: Passed CI tests Anatoly Litovsky: Verified -- To view, visit https://gerrit.ovirt.org/42508 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97d67f7881fff50ef3100a01151bff7d55d98dc4 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: Add a better check-local to prep for yaml-izing
by fabiand＠redhat.com 22 Jun '15

22 Jun '15

Fabian Deutsch has submitted this change and it was merged. Change subject: Add a better check-local to prep for yaml-izing ...................................................................... Add a better check-local to prep for yaml-izing Make the linters work nicely, so we can use the standard automation framework for upstream jenkins jobs. Change-Id: I7bb2283dd4c4de6acd2eece6884fffb456c0b680 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/42505 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Continuous-Integration: Fabian Deutsch <fabiand(a)redhat.com> Tested-by: Fabian Deutsch <fabiand(a)redhat.com> --- M Makefile.am M src/Makefile.am A src/Makefile.check M src/engine_page.py 4 files changed, 57 insertions(+), 55 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved; Passed CI tests Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/42505 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7bb2283dd4c4de6acd2eece6884fffb456c0b680 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: clone context of internal cmd in non-responding
by masayag＠redhat.com 22 Jun '15

22 Jun '15

Moti Asayag has submitted this change and it was merged. Change subject: clone context of internal cmd in non-responding ...................................................................... clone context of internal cmd in non-responding When we run internal action from the non-responding treatment scenario we should do that with the parent context clone and without an execution context That means that internal action will influence the parent command state but only the parent command will be shown in the task tree Additional information is reported to the log If we require to show those internal commands as steps of the non-responding treatment command we should change the code that create those commands to create and handle job steps Change-Id: Idddc9aef34fd4465fd9828a88c9775d0f96e54e0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1218548 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/RestartVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42606 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idddc9aef34fd4465fd9828a88c9775d0f96e54e0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: Fixed an issue with canDoAction message transalation
by sabose＠redhat.com 22 Jun '15

22 Jun '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Fixed an issue with canDoAction message transalation ...................................................................... gluster: Fixed an issue with canDoAction message transalation Change-Id: I604a961241d5067fba9bc07f5046e83f005e1b3b Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42654 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I604a961241d5067fba9bc07f5046e83f005e1b3b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: gluster: Fixed an issue with canDoAction message transalation
by sabose＠redhat.com 22 Jun '15

22 Jun '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Fixed an issue with canDoAction message transalation ...................................................................... gluster: Fixed an issue with canDoAction message transalation Change-Id: I604a961241d5067fba9bc07f5046e83f005e1b3b Bug-URL: https://bugzilla.redhat.com/1228063 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42653 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I604a961241d5067fba9bc07f5046e83f005e1b3b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: uutils: crypto: EnvelopeEncryptDecrypt: add artifact and ver...
by masayag＠redhat.com 22 Jun '15

22 Jun '15

Moti Asayag has submitted this change and it was merged. Change subject: uutils: crypto: EnvelopeEncryptDecrypt: add artifact and version ...................................................................... uutils: crypto: EnvelopeEncryptDecrypt: add artifact and version Change-Id: I81175bd9c32f3561c0de6c202d5bd9d444f4a071 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/crypto/EnvelopeEncryptDecrypt.java M backend/manager/modules/uutils/src/test/java/org/ovirt/engine/core/uutils/crypto/EnvelopeEncryptDecryptTest.java 2 files changed, 14 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42563 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81175bd9c32f3561c0de6c202d5bd9d444f4a071 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: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: uutils: crypto: add EnvelopePBE
by masayag＠redhat.com 22 Jun '15

22 Jun '15

Moti Asayag has submitted this change and it was merged. Change subject: uutils: crypto: add EnvelopePBE ...................................................................... uutils: crypto: add EnvelopePBE Change-Id: I029b01a4d65fb82cdaa04ee1ba10ffc6475b216d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- A backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/crypto/EnvelopePBE.java A backend/manager/modules/uutils/src/test/java/org/ovirt/engine/core/uutils/crypto/EnvelopePBETest.java 2 files changed, 148 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42539 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I029b01a4d65fb82cdaa04ee1ba10ffc6475b216d Gerrit-PatchSet: 4 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: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: Revert "packaging: setup: Disable pki renewal for 3.5"
by piotr.kliczewski＠gmail.com 22 Jun '15

22 Jun '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: Revert "packaging: setup: Disable pki renewal for 3.5" ...................................................................... Revert "packaging: setup: Disable pki renewal for 3.5" This reverts commit 33e47415416c1cef5e30fb85218d0fd3e5f02207. Bug-Url: https://bugzilla.redhat.com/1214860 Change-Id: I68aeaad978a8ede8fea64a313a205f074776f510 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py 1 file changed, 1 insertion(+), 3 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/42649 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68aeaad978a8ede8fea64a313a205f074776f510 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: Fix dependency on aaa-jdbc
by sbonazzo＠redhat.com 22 Jun '15

22 Jun '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: Fix dependency on aaa-jdbc ...................................................................... packaging: Fix dependency on aaa-jdbc Fixes dependency on aaa-jdbc extension by using real name instead of %{name}, because aaa-jdbc extension package is not renamed in downstream packaging. Change-Id: I4e3016c65401106c4aa7aed2933fdc445ed7fc75 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Verified Eyal Edri: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/42593 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e3016c65401106c4aa7aed2933fdc445ed7fc75 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Forbid hot plug availability when VM is suspended
by derez＠redhat.com 21 Jun '15

21 Jun '15

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Forbid hot plug availability when VM is suspended ...................................................................... webadmin: Forbid hot plug availability when VM is suspended When VM is on suspended status, the "Activate" and "Deactivate" buttons under Virtual Machines -> Disks were available for VirtIO disks and could be clicked. Since we don't support disk hot plug while VM's are in suspended status, and since it fails in the CDA anyway, this patch changes these buttons availability in this case. Change-Id: Ia2d610c3dda4445f6a071ba23a2b6abf6699fbc9 Bug-Url: https://bugzilla.redhat.com/1234098 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Allon Mureinik: Looks good to me, but someone else must approve Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/42610 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2d610c3dda4445f6a071ba23a2b6abf6699fbc9 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0