July 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[master]: sdk: Regenerate agains the latest API
by juan.hernandez＠redhat.com 04 Jul '14

04 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Regenerate agains the latest API ...................................................................... sdk: Regenerate agains the latest API This patch regenerates the SDK using the latest API obtained from the engine built from commit db10f7e. Change-Id: I9e803717ce2df9cfe1bb3bb0857f3fe8c0da1306 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 4 files changed, 13,241 insertions(+), 988 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29579 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e803717ce2df9cfe1bb3bb0857f3fe8c0da1306 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[master]: init scripts: fixed the return value of status command
by msivak＠redhat.com 04 Jul '14

04 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: init scripts: fixed the return value of status command ...................................................................... init scripts: fixed the return value of status command Change-Id: I3a7f74248c803e2c96350cb70e1e94c52b24d745 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109545 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M initscripts/ovirt-ha-agent.init.in M initscripts/ovirt-ha-broker.init.in 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/29574 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a7f74248c803e2c96350cb70e1e94c52b24d745 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Deploy using modules
by juan.hernandez＠redhat.com 04 Jul '14

04 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Deploy using modules ...................................................................... restapi: Deploy using modules Currently the RESTAPI application is deployed as a nested .war file inside the .ear of the engine, actually as two .war files, one for /api, and another for /ovirt-engine/api. This double deployment causes class cast exceptions because classes are loaded twice. To avoid this issues this patch changes the RESTAPI application so that it is deployed outside of the .ear and users JBoss modules for all its components, so that no class will be loaded twice. Change-Id: Ie4079277f60e5521d9b61379fa78713a5c4ccc10 Bug-Url: https://bugzilla.redhat.com/1113485 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M backend/manager/modules/branding/pom.xml A backend/manager/modules/branding/src/main/modules/org/ovirt/engine/core/branding/main/module.xml M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/PackageExplorer.java A backend/manager/modules/restapi/interface/common/jaxrs/src/main/modules/org/ovirt/engine/api/interface-common-jaxrs/main/module.xml M backend/manager/modules/restapi/interface/definition/pom.xml D backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/FileUtils.java A backend/manager/modules/restapi/interface/definition/src/main/modules/org/ovirt/engine/api/restapi-definition/main/module.xml M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/ValidatorLocator.java A backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/MappingLocator.java A backend/manager/modules/restapi/types/src/main/modules/org/ovirt/engine/api/restapi-types/main/module.xml M backend/manager/modules/restapi/webapp/pom.xml D backend/manager/modules/restapi/webapp/src/main/resources/META-INF/MANIFEST.MF A backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/jboss-deployment-structure.xml A backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/jboss-web.xml M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml M ear/pom.xml M ear/src/main/resources/META-INF/MANIFEST.MF M ovirt-engine.spec.in M packaging/services/ovirt-engine/ovirt-engine.conf.in 25 files changed, 524 insertions(+), 239 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29474 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4079277f60e5521d9b61379fa78713a5c4ccc10 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: packaging: setup: allow reports on separate host
by ydary＠redhat.com 04 Jul '14

04 Jul '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: allow reports on separate host ...................................................................... packaging: setup: allow reports on separate host Change-Id: I426f040446e6288f481f9a256ddbfa35ce729e0b Bug-Url: https://bugzilla.redhat.com/1080998 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M .gitignore M Makefile M ovirt-engine-reports.spec.in A ovirt-engine-reports/postgres/module.xml A packaging/firewalld/reports/ovirt-reports-http.xml.in A packaging/firewalld/reports/ovirt-reports-https.xml.in A packaging/services/ovirt-engine-reportsd/__init__.py A packaging/services/ovirt-engine-reportsd/config.py.in A packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd-logging.properties.in A packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.conf.in A packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.py A packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.systemd.in A packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.sysv.in A packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.xml.in M packaging/setup/ovirt_engine_setup/reports/config.py.in M packaging/setup/ovirt_engine_setup/reports/constants.py M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/core/__init__.py M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/core/misc.py A packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/core/protocols.py A packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/core/service.py M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-reports/db/connection.py A packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-reports/core/__init__.py A packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-reports/core/service.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/apache/reports.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/__init__.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/firewall.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/protocols.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/core/__init__.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/core/misc.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/core/service.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/db/connection.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/pki/__init__.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/pki/apache.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/pki/jboss.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/pki/misc.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/provisioning/postgres.py D packaging/sys-etc/ovirt-engine/engine.conf.d/20-ovirt-engine-reports.conf.in 38 files changed, 2,917 insertions(+), 222 deletions(-) Approvals: Yedidyah Bar David: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28479 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I426f040446e6288f481f9a256ddbfa35ce729e0b Gerrit-PatchSet: 41 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[sdk_3.5]: codegen: Handle duplicated URL segments
by juan.hernandez＠redhat.com 04 Jul '14

04 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: codegen: Handle duplicated URL segments ...................................................................... codegen: Handle duplicated URL segments There are some entities and collections whose names can't be directly derived by the corresponding URL segments. For example, the following two URLs contain segments that are equal: hosts/{host:id}/numanodes vms/{vm:id}/numanodes But the corresponding types are "NumaNode" in the first case and "VirtualNumaNode" in the second, so it isn't possible to derive the type name using only the URL segment. To address this issue this patch introduces a table of exceptions that the generator will lookup. This table will initially contain the exception in the example above. Change-Id: I7fcbfe5e490bb23aa2e1785560dd016d63e3a9ae Bug-Url: https://bugzilla.redhat.com/1116119 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 3586ad0c68c6875827964ed934deb84aa27db5b3) --- M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/rsdl/RsdlCodegen.java M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/utils/ArrayUtils.java 2 files changed, 133 insertions(+), 33 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29576 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fcbfe5e490bb23aa2e1785560dd016d63e3a9ae Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[master]: codegen: Handle duplicated URL segments
by juan.hernandez＠redhat.com 04 Jul '14

04 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: codegen: Handle duplicated URL segments ...................................................................... codegen: Handle duplicated URL segments There are some entities and collections whose names can't be directly derived by the corresponding URL segments. For example, the following two URLs contain segments that are equal: hosts/{host:id}/numanodes vms/{vm:id}/numanodes But the corresponding types are "NumaNode" in the first case and "VirtualNumaNode" in the second, so it isn't possible to derive the type name using only the URL segment. To address this issue this patch introduces a table of exceptions that the generator will lookup. This table will initially contain the exception in the example above. Change-Id: I7fcbfe5e490bb23aa2e1785560dd016d63e3a9ae Bug-Url: https://bugzilla.redhat.com/1116119 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/rsdl/RsdlCodegen.java M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/utils/ArrayUtils.java 2 files changed, 133 insertions(+), 33 deletions(-) Approvals: Jason Liao: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29569 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fcbfe5e490bb23aa2e1785560dd016d63e3a9ae Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jason Liao <chuan.liao(a)hp.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk-java[sdk_3.5]: codegen: Handle plural exceptions
by juan.hernandez＠redhat.com 04 Jul '14

04 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: codegen: Handle plural exceptions ...................................................................... codegen: Handle plural exceptions The rules for forming plurals are more complicated than just adding a "s" to the end of the word. We need to implement them correctly, or have a table of exceptions. For simplicity this patch introduces a table of exceptions. Change-Id: I02d75c47af205f25dfe00c1c98dc0045f5d463f6 Bug-Url: https://bugzilla.redhat.com/1113548 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit c8d3d7484ec08109b49c42c717ace5e187bd23f7) --- M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/utils/StringUtils.java 1 file changed, 39 insertions(+), 15 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29575 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02d75c47af205f25dfe00c1c98dc0045f5d463f6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[master]: codegen: Handle plural exceptions
by juan.hernandez＠redhat.com 04 Jul '14

04 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: codegen: Handle plural exceptions ...................................................................... codegen: Handle plural exceptions The rules for forming plurals are more complicated than just adding a "s" to the end of the word. We need to implement them correctly, or have a table of exceptions. For simplicity this patch introduces a table of exceptions. Change-Id: I02d75c47af205f25dfe00c1c98dc0045f5d463f6 Bug-Url: https://bugzilla.redhat.com/1113548 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/utils/StringUtils.java 1 file changed, 39 insertions(+), 15 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29568 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02d75c47af205f25dfe00c1c98dc0045f5d463f6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Rename "kdump detection" to "kdump integration"
by oourfali＠redhat.com 04 Jul '14

04 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: Rename "kdump detection" to "kdump integration" ...................................................................... webadmin: Rename "kdump detection" to "kdump integration" Renames term "kdump detection" to "kdump integration" in user visible parts (UI and logs). Change-Id: I4bdcf78eb27902951f0298b13bf671bebd828fc6 Bug-Url: https://bugzilla.redhat.com/1079821 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 5 files changed, 11 insertions(+), 13 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Martin Peřina: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29478 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4bdcf78eb27902951f0298b13bf671bebd828fc6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: Moving question about WSP configuration to...
by oschreib＠redhat.com 04 Jul '14

04 Jul '14

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: Moving question about WSP configuration to product options ...................................................................... packaging: setup: Moving question about WSP configuration to product options Moving question about websocket proxy configuration to product options section. Change-Id: I594f6d396b3ad75dabc44dc187d5ac99687ad78b Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/websocket_proxy/config.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Simone Tiraboschi: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29178 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I594f6d396b3ad75dabc44dc187d5ac99687ad78b Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0