- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-java[master]: sdk: Regenerate against the latest API
by juan.hernandez＠redhat.com 23 Jun '15

23 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Regenerate against the latest API ...................................................................... sdk: Regenerate against the latest API This patch regenerates the SDK using the latest API obtained from the engine built from commit 304b7f5. Change-Id: Ieeb6a32982a80a39888b616e57d78363308aaa78 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/resources/api.xsd M generator/src/main/resources/rsdl.xml M sdk/Makefile M sdk/src/main/java/org/ovirt/engine/sdk/Api.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Cluster.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/ClusterPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Clusters.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/CpuProfilePermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterClusterPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterIscsiBondStorageConnection.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuota.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotaPermission.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotaPermissions.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotaQuotaClusterLimit.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotaQuotaClusterLimits.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotaQuotaStorageLimit.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotaQuotaStorageLimits.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterQuotas.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DataCenterStorageDomainDisks.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/DiskProfilePermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Disks.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Events.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/GroupPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/GroupRole.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/HostPermissions.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/Icon.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/Icons.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/InstanceTypes.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/NetworkPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackImageProvider.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackImageProviders.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackNetworkProvider.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackNetworkProviders.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackVolumeProvider.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackVolumeProviderOpenstackVolumeAuthenticationKey.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackVolumeProviderOpenstackVolumeAuthenticationKeys.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/OpenStackVolumeProviders.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Permissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageDomain.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageDomainDisks.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageDomainImage.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageDomainPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Template.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/TemplatePermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Templates.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/UserPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/UserRole.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VM.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMDisk.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMDisks.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMGraphicsConsole.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMGraphicsConsoles.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMs.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VmPoolPermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VnicProfilePermissions.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Action.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/BaseResource.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/BaseResources.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/DataCenter.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Disk.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Display.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/DisplayDisconnectActions.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/ExternalStatuses.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/GraphicsConsole.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/GraphicsConsoles.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/GraphicsTypes.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/GuestOperatingSystem.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Host.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/IO.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/Icon.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/Icons.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/KSM.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/Kernel.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/LogicalUnits.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/ObjectFactory.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/OpenstackVolumeAuthenticationKey.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/OpenstackVolumeAuthenticationKeys.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/OperatingSystemInfo.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Quota.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/QuotaClusterLimit.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/QuotaClusterLimits.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/QuotaModeTypes.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/QuotaStorageLimit.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/QuotaStorageLimits.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/StorageDomain.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/TimeZone.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/VM.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/VersionCaps.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/VmBase.java 91 files changed, 7,737 insertions(+), 290 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42767 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieeb6a32982a80a39888b616e57d78363308aaa78 Gerrit-PatchSet: 1 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: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: Fixing the artifact building script for CI
by fabiand＠redhat.com 23 Jun '15

23 Jun '15

Fabian Deutsch has submitted this change and it was merged. Change subject: Fixing the artifact building script for CI ...................................................................... Fixing the artifact building script for CI Change-Id: I96361e8227986372a0ab21c8410c8b615b920608 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/42758 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> --- M automation/build-artifacts.sh 1 file changed, 6 insertions(+), 8 deletions(-) Approvals: Fabian Deutsch: Looks good to me, approved Jenkins CI: Passed CI tests Anatoly Litovsky: Verified -- To view, visit https://gerrit.ovirt.org/42758 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96361e8227986372a0ab21c8410c8b615b920608 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix mocking for _perClusterProvider.get()
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix mocking for _perClusterProvider.get() ...................................................................... core: Fix mocking for _perClusterProvider.get() Changed mocking to use the doReturn(...).when(...) style to overcome Mockito's shortcomings with the traditional when(...).thenReturn(...) style. Change-Id: Iefe2e25028f0493eef3113e7e67b01be1e7cd972 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42618 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iefe2e25028f0493eef3113e7e67b01be1e7cd972 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Add missing configuration mocking
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Add missing configuration mocking ...................................................................... core: Add missing configuration mocking Added missing configuration mocks to AffinityRulesEnforcementManagerTest. Change-Id: I806d6caa4ce380ac7e5acaf0ecbe8fe107a68d17 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42617 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I806d6caa4ce380ac7e5acaf0ecbe8fe107a68d17 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: AffinityRulesEnforcementManagerTest imports
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: AffinityRulesEnforcementManagerTest imports ...................................................................... core: AffinityRulesEnforcementManagerTest imports Statically import org.ovirt.engine.core.utils.MockConfigRule.mockConfig and org.junit.Assert.assertNotNull to make the code cleaner. Change-Id: I66fdbf418ae5f1074404ef12573221c44de86384 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 7 insertions(+), 6 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42616 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66fdbf418ae5f1074404ef12573221c44de86384 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Cleanup members' modifiers
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Cleanup members' modifiers ...................................................................... core: Cleanup members' modifiers Made instance members private, as per proper Java conventions, where possible. Change-Id: Ia56ab4aa1b579bd7f3bf28910ebe211568d530ba Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 9 insertions(+), 9 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42622 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia56ab4aa1b579bd7f3bf28910ebe211568d530ba Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Removed redundant System.out.printlns
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Removed redundant System.out.printlns ...................................................................... core: Removed redundant System.out.printlns Removed printing to stdout, which have no place in an automatic unit test. Change-Id: Iea60a2b3d0c0265397899894bdc3595036433c29 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 0 insertions(+), 9 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42627 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea60a2b3d0c0265397899894bdc3595036433c29 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: unignore AffinityRulesEnforcementManagerTest
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: unignore AffinityRulesEnforcementManagerTest ...................................................................... core: unignore AffinityRulesEnforcementManagerTest Since AffinityRulesEnforcementManagerTest no longer fails, the @Ignore annotation can be removed. Change-Id: Ib680f8576617c905a490df827b6813751b97f9b0 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42621 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib680f8576617c905a490df827b6813751b97f9b0 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: TestingAffinityRulesEnforcementPerCluster
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: TestingAffinityRulesEnforcementPerCluster ...................................................................... core: TestingAffinityRulesEnforcementPerCluster Previously, this test used an anonymous inner extension of AffinityRulesEnforcementPerCluster that assigned mocks to its DAOs in the wakeup() method. This behavior caused a NullPointerException since the DAOs were also used elsewhere. This method extracts the anonymous class to a named inner class to allow setting the DAOs in the constructor and use throughout the class' life. Change-Id: I2b722fb78d1ec32652a0315e1fcc88ebd46ff8e1 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 10 insertions(+), 10 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42619 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b722fb78d1ec32652a0315e1fcc88ebd46ff8e1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix assertion
by amureini＠redhat.com 23 Jun '15

23 Jun '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix assertion ...................................................................... core: Fix assertion Fixed the assertion in AffinityRulesEnforcementManagerTest#positiveEnforcementUAGTest to have the test run. Change-Id: I9c2b1a349d045735b36b480cc214412b542d0733 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42620 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c2b1a349d045735b36b480cc214412b542d0733 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0