Change in jenkins[master]: jobs: Add yaml job for ovirt-register
by tlitovsk＠redhat.com
Anatoly Litovsky has submitted this change and it was merged.
Change subject: jobs: Add yaml job for ovirt-register
......................................................................
jobs: Add yaml job for ovirt-register
Change-Id: Ifcc0c5018373671e0574cf8d8d605a69124b9d72
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com>
---
A jobs/confs/yaml/jobs/ovirt-register/ovirt-register.yaml
A jobs/confs/yaml/scms/ovirt-register-scms.yaml
2 files changed, 35 insertions(+), 0 deletions(-)
Approvals:
Douglas Schilling Landgraf: Verified
Jenkins CI: Passed CI tests
Anatoly Litovsky: Looks good to me; Ready for merge
--
To view, visit https://gerrit.ovirt.org/43291
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifcc0c5018373671e0574cf8d8d605a69124b9d72
Gerrit-PatchSet: 5
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
9 years, 5 months
Change in ovirt-engine[master]: restapi: Add 'placeholder' attribute to VmHostDevice resources
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Add 'placeholder' attribute to VmHostDevice resources
......................................................................
restapi: Add 'placeholder' attribute to VmHostDevice resources
So far the backend only returned the devices that are to
be attached to the VM. Now it returns all host devices
configured for the VM and distinguishes IOMMU placeholder
devices with additional boolean flag.
This patch exposes to the REST users that some host device
is configured on the VM only as a placeholder using the
optional <placeholder>true</placeholder> element.
Change-Id: I76e9945cb680537a83b036b6f87cf5aeb33fa420
Signed-off-by: Martin Betak <mbetak(a)redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDeviceResource.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDevicesResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendHostDevicesResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostDeviceResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostDevicesResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDeviceResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDevicesResourceTest.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostDeviceMapper.java
9 files changed, 59 insertions(+), 19 deletions(-)
Approvals:
Juan Hernandez: Looks good to me, approved
Jenkins CI: Passed CI tests
Martin Betak: Verified
--
To view, visit https://gerrit.ovirt.org/43316
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I76e9945cb680537a83b036b6f87cf5aeb33fa420
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: core: null check in CommandBase.parentHasCallback()
by ofrenkel＠redhat.com
Omer Frenkel has submitted this change and it was merged.
Change subject: core: null check in CommandBase.parentHasCallback()
......................................................................
core: null check in CommandBase.parentHasCallback()
Fixed NullPointerException thrown in CommandBase.parentHasCallback(),
when parentCommand is set, but parentParameters are not set in the
command's parameters (for example, AddVmPoolWithVmsCommand called from
CommonVmPoolWithVmsCommand).
Change-Id: I2e24a042cb990e62367a160627657b95bf2921e3
Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Shmuel Leib Melamud: Verified
Omer Frenkel: Looks good to me, but someone else must approve
Liron Aravot: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/43251
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2e24a042cb990e62367a160627657b95bf2921e3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in jenkins[master]: build: drop fc20 builds from master
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: build: drop fc20 builds from master
......................................................................
build: drop fc20 builds from master
Dropped fc20 builds from all projects.
For reference: http://lists.ovirt.org/pipermail/users/2015-June/033454.html
Change-Id: Ieb4711e2a7c207b592a48ba132b1f809957da48e
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M jobs/confs/yaml/builders/copy-create-job-artifact.yaml
M jobs/confs/yaml/jobs/ebay-cors-filter/ebay-cors-filter.yaml
M jobs/confs/yaml/jobs/ioprocess/ioprocess_create-rpms.yaml
M jobs/confs/yaml/jobs/jasper-reports/jasper-reports_create-rpms.yaml
M jobs/confs/yaml/jobs/mom/mom_create-rpms.yaml
M jobs/confs/yaml/jobs/otopi/otopi_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-dwh/ovirt-dwh_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-engine-extension-aaa-jdbc/ovirt-engine-extension-aaa-jdbc_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-engine-extension-aaa-ldap/ovirt-engine-extension-aaa-ldap_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-engine-extension-aaa-misc/ovirt-engine-extension-aaa-misc_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-engine-extension-logger-log4j/ovirt-engine-extension-logger-log4j_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-engine-sdk/ovirt-engine-sdk_build-artifacts.yaml
M jobs/confs/yaml/jobs/ovirt-engine-wildfly-overlay/ovirt-engine-wildfly-overlay.yaml
M jobs/confs/yaml/jobs/ovirt-engine-wildfly/ovirt-engine-wildfly.yaml
M jobs/confs/yaml/jobs/ovirt-host-deploy/ovirt-host-deploy_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-image-uploader/ovirt-image-uploader_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-log-collector/ovirt-log-collector_create-rpms.yaml
M jobs/confs/yaml/jobs/ovirt-reports/ovirt-reports_create-rpms.yaml
M jobs/confs/yaml/jobs/repoman/repoman_build-artifacts.yaml
M jobs/confs/yaml/jobs/unboundid-ldapsdk/unboundid-ldapsdk_create-rpms.yaml
M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml
23 files changed, 12 insertions(+), 23 deletions(-)
Approvals:
Fabian Deutsch: Looks good to me
Piotr Kliczewski: Looks good to me
Sandro Bonazzola: Verified; Ready for merge
Martin Peřina: Looks good to me
Juan Hernández: Looks good to me
Eyal Edri: Looks good to me
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/43304
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieb4711e2a7c207b592a48ba132b1f809957da48e
Gerrit-PatchSet: 4
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Juan Hernández
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
9 years, 5 months
Change in ovirt-engine[master]: core: Invoke FullList command using the resource manager
by rgolan＠redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: core: Invoke FullList command using the resource manager
......................................................................
core: Invoke FullList command using the resource manager
Direct constructor usage of commands is a violation of the API
Bug-Url: https://bugzilla.redhat.com/1160094
Change-Id: I3c9c2314a1c0d5dd307b6401739c3b052dd3ec68
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeStatusCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
R backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/FullListVDSCommand.java
4 files changed, 15 insertions(+), 12 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Roy Golan: Verified; Passed CI tests
Omer Frenkel: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/42293
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3c9c2314a1c0d5dd307b6401739c3b052dd3ec68
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: core: refactor live merge code to handle async child command
by amureini＠redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: core: refactor live merge code to handle async child command
......................................................................
core: refactor live merge code to handle async child command
In the spirit of DRY, combine two blocks of code that were largely the
same.
Change-Id: Ife74a1787c2671fbdb2d2c4dc99e38b088f69dfe
Signed-off-by: Greg Padgett <gpadgett(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskLiveCommand.java
1 file changed, 26 insertions(+), 27 deletions(-)
Approvals:
Greg Padgett: Verified
Jenkins CI: Passed CI tests
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/43364
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ife74a1787c2671fbdb2d2c4dc99e38b088f69dfe
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: core: handle Live Merge when top volume is larger than base
by amureini＠redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: core: handle Live Merge when top volume is larger than base
......................................................................
core: handle Live Merge when top volume is larger than base
When a Live Merge is perfomed today, it internally performs a block
commit operation to copy data from a volume to its parent. If the disk
had been extended after the snapshot was created, the sizes of the two
volumes will not match.
In most cases the size is adjusted properly by the underlying processes
performing the merge; to account for this, engine will update the size
of the volume in the database to reflect the new, correct size.
In the case of a raw volume on block storage, an explicit extension
needs to take place before the merge. For this, a new child command of
Live Merge has been added to run before the merge starts.
For further detail, see comments to:
https://bugzilla.redhat.com/show_bug.cgi?id=1232481
Change-Id: I628323b799a37691c40ea97590d4967f3ebf4a2b
Bug-Url: https://bugzilla.redhat.com/1236758
Signed-off-by: Greg Padgett <gpadgett(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExtendImageSizeCommand.java
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeExtendCommand.java
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeExtendCommandCallback.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskLiveCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ExtendImageSizeParameters.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RemoveSnapshotSingleDiskLiveStep.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
7 files changed, 212 insertions(+), 3 deletions(-)
Approvals:
Greg Padgett: Verified
Jenkins CI: Passed CI tests
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/43025
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I628323b799a37691c40ea97590d4967f3ebf4a2b
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: core: dont update devices if vm is not updated
by ofrenkel＠redhat.com
Omer Frenkel has submitted this change and it was merged.
Change subject: core: dont update devices if vm is not updated
......................................................................
core: dont update devices if vm is not updated
vms monitoring should skip updating devices of vm that is not marked for
update. if devices-hash changes, the vm should be in the vmDynamicToSave
list, because the analyzer will identify the hash is different.
this also ensures that the VmManager lock is taken for this vm and its
data was not invalidated by other command.
Change-Id: Idc60abd6ba072fe202f1cd50de2879800a4a0ee8
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1230082
Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
1 file changed, 10 insertions(+), 4 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Omer Frenkel: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/43344
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idc60abd6ba072fe202f1cd50de2879800a4a0ee8
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: restapi: Don't require Resteasy JAXB provider
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Don't require Resteasy JAXB provider
......................................................................
restapi: Don't require Resteasy JAXB provider
The RESTAPI has its own custom JAXB provider, so it doesn't need to
depend on the Resteasy one during build or runtime. This patch removes
those dependencies.
Change-Id: Idc8a9eded5263ad625530a163b0621f8d2f9e1df
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M backend/manager/modules/restapi/jaxrs/pom.xml
M backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml
2 files changed, 0 insertions(+), 7 deletions(-)
Approvals:
Ori Liel: Looks good to me, approved
Juan Hernandez: Verified; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/43196
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idc8a9eded5263ad625530a163b0621f8d2f9e1df
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: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: restapi: Don't use internal Resteasy exceptions
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Don't use internal Resteasy exceptions
......................................................................
restapi: Don't use internal Resteasy exceptions
This patch replaces the use of the internal Resteasy "NotFoundException"
class with a standard JAX-RS alternative.
Change-Id: Ib30361129bef91323b94739dea0d44317ae8742f
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQuotaClusterLimitResource.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQuotaStorageLimitResource.java
2 files changed, 8 insertions(+), 4 deletions(-)
Approvals:
Ori Liel: Looks good to me, approved
Juan Hernandez: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/43186
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib30361129bef91323b94739dea0d44317ae8742f
Gerrit-PatchSet: 4
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
9 years, 5 months