Change in ovirt-engine[master]: automation: add isort check
by sbonazzo@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: python: cleanup import lines using isort
by sbonazzo@redhat.com
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>
8 years, 3 months
Change in jenkins[master]: ovirt-engine: added fc24 on master check patch
by sbonazzo@redhat.com
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>
8 years, 3 months
Change in jenkins[master]: ovirt-engine: drop 4.0.2 jobs
by sbonazzo@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: automation: dropped el6 configs
by sbonazzo@redhat.com
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>
8 years, 3 months
Change in ovirt-engine-sdk[sdk_4.0]: README: add missing quote.
by omachace@redhat.com
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>
8 years, 3 months
Change in ovirt-engine-sdk-java[sdk_4.0]: Add missing required packages to spec files
by omachace@redhat.com
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>
8 years, 3 months
Change in ovirt-engine-sdk-java[master]: Add missing required packages to spec files
by omachace@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Bind PluginActionButtonHandler as eager singleton
by piotr.kliczewski@gmail.com
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>
8 years, 3 months
Change in ovirt-engine[master]: packaging: require otopi 1.5.2
by sbonazzo@redhat.com
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>
8 years, 3 months