July 2015 - Engine-commits - oVirt List Archives

Change in jenkins[master]: ovirt-live: new job
by sbonazzo＠redhat.com 29 Jul '15

29 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-live: new job ...................................................................... ovirt-live: new job Requires: https://gerrit.ovirt.org/44113 Change-Id: Ia2e0a686ae650978c83b8a5e5add5406e8c5145b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A jobs/confs/yaml/jobs/ovirt-live/ovirt-live_build-artifacts.yaml A jobs/confs/yaml/scms/ovirt-live.yaml 2 files changed, 47 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/44082 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2e0a686ae650978c83b8a5e5add5406e8c5145b Gerrit-PatchSet: 4 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 <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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>

1 0

Change in ovirt-live[master]: automation: add check-patch
by sbonazzo＠redhat.com 29 Jul '15

29 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: add check-patch ...................................................................... automation: add check-patch execute make distcheck and verify python modules. Change-Id: I474661876f52e00f9c0c67845174f3ae50f5bf77 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A automation/check-patch.req A automation/check-patch.sh 2 files changed, 18 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests Simone Tiraboschi: Looks good to me, but someone else must approve David Caro: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44113 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I474661876f52e00f9c0c67845174f3ae50f5bf77 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Support changing memballoon device while VM is up
by ofrenkel＠redhat.com 29 Jul '15

29 Jul '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Support changing memballoon device while VM is up ...................................................................... core: Support changing memballoon device while VM is up Added support of enabling/disabling memballoon device while VM is running. New state of the device is stored in NEXT_RUN snapshot and applied when VM goes down. Change-Id: I8268c4e54f762f8dd70d2f94ec996c4db9857128 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1240660 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsBalloonEnabledQuery.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/IsBalloonEnabledQueryTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmManagementParametersBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 4 files changed, 29 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Omer Frenkel: Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/43561 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8268c4e54f762f8dd70d2f94ec996c4db9857128 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Add the new engine artifact jobs to publisher
by dcaroest＠redhat.com 29 Jul '15

29 Jul '15

David Caro has submitted this change and it was merged. Change subject: Add the new engine artifact jobs to publisher ...................................................................... Add the new engine artifact jobs to publisher Replacing the old one Change-Id: I14c8b70615308e77ca1e2a70ae960bab603bbeea Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-master.yaml 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/44094 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14c8b70615308e77ca1e2a70ae960bab603bbeea Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: Moving ovirt rpms building to standard ci
by dcaroest＠redhat.com 29 Jul '15

29 Jul '15

David Caro has submitted this change and it was merged. Change subject: Moving ovirt rpms building to standard ci ...................................................................... Moving ovirt rpms building to standard ci Right now that means using the mock_runner script Change-Id: I2a4621a0a4a53924d1ea49bea5d41a67dfa1c98b Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/confs/yaml/jobs/ovirt-engine/ovirt-engine_build-artifacts.yaml D jobs/confs/yaml/jobs/ovirt-engine/ovirt-engine_create-rpms.yaml 2 files changed, 18 insertions(+), 53 deletions(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/43719 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a4621a0a4a53924d1ea49bea5d41a67dfa1c98b Gerrit-PatchSet: 6 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: Add support for timed jobs to standard ci
by dcaroest＠redhat.com 29 Jul '15

29 Jul '15

David Caro has submitted this change and it was merged. Change subject: Add support for timed jobs to standard ci ...................................................................... Add support for timed jobs to standard ci It complicates a bit the template (another parameter) but allows timed triggers as we need for engine Change-Id: I8ec0302026a59891e4be852225489c6055294189 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/jobs/fabic-ovirt/fabric-ovirt_standard.yaml M jobs/confs/yaml/jobs/infra-docs/infra-docs.yaml M jobs/confs/yaml/jobs/ovirt-engine-sdk/ovirt-engine-sdk_build-artifacts.yaml M jobs/confs/yaml/jobs/ovirt-node/ovirt-node-job.yaml M jobs/confs/yaml/jobs/ovirt-register/ovirt-register.yaml M jobs/confs/yaml/jobs/ovirt-setup-lib/ovirt-setup-lib_build-artifacts.yaml M jobs/confs/yaml/jobs/pthreading/pthreading.yaml M jobs/confs/yaml/jobs/repoman/repoman_build-artifacts.yaml M jobs/confs/yaml/templates/standard-stage.yaml M jobs/confs/yaml/triggers/standard.yaml 10 files changed, 32 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/44092 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ec0302026a59891e4be852225489c6055294189 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: fixing for wildfly startup timings
by sbonazzo＠redhat.com 29 Jul '15

29 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fixing for wildfly startup timings ...................................................................... packaging: setup: fixing for wildfly startup timings Fixing to accomodate with wildfly startup timings Change-Id: Ie20b0855d03dc32d7097e6cfdc34fa2e70feea23 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/check_liveliness.py M src/plugins/ovirt-hosted-engine-setup/engine/health.py 2 files changed, 4 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44089 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie20b0855d03dc32d7097e6cfdc34fa2e70feea23 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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

1 0

Change in ovirt-engine[master]: core: Solve issue with Job/JPA
by oourfali＠redhat.com 29 Jul '15

29 Jul '15

Oved Ourfali has submitted this change and it was merged. Change subject: core: Solve issue with Job/JPA ...................................................................... core: Solve issue with Job/JPA When editing a Job, an hibernate exception concerning delete cascade would sometime pop-up. This patch sovles it. Change-Id: I33c460a02416c16d046402eb6ec3260a21874dd8 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/job/Job.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Oved Ourfali: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44066 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I33c460a02416c16d046402eb6ec3260a21874dd8 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: single DC network as management
by masayag＠redhat.com 29 Jul '15

29 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: single DC network as management ...................................................................... engine: single DC network as management The engine will choose a network as management if that is the only network in the DC and potentially could serve as a such in create new cluster flow. Change-Id: Idffae9aafd94fda13669c0897b376b5a39913c64 Bug-Url: https://bugzilla.redhat.com/1209041 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/DefaultManagementNetworkFinderImpl.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/GetManagementNetworkCandidatesQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/predicate/ManagementNetworkCandidatePredicate.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/cluster/DefaultManagementNetworkFinderImplTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/dc/GetManagementNetworkCandidatesQueryTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/dc/predicate/ManagementNetworkCandidatePredicateTest.java 6 files changed, 143 insertions(+), 21 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Verified; Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/44037 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idffae9aafd94fda13669c0897b376b5a39913c64 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: systemd: keep trying to register in case of failure during s...
by dougsland＠redhat.com 28 Jul '15

28 Jul '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: systemd: keep trying to register in case of failure during start ...................................................................... systemd: keep trying to register in case of failure during start The current approach for autoinstall/autoregister is try to register as soon supervdsm and network are ready but in case it failure a new registration must be triggered manually. This patch will make ovirt-node-plugin-vdsm service restart each 60 seconds in case the service/registration couldn't be triggered properly during the boot time. This behaviour is a mimic from vdsm-reg which must be kept. Change-Id: I8a50e5c8235413d428e9da8ceeec90e017eb63ee Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/44063 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Reviewed-by: Dan Kenigsberg <danken(a)redhat.com> --- M systemd/ovirt-node-plugin-vdsm.service 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests Fabian Deutsch: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44063 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a50e5c8235413d428e9da8ceeec90e017eb63ee Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0