May 2016 - Engine-commits - oVirt List Archives

Change in ovirt-release[ovirt-3.6]: node: Hack around myhostname issue
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Hack around myhostname issue ...................................................................... node: Hack around myhostname issue Change-Id: Ia94aa2b6d267cd42a7658ab94b0630a3936c8047 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release36.spec.in 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57572 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia94aa2b6d267cd42a7658ab94b0630a3936c8047 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> 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]: node: Raise visibility of hacks
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Raise visibility of hacks ...................................................................... node: Raise visibility of hacks Change-Id: I5940cb0989bbbd77de2bfee574d1b9abd8fd8235 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57634 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5940cb0989bbbd77de2bfee574d1b9abd8fd8235 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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]: node: Hack requirement to include NM config server
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Hack requirement to include NM config server ...................................................................... node: Hack requirement to include NM config server This will be required by vdsm in 4.0, but not in 3.6. Change-Id: I61747aac16d74d3e7ec4c8a5aae0c8f335242d33 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57633 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61747aac16d74d3e7ec4c8a5aae0c8f335242d33 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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-3.6]: Add EFI dependencies for NGN
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add EFI dependencies for NGN ...................................................................... Add EFI dependencies for NGN We're pulling these out of the kickstart, to keep all dependencies in the release file.. Change-Id: I929497a33cb7747bd33d934aa1bc9fc4377044df Signed-off-by: Ryan Barry <rbarry(a)redhat.com> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release36.spec.in 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57639 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I929497a33cb7747bd33d934aa1bc9fc4377044df Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> 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]: node: Hack to allow to keep NM running
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Hack to allow to keep NM running ...................................................................... node: Hack to allow to keep NM running Change-Id: Ia284cf0487f03568a805e7b455ea9cef869bcc6d Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 15 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57632 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia284cf0487f03568a805e7b455ea9cef869bcc6d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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]: node: Hack around myhostname issue
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Hack around myhostname issue ...................................................................... node: Hack around myhostname issue Change-Id: Ia94aa2b6d267cd42a7658ab94b0630a3936c8047 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57631 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia94aa2b6d267cd42a7658ab94b0630a3936c8047 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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]: Add EFI dependencies for NGN
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add EFI dependencies for NGN ...................................................................... Add EFI dependencies for NGN We're pulling these out of the kickstart, to keep all dependencies in the release file.. Change-Id: I929497a33cb7747bd33d934aa1bc9fc4377044df Signed-off-by: Ryan Barry <rbarry(a)redhat.com> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57630 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I929497a33cb7747bd33d934aa1bc9fc4377044df Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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[master]: core: Change permissionCheckSubject for inherited command.
by mlipchuk＠redhat.com 18 May '16

18 May '16

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Change permissionCheckSubject for inherited command. ...................................................................... core: Change permissionCheckSubject for inherited command. Based on the commit Ida67a4409509652d5eeb1a953c3537051e4e61bb, StorageDomainCommandBase implements the getPermissionCheckSubjects with a new ArrayList and avoid using Singleton list. Based on that AttachStorageDomainToPoolCommand can avoid using two lists and use only the one which was created at StorageDomainCommandBase. Change-Id: Ia5c9e1b068272d2c876a43437368cccfd8437816 Signed-off-by: Maor Lipchuk <maorlipchuk(a)gmail.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/AttachStorageDomainToPoolCommand.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Maor Lipchuk: Verified; Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57608 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5c9e1b068272d2c876a43437368cccfd8437816 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Change singletonList to new Array for StorageCommandBase.
by mlipchuk＠redhat.com 18 May '16

18 May '16

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Change singletonList to new Array for StorageCommandBase. ...................................................................... core: Change singletonList to new Array for StorageCommandBase. Each command inherited from StorageDomainCommandBase, which does not implement the procedure getPermissionCheckSubjects, will risk an exception of UnsupportedOperationException once a new permission will be added to it. An example of a failure can be found when using enabling quota in a Data Center. Hence, changing permissionCheckSubjects singleton collection to use a new array list instead. Change-Id: Ida67a4409509652d5eeb1a953c3537051e4e61bb Signed-off-by: Maor Lipchuk <maorlipchuk(a)gmail.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/StorageDomainCommandBase.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Maor Lipchuk: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57584 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ida67a4409509652d5eeb1a953c3537051e4e61bb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-3.6]: node: Add mailx requirement
by sbonazzo＠redhat.com 18 May '16

18 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add mailx requirement ...................................................................... node: Add mailx requirement Adding mailx to be able to read emails from daemon/agents. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1335503 Change-Id: I952d56574af11ea4748c9d4c33c704b095bbd08e Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-release36.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57637 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I952d56574af11ea4748c9d4c33c704b095bbd08e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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