- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Use matrix parameter instead of action to force rem...
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use matrix parameter instead of action to force remove data center ...................................................................... restapi: Use matrix parameter instead of action to force remove data center The operation that removes a data center supports a "force" parameter. In order to use it the "DELETE" operation used to support an optional action parameter: DELETE /datacenters/{datacenter:id} <action> <force>true</force> </action> This patch removes the optional action parameter and replaces it with an optional matrix parameter: DELETE /datacenters/{datacenter:id};force=true Change-Id: I77873f2401f51f1c94b80ac6a05d8bb4c33e5d9e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.md M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DataCenterResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterResourceTest.java 5 files changed, 60 insertions(+), 55 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/47873 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77873f2401f51f1c94b80ac6a05d8bb4c33e5d9e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Use JAX-RS Response instead of entity in policy units
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use JAX-RS Response instead of entity in policy units ...................................................................... restapi: Use JAX-RS Response instead of entity in policy units Currently the JAX-RS resource interfaces for policy units (filters, balances and weights) return the entity type as the result of the "add" methods. All the other resources of the RESTAPI return the JAX-RS "Reponse" type. This patch changes the policy units resources so that they return "Response", like the rest of the resources. Change-Id: I8048c47e75b45f0f5be9e542a4f38e19538772a0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/BalancesResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/FiltersResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/WeightsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendBalancesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendFiltersResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendPolicyUnitsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendWeightsResource.java 7 files changed, 20 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47869 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8048c47e75b45f0f5be9e542a4f38e19538772a0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Use metamodel
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use metamodel ...................................................................... restapi: Use metamodel This patch replaces the XML schema used to define the data model with the equivalent definition using the metamodel. All business logic entities have been replaced, but infrastructure entities (like "Action" and "Capabilities", for example) haven't been replaced, they will be replaced later. Change-Id: I739d7fa1c07eea37858f12704ed1a7cf1d18266c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/metamodel/tool/src/main/java/org/ovirt/api/metamodel/tool/SchemaGenerator.java C backend/manager/modules/restapi/model/src/main/java/types/AffinityGroup.java C backend/manager/modules/restapi/model/src/main/java/types/Agent.java C backend/manager/modules/restapi/model/src/main/java/types/AgentConfiguration.java C backend/manager/modules/restapi/model/src/main/java/types/Application.java C backend/manager/modules/restapi/model/src/main/java/types/AuthorizedKey.java C backend/manager/modules/restapi/model/src/main/java/types/Balance.java C backend/manager/modules/restapi/model/src/main/java/types/Bios.java C backend/manager/modules/restapi/model/src/main/java/types/BlockStatistic.java C backend/manager/modules/restapi/model/src/main/java/types/Bonding.java C backend/manager/modules/restapi/model/src/main/java/types/Bookmark.java C backend/manager/modules/restapi/model/src/main/java/types/Boot.java C backend/manager/modules/restapi/model/src/main/java/types/BootMenu.java C backend/manager/modules/restapi/model/src/main/java/types/BrickProfileDetail.java C backend/manager/modules/restapi/model/src/main/java/types/Cdrom.java C backend/manager/modules/restapi/model/src/main/java/types/Certificate.java C backend/manager/modules/restapi/model/src/main/java/types/CloudInit.java A backend/manager/modules/restapi/model/src/main/java/types/Cluster.java C backend/manager/modules/restapi/model/src/main/java/types/Configuration.java C backend/manager/modules/restapi/model/src/main/java/types/Console.java C backend/manager/modules/restapi/model/src/main/java/types/Core.java C backend/manager/modules/restapi/model/src/main/java/types/Cpu.java C backend/manager/modules/restapi/model/src/main/java/types/CpuProfile.java C backend/manager/modules/restapi/model/src/main/java/types/CpuTopology.java C backend/manager/modules/restapi/model/src/main/java/types/CpuTune.java C backend/manager/modules/restapi/model/src/main/java/types/CustomProperty.java C backend/manager/modules/restapi/model/src/main/java/types/DataCenter.java A backend/manager/modules/restapi/model/src/main/java/types/Device.java A backend/manager/modules/restapi/model/src/main/java/types/Disk.java C backend/manager/modules/restapi/model/src/main/java/types/DiskProfile.java C backend/manager/modules/restapi/model/src/main/java/types/DiskSnapshot.java A backend/manager/modules/restapi/model/src/main/java/types/Display.java C backend/manager/modules/restapi/model/src/main/java/types/Dns.java C backend/manager/modules/restapi/model/src/main/java/types/Domain.java C backend/manager/modules/restapi/model/src/main/java/types/EntityProfileDetail.java C backend/manager/modules/restapi/model/src/main/java/types/ErrorHandling.java A backend/manager/modules/restapi/model/src/main/java/types/Event.java C backend/manager/modules/restapi/model/src/main/java/types/ExternalComputeResource.java C backend/manager/modules/restapi/model/src/main/java/types/ExternalDiscoveredHost.java C backend/manager/modules/restapi/model/src/main/java/types/ExternalHost.java C backend/manager/modules/restapi/model/src/main/java/types/ExternalHostGroup.java C backend/manager/modules/restapi/model/src/main/java/types/ExternalHostProvider.java C backend/manager/modules/restapi/model/src/main/java/types/ExternalProvider.java C backend/manager/modules/restapi/model/src/main/java/types/FencingPolicy.java C backend/manager/modules/restapi/model/src/main/java/types/File.java C backend/manager/modules/restapi/model/src/main/java/types/Filter.java C backend/manager/modules/restapi/model/src/main/java/types/Floppy.java C backend/manager/modules/restapi/model/src/main/java/types/FopStatistic.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterBrick.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterBrickAdvancedDetails.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterBrickMemoryInfo.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterClient.java A backend/manager/modules/restapi/model/src/main/java/types/GlusterHook.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterMemoryPool.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterServerHook.java A backend/manager/modules/restapi/model/src/main/java/types/GlusterVolume.java C backend/manager/modules/restapi/model/src/main/java/types/GlusterVolumeProfileDetails.java C backend/manager/modules/restapi/model/src/main/java/types/GraphicsConsole.java A backend/manager/modules/restapi/model/src/main/java/types/Group.java C backend/manager/modules/restapi/model/src/main/java/types/GuestOperatingSystem.java C backend/manager/modules/restapi/model/src/main/java/types/HardwareInformation.java C backend/manager/modules/restapi/model/src/main/java/types/HighAvailability.java C backend/manager/modules/restapi/model/src/main/java/types/Hook.java A backend/manager/modules/restapi/model/src/main/java/types/Host.java A backend/manager/modules/restapi/model/src/main/java/types/HostDevice.java C backend/manager/modules/restapi/model/src/main/java/types/HostDevicePassthrough.java A backend/manager/modules/restapi/model/src/main/java/types/HostNic.java A backend/manager/modules/restapi/model/src/main/java/types/HostStorage.java C backend/manager/modules/restapi/model/src/main/java/types/HostedEngine.java C backend/manager/modules/restapi/model/src/main/java/types/Icon.java C backend/manager/modules/restapi/model/src/main/java/types/Image.java A backend/manager/modules/restapi/model/src/main/java/types/Initialization.java C backend/manager/modules/restapi/model/src/main/java/types/InstanceType.java C backend/manager/modules/restapi/model/src/main/java/types/Io.java C backend/manager/modules/restapi/model/src/main/java/types/Ip.java C backend/manager/modules/restapi/model/src/main/java/types/IpAddressAssignment.java C backend/manager/modules/restapi/model/src/main/java/types/IscsiBond.java A backend/manager/modules/restapi/model/src/main/java/types/IscsiDetails.java C backend/manager/modules/restapi/model/src/main/java/types/Job.java C backend/manager/modules/restapi/model/src/main/java/types/KatelloErratum.java C backend/manager/modules/restapi/model/src/main/java/types/Kernel.java C backend/manager/modules/restapi/model/src/main/java/types/Ksm.java C backend/manager/modules/restapi/model/src/main/java/types/Label.java A backend/manager/modules/restapi/model/src/main/java/types/LogicalUnit.java C backend/manager/modules/restapi/model/src/main/java/types/Mac.java C backend/manager/modules/restapi/model/src/main/java/types/MacPool.java C backend/manager/modules/restapi/model/src/main/java/types/MemoryOverCommit.java C backend/manager/modules/restapi/model/src/main/java/types/MemoryPolicy.java C backend/manager/modules/restapi/model/src/main/java/types/Method.java C backend/manager/modules/restapi/model/src/main/java/types/MigrationOptions.java A backend/manager/modules/restapi/model/src/main/java/types/Network.java A backend/manager/modules/restapi/model/src/main/java/types/NetworkAttachment.java C backend/manager/modules/restapi/model/src/main/java/types/NetworkConfiguration.java C backend/manager/modules/restapi/model/src/main/java/types/NfsProfileDetail.java A backend/manager/modules/restapi/model/src/main/java/types/Nic.java C backend/manager/modules/restapi/model/src/main/java/types/NicConfiguration.java C backend/manager/modules/restapi/model/src/main/java/types/NumaNode.java C backend/manager/modules/restapi/model/src/main/java/types/NumaNodePin.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackImage.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackImageProvider.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackNetwork.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackNetworkProvider.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackProvider.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackSubnet.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackVolumeProvider.java C backend/manager/modules/restapi/model/src/main/java/types/OpenStackVolumeType.java C backend/manager/modules/restapi/model/src/main/java/types/OpenstackVolumeAuthenticationKey.java C backend/manager/modules/restapi/model/src/main/java/types/OperatingSystem.java C backend/manager/modules/restapi/model/src/main/java/types/OperatingSystemInfo.java C backend/manager/modules/restapi/model/src/main/java/types/Option.java C backend/manager/modules/restapi/model/src/main/java/types/Package.java C backend/manager/modules/restapi/model/src/main/java/types/Payload.java A backend/manager/modules/restapi/model/src/main/java/types/Permission.java C backend/manager/modules/restapi/model/src/main/java/types/Permit.java C backend/manager/modules/restapi/model/src/main/java/types/PmProxy.java C backend/manager/modules/restapi/model/src/main/java/types/PortMirroring.java A backend/manager/modules/restapi/model/src/main/java/types/PowerManagement.java C backend/manager/modules/restapi/model/src/main/java/types/Product.java C backend/manager/modules/restapi/model/src/main/java/types/ProfileDetail.java C backend/manager/modules/restapi/model/src/main/java/types/Property.java C backend/manager/modules/restapi/model/src/main/java/types/ProxyTicket.java A backend/manager/modules/restapi/model/src/main/java/types/Qos.java C backend/manager/modules/restapi/model/src/main/java/types/Quota.java C backend/manager/modules/restapi/model/src/main/java/types/QuotaClusterLimit.java C backend/manager/modules/restapi/model/src/main/java/types/QuotaStorageLimit.java C backend/manager/modules/restapi/model/src/main/java/types/Range.java C backend/manager/modules/restapi/model/src/main/java/types/Rate.java C backend/manager/modules/restapi/model/src/main/java/types/ReportedConfiguration.java C backend/manager/modules/restapi/model/src/main/java/types/ReportedDevice.java C backend/manager/modules/restapi/model/src/main/java/types/RngDevice.java C backend/manager/modules/restapi/model/src/main/java/types/Role.java C backend/manager/modules/restapi/model/src/main/java/types/SchedulingPolicy.java C backend/manager/modules/restapi/model/src/main/java/types/SchedulingPolicyUnit.java C backend/manager/modules/restapi/model/src/main/java/types/SeLinux.java C backend/manager/modules/restapi/model/src/main/java/types/SerialNumber.java C backend/manager/modules/restapi/model/src/main/java/types/Session.java C backend/manager/modules/restapi/model/src/main/java/types/SkipIfConnectivityBroken.java C backend/manager/modules/restapi/model/src/main/java/types/SkipIfSdActive.java C backend/manager/modules/restapi/model/src/main/java/types/Snapshot.java C backend/manager/modules/restapi/model/src/main/java/types/Spm.java C backend/manager/modules/restapi/model/src/main/java/types/Ssh.java C backend/manager/modules/restapi/model/src/main/java/types/SshPublicKey.java C backend/manager/modules/restapi/model/src/main/java/types/Sso.java A backend/manager/modules/restapi/model/src/main/java/types/Statistic.java C backend/manager/modules/restapi/model/src/main/java/types/StatisticKind.java C backend/manager/modules/restapi/model/src/main/java/types/StatisticUnit.java C backend/manager/modules/restapi/model/src/main/java/types/Status.java C backend/manager/modules/restapi/model/src/main/java/types/Step.java A backend/manager/modules/restapi/model/src/main/java/types/StorageConnection.java C backend/manager/modules/restapi/model/src/main/java/types/StorageConnectionExtension.java A backend/manager/modules/restapi/model/src/main/java/types/StorageDomain.java C backend/manager/modules/restapi/model/src/main/java/types/Tag.java C backend/manager/modules/restapi/model/src/main/java/types/Template.java C backend/manager/modules/restapi/model/src/main/java/types/TemplateVersion.java C backend/manager/modules/restapi/model/src/main/java/types/Ticket.java C backend/manager/modules/restapi/model/src/main/java/types/TimeZone.java C backend/manager/modules/restapi/model/src/main/java/types/TransparentHugePages.java C backend/manager/modules/restapi/model/src/main/java/types/UnmanagedNetwork.java C backend/manager/modules/restapi/model/src/main/java/types/Usb.java A backend/manager/modules/restapi/model/src/main/java/types/User.java C backend/manager/modules/restapi/model/src/main/java/types/Value.java C backend/manager/modules/restapi/model/src/main/java/types/ValueType.java C backend/manager/modules/restapi/model/src/main/java/types/VcpuPin.java R backend/manager/modules/restapi/model/src/main/java/types/Vendor.java C backend/manager/modules/restapi/model/src/main/java/types/Version.java C backend/manager/modules/restapi/model/src/main/java/types/VirtioScsi.java C backend/manager/modules/restapi/model/src/main/java/types/VirtualNumaNode.java C backend/manager/modules/restapi/model/src/main/java/types/Vlan.java A backend/manager/modules/restapi/model/src/main/java/types/Vm.java A backend/manager/modules/restapi/model/src/main/java/types/VmBase.java C backend/manager/modules/restapi/model/src/main/java/types/VmPlacementPolicy.java A backend/manager/modules/restapi/model/src/main/java/types/VmPool.java C backend/manager/modules/restapi/model/src/main/java/types/VmSummary.java C backend/manager/modules/restapi/model/src/main/java/types/VnicProfile.java C backend/manager/modules/restapi/model/src/main/java/types/VolumeGroup.java C backend/manager/modules/restapi/model/src/main/java/types/Watchdog.java C backend/manager/modules/restapi/model/src/main/java/types/Weight.java 178 files changed, 2,925 insertions(+), 5,124 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46478 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I739d7fa1c07eea37858f12704ed1a7cf1d18266c Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: restapi: Populate "placement_policy.hosts" even with only on...
by piotr.kliczewski＠gmail.com 30 Oct '15

30 Oct '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Populate "placement_policy.hosts" even with only one host ...................................................................... restapi: Populate "placement_policy.hosts" even with only one host Currently the "placement_policy.hosts" element is populated only if the VM is pinned to 2 or more hosts. When it is pinned to only one host the element isn't populated, only "placement_policy.host" is populated. This patch changes the RESTAPI so that the multiple hosts will be always populated, even if there is only one host. Change-Id: I1819a1a2a35ca216071e225c319fb4423d7c3629 Bug-Url: https://bugzilla.redhat.com/1275733 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Dudi Maroshi: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47793 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1819a1a2a35ca216071e225c319fb4423d7c3629 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Use Java 8 as source and target
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: core: Use Java 8 as source and target ...................................................................... core: Use Java 8 as source and target This patch changes the POMs so that the new Java 8 capabilities can be used in all the sub-projects excepts those whose source is used by the GWT compiler, as it doesn't support Java 8. The patch also fixes some minor issues in the code that the Java 8 compiler doesn't accept, as it is more strict in some areas, specially generics. Change-Id: I0c78347cab79e5d79e096424df94720b4c342edf Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/aaa/AAAServiceImpl.java M backend/manager/modules/common/pom.xml M backend/manager/modules/compat/pom.xml M backend/manager/modules/restapi/metamodel/pom.xml M backend/manager/modules/restapi/model/pom.xml M backend/manager/modules/searchbackend/pom.xml M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/log/LoggedUtilsTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M build-tools-root/ovirt-findbugs-filters/pom.xml M frontend/webadmin/modules/pom.xml M pom.xml 11 files changed, 205 insertions(+), 29 deletions(-) Approvals: Juan Hernandez: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Moti Asayag: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46288 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c78347cab79e5d79e096424df94720b4c342edf Gerrit-PatchSet: 15 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: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine-extension-logger-log4j: drop el6 on master
by dcaroest＠redhat.com 30 Oct '15

30 Oct '15

David Caro has submitted this change and it was merged. Change subject: ovirt-engine-extension-logger-log4j: drop el6 on master ...................................................................... ovirt-engine-extension-logger-log4j: drop el6 on master Change-Id: I52a932310f3102ffb6a40bc8e85e6631661a2a82 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-extension-logger-log4j/ovirt-engine-extension-logger-log4j_create-rpms.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml 2 files changed, 11 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for review Anton Marchukov: Looks good to me Jenkins CI: Passed CI tests David Caro: Ready for merge -- To view, visit https://gerrit.ovirt.org/47886 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52a932310f3102ffb6a40bc8e85e6631661a2a82 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: Making slave cleanup more robust
by dcaroest＠redhat.com 30 Oct '15

30 Oct '15

David Caro has submitted this change and it was merged. Change subject: Making slave cleanup more robust ...................................................................... Making slave cleanup more robust When a slave is in this state when is connected but not responding, some of the methods there throw exception, that made the job fail and not cleanup any hosts, now it will continue with the rest of hosts and show a stack trace Change-Id: I527439b57cb9a28711a39508eaf6eeea33014f54 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/groovy-scripts/cleanup_workspaces.groovy 1 file changed, 17 insertions(+), 16 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/47884 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I527439b57cb9a28711a39508eaf6eeea33014f54 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: packaging: setup: avoid asking about mgmt bridge nic on addi...
by sbonazzo＠redhat.com 30 Oct '15

30 Oct '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: avoid asking about mgmt bridge nic on additional hosts ...................................................................... packaging: setup: avoid asking about mgmt bridge nic on additional hosts On additional host we are not directly creating the management bridge. We simply call host.add via REST API and the engine will do the rest via host-deploy. So avoid asking about the interface to create the management bridge on cause we are not honoring it. The engine will choose it exactly as it does for regular non HE hosts. Bug-Url: https://bugzilla.redhat.com/1269430 Change-Id: Ibcb56997b17fe04dbccf25fa2a3bd9c79e5da8e9 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47505 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibcb56997b17fe04dbccf25fa2a3bd9c79e5da8e9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Remove explicit class names from XML schema
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Remove explicit class names from XML schema ...................................................................... restapi: Remove explicit class names from XML schema Currently the XML schema uses JAXB annotations to explicitly specify the names of some (only a few) generated Java classes. This means that the names will always be the specified ones, even when using JAXB bindings to add prefixes or suffixes. This patch removes these annotations. Change-Id: Icf515ccd1f9e3fe61d313310d5647bf2cbf002fb Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd 1 file changed, 0 insertions(+), 55 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/47852 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icf515ccd1f9e3fe61d313310d5647bf2cbf002fb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: publish-rpms-nightly-3.6: use 3.6 build of aaa-misc
by sbonazzo＠redhat.com 30 Oct '15

30 Oct '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: publish-rpms-nightly-3.6: use 3.6 build of aaa-misc ...................................................................... publish-rpms-nightly-3.6: use 3.6 build of aaa-misc Change-Id: I321bda90e7c99ab4f9d1f539a5039de7fb741fcf Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Anton Marchukov: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47870 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I321bda90e7c99ab4f9d1f539a5039de7fb741fcf 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: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0