July 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[master]: sdk: release 3.1.0.4-1
by mpastern＠redhat.com 09 Jul '12

09 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: release 3.1.0.4-1 ...................................................................... sdk: release 3.1.0.4-1 Change-Id: Ieb463bfaac773ce6b809e07a120feb2250941844 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-sdk.spec.in M setup.py 3 files changed, 17 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6070 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb463bfaac773ce6b809e07a120feb2250941844 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Don't expand entities
by mpastern＠redhat.com 09 Jul '12

09 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: Don't expand entities ...................................................................... restapi: Don't expand entities Configure RESTEasy so that the built-in unmarshaller for org.w3c.dom.Document will not expand entities. Change-Id: I2c2ca449be9f0d525f990c38384fdc6a6a004568 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6059 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c2ca449be9f0d525f990c38384fdc6a6a004568 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: rebase to latest api
by mpastern＠redhat.com 09 Jul '12

09 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: rebase to latest api ...................................................................... sdk: rebase to latest api fix typo in 'storage' based args documentation make 'name' optional in storagedomains.add() to alow importing existent SD make api.storagedomains collection searchable Change-Id: I66769cd99eefe3d51b60d359f36e6c311ed45b08 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/brokers.py 1 file changed, 40 insertions(+), 39 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6068 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66769cd99eefe3d51b60d359f36e6c311ed45b08 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Olivier Doucet <webmaster(a)ajeux.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Making code readable
by mkublin＠redhat.com 09 Jul '12

09 Jul '12

Michael Kublin has submitted this change and it was merged. Change subject: core: Making code readable ...................................................................... core: Making code readable The following patch is fixing an unreadable code What was fixed: 1. Using for each pattern 2. Removing unneeded object creation 3. Removed unneeded code 4. Make code simple Change-Id: Icb974c97a6f6f4ccf9841ec4d14cb3e216836bb1 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveDiskCommand.java 1 file changed, 12 insertions(+), 12 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5924 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb974c97a6f6f4ccf9841ec4d14cb3e216836bb1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Remove unused options from engine-config
by msalem＠redhat.com 09 Jul '12

09 Jul '12

Muli Salem has submitted this change and it was merged. Change subject: tools: Remove unused options from engine-config ...................................................................... tools: Remove unused options from engine-config This patch removes a few unused options from the engine-config.properties file. Signed-off-by: Muli Salem <msalem(a)redhat.com> Change-Id: Ie06b9c0337aa28415cf39ab8becfc604fdc758c8 --- M backend/manager/tools/engine-config/src/main/resources/engine-config.properties M backend/manager/tools/engine-config/src/test/java/org/ovirt/engine/core/config/EngineConfigLogicTest.java 2 files changed, 2 insertions(+), 9 deletions(-) Approvals: Muli Salem: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5980 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie06b9c0337aa28415cf39ab8becfc604fdc758c8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Muli Salem <msalem(a)redhat.com> Gerrit-Reviewer: Muli Salem <msalem(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Error Message For Create User With Wrong Domain
by oliel＠redhat.com 09 Jul '12

09 Jul '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Error Message For Create User With Wrong Domain ...................................................................... restapi: Error Message For Create User With Wrong Domain Change-Id: I316503b93a8b4ef8529692c6cf22fdca4d433035 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendUsersResourceBase.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5960 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I316503b93a8b4ef8529692c6cf22fdca4d433035 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: SD name not mandatory for SD import
by oliel＠redhat.com 09 Jul '12

09 Jul '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: SD name not mandatory for SD import ...................................................................... restapi: SD name not mandatory for SD import Change-Id: Ib63434f63f4468871b1554ffd3f9d2db8169962c Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml 1 file changed, 20 insertions(+), 20 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5926 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib63434f63f4468871b1554ffd3f9d2db8169962c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Map Disk Alias To Disk Name
by oliel＠redhat.com 09 Jul '12

09 Jul '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Map Disk Alias To Disk Name ...................................................................... restapi: Map Disk Alias To Disk Name Backend has both 'name' and 'alias' for disk, for historic reasons. API will expose only 'name' but behind the scenes map it to 'alias'. Change-Id: I7d85a8e3b04a121e2139df213f023c3ef606a59f Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDisksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendDisksResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/DiskMapper.java 5 files changed, 9 insertions(+), 3 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5946 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d85a8e3b04a121e2139df213f023c3ef606a59f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: release 3.1.0.4-1
by mpastern＠redhat.com 09 Jul '12

09 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: release 3.1.0.4-1 ...................................................................... cli: release 3.1.0.4-1 Change-Id: I831b691454c4f71a2a27395787decc92f1f6eb53 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-cli.spec.in M setup.py 3 files changed, 8 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6064 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I831b691454c4f71a2a27395787decc92f1f6eb53 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: do not pass empty host-subject
by mpastern＠redhat.com 09 Jul '12

09 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: do not pass empty host-subject ...................................................................... cli: do not pass empty host-subject Change-Id: I144a4ab4a01e74bdacc52991e707e7acd37fb2be Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/platform/posix/spice.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6063 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I144a4ab4a01e74bdacc52991e707e7acd37fb2be Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0