- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: core: If command is created to compensate existing business_...
by tnisan＠redhat.com 03 Jul '16

03 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: If command is created to compensate existing business_entity_snapshot records, it should be created with compensation enabled. ...................................................................... core: If command is created to compensate existing business_entity_snapshot records, it should be created with compensation enabled. It does not make sense trying to compensate without compensation enabled. Change-Id: Ic06ae5ff9b6d1e6e25f78b8f28a228dc7210b6a1 Bug-Url: https://bugzilla.redhat.com/1350399 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/59530 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic06ae5ff9b6d1e6e25f78b8f28a228dc7210b6a1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: fixing email address
by eedri＠redhat.com 02 Jul '16

02 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: fixing email address ...................................................................... fixing email address Change-Id: If8461280bfb65f5c69ef3c99f5908a2fba69c580 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt/system-tests.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/60115 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8461280bfb65f5c69ef3c99f5908a2fba69c580 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: Adds missing jackson-core dependency for hystrix RPMs
by eedri＠redhat.com 01 Jul '16

01 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: Adds missing jackson-core dependency for hystrix RPMs ...................................................................... Adds missing jackson-core dependency for hystrix RPMs Adds jackson-core to enabled EPEL packages, becuase it's required by hystrix-metrics-event-stream. This is needed only because of RPM dependency, because during runtime hystrix will use jackson-core provided by WildFly. Change-Id: Iac1e24b97ff68fa90c46be28d0474fd713d64b3a Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-el7-deps.repo.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Eyal Edri: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/60087 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac1e24b97ff68fa90c46be28d0474fd713d64b3a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Require spec_helper from .rspec
by juan.hernandez＠redhat.com 01 Jul '16

01 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Require spec_helper from .rspec ...................................................................... Require spec_helper from .rspec This patch moves the requirement of the spec helper from the Ruby source files to the ".rspec" file. Change-Id: Idb6f02d65844dce49c7c3a1a5e93eff6f61d5041 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 404d9cf8c9c8e6a62a637350eb19052cff36be63) --- A sdk/.rspec M sdk/spec/action_reader_spec.rb M sdk/spec/clusters_service_spec.rb M sdk/spec/connection_create_spec.rb M sdk/spec/connection_timeout_spec.rb M sdk/spec/connection_use_spec.rb M sdk/spec/data_centers_service_spec.rb M sdk/spec/enum_spec.rb M sdk/spec/fault_reader_spec.rb M sdk/spec/reader_spec.rb M sdk/spec/service_spec.rb M sdk/spec/spec_helper.rb M sdk/spec/storage_domain_service_spec.rb M sdk/spec/storage_domains_service_spec.rb M sdk/spec/vm_reader_spec.rb M sdk/spec/vm_service_spec.rb M sdk/spec/vm_spec.rb M sdk/spec/vm_writer_spec.rb M sdk/spec/vms_service_spec.rb M sdk/spec/writer_spec.rb M sdk/spec/xml_reader_spec.rb M sdk/spec/xml_writer_spec.rb 22 files changed, 12 insertions(+), 50 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60095 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb6f02d65844dce49c7c3a1a5e93eff6f61d5041 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine-sdk-ruby[master]: Require spec_helper from .rspec
by juan.hernandez＠redhat.com 01 Jul '16

01 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Require spec_helper from .rspec ...................................................................... Require spec_helper from .rspec This patch moves the requirement of the spec helper from the Ruby source files to the ".rspec" file. Change-Id: Idb6f02d65844dce49c7c3a1a5e93eff6f61d5041 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/.rspec M sdk/spec/action_reader_spec.rb M sdk/spec/clusters_service_spec.rb M sdk/spec/connection_create_spec.rb M sdk/spec/connection_timeout_spec.rb M sdk/spec/connection_use_spec.rb M sdk/spec/data_centers_service_spec.rb M sdk/spec/enum_spec.rb M sdk/spec/fault_reader_spec.rb M sdk/spec/reader_spec.rb M sdk/spec/service_spec.rb M sdk/spec/spec_helper.rb M sdk/spec/storage_domain_service_spec.rb M sdk/spec/storage_domains_service_spec.rb M sdk/spec/vm_reader_spec.rb M sdk/spec/vm_service_spec.rb M sdk/spec/vm_spec.rb M sdk/spec/vm_writer_spec.rb M sdk/spec/vms_service_spec.rb M sdk/spec/writer_spec.rb M sdk/spec/xml_reader_spec.rb M sdk/spec/xml_writer_spec.rb 22 files changed, 12 insertions(+), 50 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60093 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb6f02d65844dce49c7c3a1a5e93eff6f61d5041 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby 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>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Tooltip for cpuInfo
by piotr.kliczewski＠gmail.com 01 Jul '16

01 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Tooltip for cpuInfo ...................................................................... webadmin: Tooltip for cpuInfo VM Snapshot subtab shows tooltip explaining format of the displayed Number of CPU Cores field. Similar tooltip fixed for Pool General subtab. Change-Id: Iaf3eba82d57672c2820370a19f602a90daae6b7b Bug-Url: https://bugzilla.redhat.com/1296127 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmImportGeneralModelForm.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmSnapshotInfoGeneral.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationMessages.properties M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/pool/SubTabPoolGeneralView.java 4 files changed, 22 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/60081 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf3eba82d57672c2820370a19f602a90daae6b7b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: engine: Fix var substitution in error message
by piotr.kliczewski＠gmail.com 01 Jul '16

01 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine: Fix var substitution in error message ...................................................................... engine: Fix var substitution in error message Change-Id: Ifa4f05e2cb9e4da62aa0cbacef7b2cd89353dbd4 Bug-Url: https://bugzilla.redhat.com/1255590 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/RefreshGeoRepSessionsCommand.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 3 files changed, 5 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Sahina Bose: Verified -- To view, visit https://gerrit.ovirt.org/60084 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa4f05e2cb9e4da62aa0cbacef7b2cd89353dbd4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[sdk_4.0]: Release 4.0.0.Alpha11
by omachace＠redhat.com 01 Jul '16

01 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Release 4.0.0.Alpha11 ...................................................................... Release 4.0.0.Alpha11 Change-Id: I22d45c2638b8efb5fdfe4f44cd9c7ac77e36e4d9 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M automation/build-artifacts.sh M packaging/build.sh M sdk/pom.xml 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60086 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I22d45c2638b8efb5fdfe4f44cd9c7ac77e36e4d9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Fix var substitution in error message
by sabose＠redhat.com 01 Jul '16

01 Jul '16

Sahina Bose has submitted this change and it was merged. Change subject: engine: Fix var substitution in error message ...................................................................... engine: Fix var substitution in error message Change-Id: Ifa4f05e2cb9e4da62aa0cbacef7b2cd89353dbd4 Bug-Url: https://bugzilla.redhat.com/1255590 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/RefreshGeoRepSessionsCommand.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 3 files changed, 5 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Sahina Bose: Verified -- To view, visit https://gerrit.ovirt.org/56965 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa4f05e2cb9e4da62aa0cbacef7b2cd89353dbd4 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: fix UI exceptions in volumes tab
by awels＠redhat.com 01 Jul '16

01 Jul '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: fix UI exceptions in volumes tab ...................................................................... webadmin: fix UI exceptions in volumes tab This patch fixes the UI exceptions in following places 1. Exceptions are flooded in the UI, with user hower over the the empty Activities column in Gluster Volumes tab and Bricks Subtab. There is a NPE in VolumeActivityCompositeCell. getContainerElement() method. Fixed this issue by showing the composite cell always. When there is not activity in the volume then we will add a table with empty row. 2. There is an exception in UI, when there is no suitable cluster for gep rep slave. We should not try to set the selection when the list itself is empty. Change-Id: If1fcfb584dd250ec7550df98eeb4dfe7c442fc5a Bug-Url: https://bugzilla.redhat.com/1303878 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterVolumeGeoRepCreateModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabVolumeView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/gluster/SubTabVolumeBrickView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/VolumeActivityCompositeCell.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/VolumeActivitySeperatorCell.java 5 files changed, 24 insertions(+), 29 deletions(-) Approvals: Alexander Wels: Looks good to me, approved anmolbabu: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Sahina Bose: Looks good to me, but someone else must approve Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/54574 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1fcfb584dd250ec7550df98eeb4dfe7c442fc5a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0