- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine : Fix DAO tests TemplateOwner can't edit template and...
by oourfali＠redhat.com 06 Jul '14

06 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: engine : Fix DAO tests TemplateOwner can't edit template and add/edit vnic in userportal. ...................................................................... engine : Fix DAO tests TemplateOwner can't edit template and add/edit vnic in userportal. Fix DAO tests Change-Id: I11cf9a0979b5cf1000406df8c5cb0c2e1a7625b5 Bug-Url: https://bugzilla.redhat.com/953043 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsGroupDAOTest.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ravi Nori: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29597 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11cf9a0979b5cf1000406df8c5cb0c2e1a7625b5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Retrieving only foreman's compute resources that relat...
by oourfali＠redhat.com 04 Jul '14

04 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Retrieving only foreman's compute resources that relates to oVirt ...................................................................... core: Retrieving only foreman's compute resources that relates to oVirt Adding search field to gather only ovirt info Change-Id: I859c786df87c58abbdca722a19f2e431e72c40e9 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostProviderProxy.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29186 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I859c786df87c58abbdca722a19f2e431e72c40e9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Using signed password for ovirt-node provisioning
by oourfali＠redhat.com 04 Jul '14

04 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Using signed password for ovirt-node provisioning ...................................................................... core: Using signed password for ovirt-node provisioning Using MD5 signature util. Change-Id: I82d9e0f368e3b6e36098b33863ff7998f789688b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109540 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostProviderProxy.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29185 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82d9e0f368e3b6e36098b33863ff7998f789688b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: uutils: add CryptMD5
by oourfali＠redhat.com 04 Jul '14

04 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: uutils: add CryptMD5 ...................................................................... uutils: add CryptMD5 Change-Id: Ice3c45197dd6ff27fd20dcff1e8249c815a756a7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- A backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/crypto/CryptMD5.java 1 file changed, 134 insertions(+), 0 deletions(-) Approvals: Yaniv Bronhaim: Verified; Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28659 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ice3c45197dd6ff27fd20dcff1e8249c815a756a7 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core : TemplateOwner can't edit template and add/edit vnic i...
by oourfali＠redhat.com 04 Jul '14

04 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core : TemplateOwner can't edit template and add/edit vnic in userportal. ...................................................................... core : TemplateOwner can't edit template and add/edit vnic in userportal. When user have direct TemplateOwner role on template, he can't edit template in UserPortal and also can't add/edit vnic of template. Change-Id: I724187757513965ba6557d9d94ce7d3857a521a4 Bug-Url: https://bugzilla.redhat.com/953043 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M packaging/dbscripts/create_views.sql 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Ravi Nori: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29475 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I724187757513965ba6557d9d94ce7d3857a521a4 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

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