January 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: Set format version for ISO and Export in storage p...
by Code Review 26 Jan '17

26 Jan '17

>From Maor Lipchuk <mlipchuk(a)redhat.com>: Maor Lipchuk has submitted this change and it was merged. Change subject: webadmin: Set format version for ISO and Export in storage popup. ...................................................................... webadmin: Set format version for ISO and Export in storage popup. Set the format version for ISO and Export storage domain so manage domain will succeed to update the storage domain. Change-Id: Ic733acd5eb94f387adba548753c0067c313d09c1 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/71148 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic733acd5eb94f387adba548753c0067c313d09c1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-4.1]: build: ovirt-live-4.1.0_rc3
by Code Review 26 Jan '17

26 Jan '17

>From Lev Veyde <lveyde(a)redhat.com>: Lev Veyde has submitted this change and it was merged. Change subject: build: ovirt-live-4.1.0_rc3 ...................................................................... build: ovirt-live-4.1.0_rc3 Change-Id: I4ac040f9321357e9216ccf0363458b3e6fa21941 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Lev Veyde: Verified; Looks good to me, approved; Passed CI tests Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/71237 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ac040f9321357e9216ccf0363458b3e6fa21941 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-4.1 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-engine: add 4.1.0 branch
by Code Review 26 Jan '17

26 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: add 4.1.0 branch ...................................................................... ovirt-engine: add 4.1.0 branch Change-Id: I1803a9dc0481acadd98b68ea901ae9af3dfbdde1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/71175 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1803a9dc0481acadd98b68ea901ae9af3dfbdde1 Gerrit-PatchSet: 3 Gerrit-Project: jenkins 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-hosted-engine-setup[ovirt-hosted-engine-setup-2.1]: cloud-init: add a comment to explain how to avoid 1372260
by Code Review 26 Jan '17

26 Jan '17

>From Simone Tiraboschi <stirabos(a)redhat.com>: Simone Tiraboschi has submitted this change and it was merged. Change subject: cloud-init: add a comment to explain how to avoid 1372260 ...................................................................... cloud-init: add a comment to explain how to avoid 1372260 Due to a kernel bug (BZ#1372260), 'systemctl restart sshd' could be blocked forverer and this will block also our cloud-init script and so the deployment till hosted-engine-setup will abort at a timeout. Restart sshd in a later section (from bootcmd to runcmd) to try avoiding BZ#1372260 and, if this is still not enough, start it in background to avoid getting stuck also if sshd is not available. Change-Id: I4f3380c02ed1b95391a2fca31d82a43c5408f228 Bug-Url: https://bugzilla.redhat.com/1416023 Bug-Url: https://bugzilla.redhat.com/1409203 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> (cherry picked from commit af56f3e62d8eff236f53919c3c19b9e86866b527) --- M src/plugins/gr-he-common/vm/cloud_init.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/71232 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f3380c02ed1b95391a2fca31d82a43c5408f228 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.1 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.1]: Build: ovirt-release41 4.1.0-1
by Code Review 26 Jan '17

26 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: Build: ovirt-release41 4.1.0-1 ...................................................................... Build: ovirt-release41 4.1.0-1 Change-Id: Ifefe3e5656215b266f4de192009465e751824aa8 --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/71233 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifefe3e5656215b266f4de192009465e751824aa8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.1 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[master]: webadmin: initialize TabIndexes with 1 by default
by Code Review 26 Jan '17

26 Jan '17

>From Daniel Erez <derez(a)redhat.com>: Daniel Erez has submitted this change and it was merged. Change subject: webadmin: initialize TabIndexes with 1 by default ...................................................................... webadmin: initialize TabIndexes with 1 by default As defined at MDN [1], tabindex should be set to a positive value, as value '0' means that the element should be placed after elements with a positive tabindex. For now, this change fixes tab order in 'New Disk' dialog. I.e. when 'Size' is focused, pressing 'tab' should focus on 'Alias' (and not on search bar, as it did previously). [1] https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/tabindex Change-Id: I91dbedee38d893302f1006c9ce1e58d146ce7cce Bug-Url: https://bugzilla.redhat.com/?????? Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractModelBoundPopupView.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Daniel Erez: Verified Greg Sheremeta: Looks good to me, approved Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/71199 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91dbedee38d893302f1006c9ce1e58d146ce7cce Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: cloud-init: add a comment to explain how to avoid 1372260
by Code Review 26 Jan '17

26 Jan '17

>From Simone Tiraboschi <stirabos(a)redhat.com>: Simone Tiraboschi has submitted this change and it was merged. Change subject: cloud-init: add a comment to explain how to avoid 1372260 ...................................................................... cloud-init: add a comment to explain how to avoid 1372260 Due to a kernel bug (BZ#1372260), 'systemctl restart sshd' could be blocked forverer and this will block also our cloud-init script and so the deployment till hosted-engine-setup will abort at a timeout. Restart sshd in a later section (from bootcmd to runcmd) to try avoiding BZ#1372260 and, if this is still not enough, start it in background to avoid getting stuck also if sshd is not available. Change-Id: I4f3380c02ed1b95391a2fca31d82a43c5408f228 Bug-Url: https://bugzilla.redhat.com/1416023 Bug-Url: https://bugzilla.redhat.com/1409203 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/gr-he-common/vm/cloud_init.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/71230 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f3380c02ed1b95391a2fca31d82a43c5408f228 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: repos: versioned virtio-win-stable
by Code Review 26 Jan '17

26 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: repos: versioned virtio-win-stable ...................................................................... repos: versioned virtio-win-stable Avoid "Repository virtio-win-stable is listed more than once in the configuration" warning when previous releases rpm is installed. Change-Id: I3984f94f9ef4f0323b29d8bcd07a19351a5a1e07 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1415705 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-el7-ppc64le-deps.repo.in M ovirt-el7-x86_64-deps.repo.in M ovirt-f24-deps.repo.in 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/71229 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3984f94f9ef4f0323b29d8bcd07a19351a5a1e07 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release 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.1]: el7: split x86_64 and ppc64le deps
by Code Review 26 Jan '17

26 Jan '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: el7: split x86_64 and ppc64le deps ...................................................................... el7: split x86_64 and ppc64le deps CentOS mirror and SIGs are not using same repos for different arches. Splitted the repo config based on arch. Bump to final release Change-Id: I7dfb6107b794a898aca7bc95ee3be76fa7dc411c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1410413 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 7ee42ea54659c45d345f3a9a54a76c4996cd9a05) --- M Makefile.am M configure.ac A ovirt-el7-ppc64le-deps.repo.in R ovirt-el7-x86_64-deps.repo.in M ovirt-release41.spec.in 5 files changed, 63 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/71227 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7dfb6107b794a898aca7bc95ee3be76fa7dc411c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.1 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-hosted-engine-setup[ovirt-hosted-engine-setup-2.1]: cloud-init: restarting sshd in background
by Code Review 26 Jan '17

26 Jan '17

>From Simone Tiraboschi <stirabos(a)redhat.com>: Simone Tiraboschi has submitted this change and it was merged. Change subject: cloud-init: restarting sshd in background ...................................................................... cloud-init: restarting sshd in background Due to a kernel bug (BZ#1372260), 'systemctl restart sshd' could be blocked forverer and this will block also our cloud-init script and so the deployment till hosted-engine-setup will abort at a timeout. Restart sshd in background and in a later section. Change-Id: I66d4ee1f6a706866e8dab0701a16e18a6cd0503b Bug-Url: https://bugzilla.redhat.com/1416023 Bug-Url: https://bugzilla.redhat.com/1409203 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/gr-he-common/vm/cloud_init.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/71206 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66d4ee1f6a706866e8dab0701a16e18a6cd0503b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.1 Gerrit-Owner: Simone Tiraboschi <stirabos(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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0