- Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[ovirt-3.5]: hook: Start vdsmd explicitly again
by fabiand＠redhat.com 07 Nov '14

07 Nov '14

Fabian Deutsch has submitted this change and it was merged. Change subject: hook: Start vdsmd explicitly again ...................................................................... hook: Start vdsmd explicitly again It's not really clear, but … If a dependency is brought down during a service is queued, then the start job is canceled. And even if the depdendency comes up again, the unit is not enqueued again. Change-Id: I4796c96a8cd97d19beeafabc6c2a52b3817b9d12 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156369 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M hooks/Makefile.am A hooks/on-boot/90-start-vdsm M ovirt-node-plugin-vdsm.spec.in 4 files changed, 27 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34950 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4796c96a8cd97d19beeafabc6c2a52b3817b9d12 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[ovirt-3.5]: page: Use package name everywhere
by fabiand＠redhat.com 07 Nov '14

07 Nov '14

Fabian Deutsch has submitted this change and it was merged. Change subject: page: Use package name everywhere ...................................................................... page: Use package name everywhere Change-Id: Ifee0293d0b7521e254a619e76f1a0255d65e4418 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1152428 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M src/engine_page.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34949 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifee0293d0b7521e254a619e76f1a0255d65e4418 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[ovirt-3.5]: hooks: Move sanlock upgrade hook to post-upgrade
by fabiand＠redhat.com 07 Nov '14

07 Nov '14

Fabian Deutsch has submitted this change and it was merged. Change subject: hooks: Move sanlock upgrade hook to post-upgrade ...................................................................... hooks: Move sanlock upgrade hook to post-upgrade Previously the sanlock upgrade loogic was run on boot, and that broken vdsm's boot on el7/systemd based systems. Now the logic is run post-upgrade and not at boot time. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156369 Change-Id: I53d63c393e884901eb45f1ac38e8bf2795dbaeb7 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M hooks/Makefile.am R hooks/on-boot/02-vdsm-sebool-config R hooks/post-upgrade/01-sanlock-check M ovirt-node-plugin-vdsm.spec.in 4 files changed, 11 insertions(+), 4 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34948 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53d63c393e884901eb45f1ac38e8bf2795dbaeb7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: hook: Start vdsmd explicitly again
by fabiand＠redhat.com 07 Nov '14

07 Nov '14

Fabian Deutsch has submitted this change and it was merged. Change subject: hook: Start vdsmd explicitly again ...................................................................... hook: Start vdsmd explicitly again It's not really clear, but … If a dependency is brought down during a service is queued, then the start job is canceled. And even if the depdendency comes up again, the unit is not enqueued again. Change-Id: I4796c96a8cd97d19beeafabc6c2a52b3817b9d12 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156369 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M hooks/Makefile.am A hooks/on-boot/90-start-vdsm M ovirt-node-plugin-vdsm.spec.in 4 files changed, 27 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34947 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4796c96a8cd97d19beeafabc6c2a52b3817b9d12 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: page: Use package name everywhere
by fabiand＠redhat.com 07 Nov '14

07 Nov '14

Fabian Deutsch has submitted this change and it was merged. Change subject: page: Use package name everywhere ...................................................................... page: Use package name everywhere Change-Id: Ifee0293d0b7521e254a619e76f1a0255d65e4418 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1152428 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M src/engine_page.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34930 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifee0293d0b7521e254a619e76f1a0255d65e4418 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: hooks: Move sanlock upgrade hook to post-upgrade
by fabiand＠redhat.com 07 Nov '14

07 Nov '14

Fabian Deutsch has submitted this change and it was merged. Change subject: hooks: Move sanlock upgrade hook to post-upgrade ...................................................................... hooks: Move sanlock upgrade hook to post-upgrade Previously the sanlock upgrade loogic was run on boot, and that broken vdsm's boot on el7/systemd based systems. Now the logic is run post-upgrade and not at boot time. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156369 Change-Id: I53d63c393e884901eb45f1ac38e8bf2795dbaeb7 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M hooks/Makefile.am R hooks/on-boot/02-vdsm-sebool-config R hooks/post-upgrade/01-sanlock-check M ovirt-node-plugin-vdsm.spec.in 5 files changed, 12 insertions(+), 5 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34842 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53d63c393e884901eb45f1ac38e8bf2795dbaeb7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Avoid NPE in request for CDROMs of wrong VM
by juan.hernandez＠redhat.com 07 Nov '14

07 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Avoid NPE in request for CDROMs of wrong VM ...................................................................... restapi: Avoid NPE in request for CDROMs of wrong VM Currently when we receive a request for the collection of CDROMs of a VM we don't take into account that the query performed to find the VM can return null if there isn't such a VM (the identifier is wrong, or the VM has been removed). To avoid that NPE this patch checks the result of the query and will respond with an empty list in those cases. Change-Id: Id17ef36193c21e113d4abafeee0c769a2a948bd3 Bug-Url: https://bugzilla.redhat.com/1150953 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/AbstractBackendReadOnlyDevicesResource.java 1 file changed, 8 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34748 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id17ef36193c21e113d4abafeee0c769a2a948bd3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: jsonrpc: version bump
by sbonazzo＠redhat.com 07 Nov '14

07 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump Bug-Url: https://bugzilla.redhat.com/1154398 Change-Id: Ib69768ab2b384136308e16624a365c98583fba63 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34878 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib69768ab2b384136308e16624a365c98583fba63 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in mom[master]: build: Fix mom.spec.in changelog
by alitke＠redhat.com 07 Nov '14

07 Nov '14

Adam Litke has submitted this change and it was merged. Change subject: build: Fix mom.spec.in changelog ...................................................................... build: Fix mom.spec.in changelog During the rewrite of the mom build process (707326d4) an old bug in our spec file crept back in. The 12th of January in 2011 was on a Wednesday. Change-Id: Ic50c5f6ff5c9841cc22c3d5be5e118e2d65f5806 Signed-off-by: Adam Litke <alitke(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/34937 Reviewed-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M mom.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Adam Litke: Verified; Looks good to me, approved Sandro Bonazzola: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34937 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic50c5f6ff5c9841cc22c3d5be5e118e2d65f5806 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging: spec: add support for EL7 builds
by piotr.kliczewski＠gmail.com 07 Nov '14

07 Nov '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: spec: add support for EL7 builds ...................................................................... packaging: spec: add support for EL7 builds Change-Id: Id6fcef7e48cc1f616b09b4c3cb42b9bb92a8259f Bug-Url: https://bugzilla.redhat.com/1159293 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 20 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34875 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6fcef7e48cc1f616b09b4c3cb42b9bb92a8259f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0