August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: automation: add isort check
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: add isort check ...................................................................... automation: add isort check Added isort check for python code Issue: https://ovirt-jira.atlassian.net/browse/OVIRT-702 Change-Id: Icc4227e66641934153873c28cf53519ae3fce8d9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A .isort.cfg M Makefile A automation/check-patch.packages.fc24 M build/python-check.sh.in 4 files changed, 39 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62866 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc4227e66641934153873c28cf53519ae3fce8d9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: python: cleanup import lines using isort
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: python: cleanup import lines using isort ...................................................................... python: cleanup import lines using isort Ran again isort for passing automation checks introduced in next commit. Change-Id: I9db9cf798706e8fd364ecbf8470234078ef38a77 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.py M packaging/services/ovirt-engine/ovirt-engine.py M packaging/services/ovirt-fence-kdump-listener/listener.py M packaging/services/ovirt-fence-kdump-listener/ovirt-fence-kdump-listener.py M packaging/services/ovirt-websocket-proxy/ovirt-websocket-proxy.py M packaging/setup/plugins/ovirt-engine-common/base/network/process_firewalld_services.py 6 files changed, 11 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62865 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9db9cf798706e8fd364ecbf8470234078ef38a77 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-engine: added fc24 on master check patch
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: added fc24 on master check patch ...................................................................... ovirt-engine: added fc24 on master check patch Added fc24 check patch to master since isort is not available in el7 Change-Id: I7481d37a4de7574491e68e411a9aa8aac358e3cd Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 9 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/62868 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7481d37a4de7574491e68e411a9aa8aac358e3cd Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-engine: drop 4.0.2 jobs
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: drop 4.0.2 jobs ...................................................................... ovirt-engine: drop 4.0.2 jobs 4.0.2 has been released, dropping jobs Change-Id: Ia211337654a796e0497954aa856ce6913bfa41d8 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62867 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia211337654a796e0497954aa856ce6913bfa41d8 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: automation: dropped el6 configs
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: dropped el6 configs ...................................................................... automation: dropped el6 configs el6 is not supported anymore since 4.0. Dropping el6 configuration. Change-Id: I5f326c79a9b4b4d3250cc0b3718508c330f059ea Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- D automation/check-patch.packages.el6 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62864 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f326c79a9b4b4d3250cc0b3718508c330f059ea Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.0]: README: add missing quote.
by omachace＠redhat.com 26 Aug '16

26 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: README: add missing quote. ...................................................................... README: add missing quote. Change-Id: I71e2c07e4c42eaa8e890793a37d8f904d92a42d2 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> (cherry picked from commit edcc8191e227efebf274b6c155ff6f628f3eab1f) --- M sdk/README.adoc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62847 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71e2c07e4c42eaa8e890793a37d8f904d92a42d2 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-java[sdk_4.0]: Add missing required packages to spec files
by omachace＠redhat.com 26 Aug '16

26 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add missing required packages to spec files ...................................................................... Add missing required packages to spec files This patch adds missing required packages to all spec files. In addition it removes maven profiles, as it's not neede anymore. Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 224b9ce068a935c613223386c908896e3453cfeb) --- M generator/pom.xml M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in M sdk/pom.xml 5 files changed, 62 insertions(+), 65 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62863 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java 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-java[master]: Add missing required packages to spec files
by omachace＠redhat.com 26 Aug '16

26 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add missing required packages to spec files ...................................................................... Add missing required packages to spec files This patch adds missing required packages to all spec files. In addition it removes maven profiles, as it's not neede anymore. Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/pom.xml M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in M sdk/pom.xml 5 files changed, 62 insertions(+), 65 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62670 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad96cf594d0d286708c4adb71cfa405787bae365 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java 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[ovirt-engine-4.0]: webadmin: Bind PluginActionButtonHandler as eager singleton
by piotr.kliczewski＠gmail.com 26 Aug '16

26 Aug '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Bind PluginActionButtonHandler as eager singleton ...................................................................... webadmin: Bind PluginActionButtonHandler as eager singleton This fixes an issue where main/sub tab action buttons added by UI plugins would not appear at all. The handler class must be bound as eager singleton to ensure that all AddTabActionButtonEvent instances are processed. Change-Id: Ief464ec72b4b9e3cfdfc2a476c600f2e7501d094 Bug-Url: https://bugzilla.redhat.com/1369235 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseUtilsModule.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/62819 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief464ec72b4b9e3cfdfc2a476c600f2e7501d094 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: require otopi 1.5.2
by sbonazzo＠redhat.com 26 Aug '16

26 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: require otopi 1.5.2 ...................................................................... packaging: require otopi 1.5.2 Require new otopi libraries in order to include fixes addressed by the new release. Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Related-To: https://bugzilla.redhat.com/1350869 Related-To: https://bugzilla.redhat.com/1369099 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62860 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15cb6b16b82118925c645ed9c640049afb0c1554 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0