May 2016 - Engine-commits - oVirt List Archives

Change in jenkins[master]: nightly-3.6: use 3.6 cockpit-ovirt
by sbonazzo＠redhat.com 11 May '16

11 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: nightly-3.6: use 3.6 cockpit-ovirt ...................................................................... nightly-3.6: use 3.6 cockpit-ovirt use 3.6 branch of cockpit-ovirt instead of master Change-Id: I6bc9c96c379f152b450dd5f24200f9af63f3b796 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me Sandro Bonazzola: Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57332 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6bc9c96c379f152b450dd5f24200f9af63f3b796 Gerrit-PatchSet: 2 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>

1 0

Change in jenkins[master]: ioprocess: Add standard automation configuration
by dcaro＠redhat.com 11 May '16

11 May '16

David Caro has submitted this change and it was merged. Change subject: ioprocess: Add standard automation configuration ...................................................................... ioprocess: Add standard automation configuration Replace old ioprocess yaml with standard automation configuration based on pthreading. Add another project for ioprocess-0.15.1 branch, base on the previous configuration. Change-Id: I30f17c15657c2436758264d03ffa2c45bca2f9ed Signed-off-by: Nir Soffer <nsoffer(a)redhat.com> --- A jobs/confs/projects/ioprocess/ioprocess.yaml D jobs/confs/projects/ioprocess/ioprocess_create-rpms.yaml 2 files changed, 45 insertions(+), 54 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for review Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/57321 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30f17c15657c2436758264d03ffa2c45bca2f9ed Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: Engine: clear guest tools needed flag if the VM is up to date
by tnisan＠redhat.com 11 May '16

11 May '16

Tal Nisan has submitted this change and it was merged. Change subject: Engine: clear guest tools needed flag if the VM is up to date ...................................................................... Engine: clear guest tools needed flag if the VM is up to date Change-Id: I1dad8c0757e9d87e35b09073a3ca38cbeca1bfd5 Bug-Url: https://bugzilla.redhat.com/1332101 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 1 file changed, 15 insertions(+), 6 deletions(-) Approvals: Shahar Havivi: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57324 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1dad8c0757e9d87e35b09073a3ca38cbeca1bfd5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 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>

1 0

Change in ovirt-engine[master]: tools: fixed broken tests
by masayag＠redhat.com 11 May '16

11 May '16

Moti Asayag has submitted this change and it was merged. Change subject: tools: fixed broken tests ...................................................................... tools: fixed broken tests The provided json was not correct - provided the correct one Change-Id: Iaa3f60d2814abc3aaf874460021570b9eba7fa4b Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/tools/src/test/java/org/ovirt/engine/core/config/entity/helper/MigrationPoliciesValueHelperTest.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Moti Asayag: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/57329 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa3f60d2814abc3aaf874460021570b9eba7fa4b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in jenkins[master]: Build cockpit-ovirt for 3.6 and 4.0
by sbonazzo＠redhat.com 11 May '16

11 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Build cockpit-ovirt for 3.6 and 4.0 ...................................................................... Build cockpit-ovirt for 3.6 and 4.0 Currently this only builds for master. Add builders for 3.6 and 4.0 as well Change-Id: I2bd126b6deb472438eaa21d0bcecd6fdb6aa7e25 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M jobs/confs/projects/cockpit-ovirt/cockpit-ovirt.yaml 1 file changed, 4 insertions(+), 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/57255 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bd126b6deb472438eaa21d0bcecd6fdb6aa7e25 Gerrit-PatchSet: 2 Gerrit-Project: jenkins 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: Ryan Barry <rbarry(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]: engine: Support events in CoCo
by masayag＠redhat.com 11 May '16

11 May '16

Moti Asayag has submitted this change and it was merged. Change subject: engine: Support events in CoCo ...................................................................... engine: Support events in CoCo Command coordinator infrastructure will support registration to events from commands, where the command's callback will be invoked upon receiving the event from vdsm. Change-Id: I6d5b8bc3d6f57e9da96f7983eb732f52821fc170 Bug-Url: https://bugzilla.redhat.com/1284903 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CallbackTiming.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CoCoEventSubscriber.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCallbacksPoller.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCoordinatorImpl.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCoordinatorUtil.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandsRepository.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/interfaces/CommandCallback.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/interfaces/CommandCoordinator.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/CommandEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 12 files changed, 308 insertions(+), 93 deletions(-) Approvals: Moti Asayag: Verified; Passed CI tests Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52961 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d5b8bc3d6f57e9da96f7983eb732f52821fc170 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: allow vdsm jobs to run also on f23 slaves
by eedri＠redhat.com 11 May '16

11 May '16

Eyal Edri has submitted this change and it was merged. Change subject: allow vdsm jobs to run also on f23 slaves ...................................................................... allow vdsm jobs to run also on f23 slaves Change-Id: I95840f0e4a16f216644384bafda25c1a88d81dd8 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/vdsm/vdsm_standard.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Eyal Edri: Verified; Ready for merge Gil Shinar: Looks good to me Pavel Zhukov: Looks good to me Jenkins CI: Passed CI tests David Caro: Looks good to me -- To view, visit https://gerrit.ovirt.org/57316 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95840f0e4a16f216644384bafda25c1a88d81dd8 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(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: Pavel Zhukov <pzhukov(a)redhat.com>

1 0

Change in ovirt-engine[master]: build: Fix PEP8 issue during developer build
by sbonazzo＠redhat.com 11 May '16

11 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: Fix PEP8 issue during developer build ...................................................................... build: Fix PEP8 issue during developer build Change-Id: I8623b0e1328769a5180aadf619bd6794f744124e Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M build/helptag-oneline-check.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Martin Sivák: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56814 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8623b0e1328769a5180aadf619bd6794f744124e Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> 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]: packaging: setup: Add option --reconfigure-dwh-scale
by sbonazzo＠redhat.com 11 May '16

11 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Add option --reconfigure-dwh-scale ...................................................................... packaging: setup: Add option --reconfigure-dwh-scale Change-Id: I76f66f26f22b448506ba0e9135c8b1d825c6fae8 Bug-Url: https://bugzilla.redhat.com/1328805 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/bin/ovirt-engine-setup 1 file changed, 5 insertions(+), 0 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/57303 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76f66f26f22b448506ba0e9135c8b1d825c6fae8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in gerrit-admin[master]: Added docs
by eedri＠redhat.com 11 May '16

11 May '16

Eyal Edri has submitted this change and it was merged. Change subject: Added docs ...................................................................... Added docs * Documenting the bash scripts with doxygen * Documenting the python ones with sphinx * Bundling it all together in html with sphinx * Formatting the comments to be properly picked up * Added automation dir and scripts to build the docs on ci Change-Id: Iafc57df0c707ee1a06a3f9707a94968a3235cf60 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A Makefile A automation/build-artifacts.packages A automation/build-artifacts.sh A automation/check-merged.packages A automation/check-merged.sh A automation/check-patch.packages A automation/check-patch.sh A automation/common.sh A docs/Doxyfile A docs/Doxyfile_hooks A docs/Makefile A docs/requires.txt A docs/source/Adding_hooks.rst A docs/source/Bash_hooks.rst A docs/source/Bash_libs.rst A docs/source/Configuration.rst A docs/source/Hook_dispatcher.rst A docs/source/Installation.rst A docs/source/Python_hooks.rst A docs/source/Python_libs.rst A docs/source/conf.py A docs/source/index.rst M hooks/custom_hooks/change-merged.set_MODIFIED A hooks/custom_hooks/change-merged.set_MODIFIED.sh M hooks/custom_hooks/comment-added.propagate_review_values A hooks/custom_hooks/comment_added_propagate_review_values.py M hooks/custom_hooks/patchset-created.bz.0.has_bug_url A hooks/custom_hooks/patchset-created.bz.0.has_bug_url.sh M hooks/custom_hooks/patchset-created.bz.1.is_public A hooks/custom_hooks/patchset-created.bz.1.is_public.sh M hooks/custom_hooks/patchset-created.bz.2.correct_product A hooks/custom_hooks/patchset-created.bz.2.correct_product.sh M hooks/custom_hooks/patchset-created.bz.3.correct_target_milestone A hooks/custom_hooks/patchset-created.bz.3.correct_target_milestone.sh M hooks/custom_hooks/patchset-created.bz.3.correct_target_release A hooks/custom_hooks/patchset-created.bz.3.correct_target_release.sh M hooks/custom_hooks/patchset-created.bz.98.set_POST A hooks/custom_hooks/patchset-created.bz.98.set_POST.sh M hooks/custom_hooks/patchset-created.bz.99.review_ok A hooks/custom_hooks/patchset-created.bz.99.review_ok.sh M hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch A hooks/custom_hooks/patchset_created_warn_if_not_merged_to_previous_branch.py M hooks/custom_hooks/update_tracker A hooks/custom_hooks/update_tracker.py M hooks/hook-dispatcher A hooks/hook_dispatcher.py M hooks/lib/bz.sh M hooks/lib/conf.sh M hooks/lib/config.py M hooks/lib/gerrit.sh M hooks/lib/tools.sh A scripts/doxygen-bash.sed 52 files changed, 6,504 insertions(+), 631 deletions(-) Approvals: Eyal Edri: Looks good to me, approved Jenkins CI: Passed CI tests David Caro: Verified -- To view, visit https://gerrit.ovirt.org/56819 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iafc57df0c707ee1a06a3f9707a94968a3235cf60 Gerrit-PatchSet: 9 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(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: gerrit-hooks <automation(a)ovirt.org>

1 0