- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: report VMs with external MAC on VM import
by masayag＠redhat.com 12 Jul '16

12 Jul '16

Moti Asayag has submitted this change and it was merged. Change subject: engine: report VMs with external MAC on VM import ...................................................................... engine: report VMs with external MAC on VM import Change-Id: Id90802b81018194523fccdffe0eff5b764e902a2 Bug-Url: https://bugzilla.redhat.com/1209795 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/ExternalVmMacsFinder.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/exportimport/ImportVMFromConfigurationCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/vm/ExternalVmMacsFinderTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 6 files changed, 203 insertions(+), 21 deletions(-) Approvals: Martin Mucha: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/59794 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id90802b81018194523fccdffe0eff5b764e902a2 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: jpackage: move to another mirror
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: jpackage: move to another mirror ...................................................................... jpackage: move to another mirror jpackage mirror on heanet is not available anymore. Moving to another mirror. Change-Id: I76d0948a2841d2074760baf4bf952deb2de292ef Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-extension-aaa-jdbc/ovirt-engine-extension-aaa-jdbc_create-rpms.yaml M jobs/packaging/repo_closure_check.sh 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60603 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76d0948a2841d2074760baf4bf952deb2de292ef 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: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_3.6]: sdk: Add .spec file for Fedora 24
by juan.hernandez＠redhat.com 12 Jul '16

12 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add .spec file for Fedora 24 ...................................................................... sdk: Add .spec file for Fedora 24 This patch adds a .spec file template for Fedora 24. Change-Id: I5ee48700e6ad34107cb1d918955415cd3dfad865 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/spec.fc24.in 1 file changed, 411 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/60606 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ee48700e6ad34107cb1d918955415cd3dfad865 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Yamlized find-bugs jobs
by eedri＠redhat.com 12 Jul '16

12 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: Yamlized find-bugs jobs ...................................................................... Yamlized find-bugs jobs Change-Id: If04e630f89fee3982f676d785dd7e8d3d99ad7ed Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A jobs/confs/groovy-scripts/findbugs-postbuild.groovy A jobs/confs/projects/ovirt-engine/ovirt-engine_findbugs.yaml A jobs/confs/shell-scripts/findbugs_git_info.sh A jobs/confs/shell-scripts/findbugs_mvn.sh A jobs/confs/yaml/builders/find-bugs_builders.yaml A jobs/confs/yaml/publishers/findbugs-publishers.yaml M jobs/confs/yaml/scms/jenkins.yaml A jobs/confs/yaml/templates/find-bugs.yaml 8 files changed, 275 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59176 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If04e630f89fee3982f676d785dd7e8d3d99ad7ed Gerrit-PatchSet: 15 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: Revert "automation: drop suffix for release"
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "automation: drop suffix for release" ...................................................................... Revert "automation: drop suffix for release" This reverts commit 1204c1cb511df5e73b74ad74da27a9ef6f23a7e8. Change-Id: I9d4b35ca72d5fc840666683c797afe025c0aff22 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60598 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d4b35ca72d5fc840666683c797afe025c0aff22 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: evict event from cache when manually removed
by tnisan＠redhat.com 12 Jul '16

12 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: evict event from cache when manually removed ...................................................................... core: evict event from cache when manually removed When a event or alert is removed manually from the display in UI, we should also evict the corresponding cache entry for events that have 6he flood mechanism. This patch evict the corresponding event from the cache when it is manually deleted by the user. This solves the following problem : Event E1 with flood time of T1 is raised by the system The issue is fixed by the application administrator Event E1 is raised again within the T1 period The bug was that the second event was suppressed by the flooding mechanism since the cache still holds the first event. In order to solve this BZ there was a need to add code to the RemoveAuditLogByIdCommand to evict the event from the cache, this is done by composing the cache key from the command using the information from the event which is the candidate for the delete operation and generating a this key with the sysadmin user. Change-Id: I0e302aa0bb218f7986cac57b13a90f689d956e19 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1219147 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveAuditLogByIdCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/TimeoutBase.java 3 files changed, 59 insertions(+), 9 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Eli Mesika: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60591 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e302aa0bb218f7986cac57b13a90f689d956e19 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: automation: drop suffix for release
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: drop suffix for release ...................................................................... automation: drop suffix for release Change-Id: Ie8051540367ac3472f097b7c1d4cc4e979d43b9a --- M automation/build-artifacts.sh 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8051540367ac3472f097b7c1d4cc4e979d43b9a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: build: post ovirt-host-deploy-1.5.1
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-host-deploy-1.5.1 ...................................................................... build: post ovirt-host-deploy-1.5.1 Change-Id: Ic51dca1d29efe7aa397705a3172f37ba25712412 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M configure.ac M po/pt_BR.po 4 files changed, 9 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60593 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic51dca1d29efe7aa397705a3172f37ba25712412 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: build: ovirt-host-deploy-1.5.1
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-host-deploy-1.5.1 ...................................................................... build: ovirt-host-deploy-1.5.1 Change-Id: Ic8241484bcb73aa28d30bb395e369071448c2c90 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ChangeLog M configure.ac M ovirt-host-deploy.spec.in M po/pt_BR.po 4 files changed, 14 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60592 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8241484bcb73aa28d30bb395e369071448c2c90 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: build: ovirt-release40-4.0.1
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release40-4.0.1 ...................................................................... build: ovirt-release40-4.0.1 Change-Id: I0830b5052c90135646339a708dfd698273a90c61 --- M configure.ac M ovirt-release40.spec.in 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60594 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0830b5052c90135646339a708dfd698273a90c61 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0