- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: storage: calling getStorageDomainStats also on FC
by msivak＠redhat.com 18 Dec '15

18 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: storage: calling getStorageDomainStats also on FC ...................................................................... storage: calling getStorageDomainStats also on FC getStorageDomainStats has the side effect of causing a Storage Domain refresh including all its tree under /rhev/data-center/... We need it also for FC! Change-Id: I193a63be1a9de4ef70f5947931e4b631e89fc6db Bug-Url: https://bugzilla.redhat.com/1258465 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_server.py 1 file changed, 6 insertions(+), 5 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/50696 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I193a63be1a9de4ef70f5947931e4b631e89fc6db Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: upgrade: fix upgrade from partially upgraded condition
by msivak＠redhat.com 18 Dec '15

18 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: upgrade: fix upgrade from partially upgraded condition ...................................................................... upgrade: fix upgrade from partially upgraded condition The upgrade procedure has to move the hosted-engine configuration to the shared storage and it has to disconnect the hosted-engine storage domain from its bootstrap storage pool. The two tasks can be performed by different attempts. _move_to_shared_conf was connecting the bootstrap storage pool leaving it connected for further steps. _remove_storage_pool was assuming that it was already connected. But if the configuration volume is OK it skips _move_to_shared_conf and, if the system has been rebooted in the middle, _remove_storage_pool fails cause the storage pool is not connected. Connecting the storage pool if needed also from _remove_storage_pool Change-Id: Ibc5782d2f245129b2bb2f7e70f2558d820eb4267 Bug-Url: https://bugzilla.redhat.com/1292652 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/upgrade.py 1 file changed, 21 insertions(+), 4 deletions(-) Approvals: Martin Sivák: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/50700 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc5782d2f245129b2bb2f7e70f2558d820eb4267 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: Remove Sys V support and fix the systemd unit files
by msivak＠redhat.com 18 Dec '15

18 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: Remove Sys V support and fix the systemd unit files ...................................................................... Remove Sys V support and fix the systemd unit files Change-Id: I9ba6ecce31a9216935f3c67d8a76a99177e525f4 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M initscripts/Makefile.am D initscripts/ovirt-ha-agent.init.in R initscripts/ovirt-ha-agent.service.in D initscripts/ovirt-ha-broker.init.in D initscripts/ovirt-ha-broker.service A initscripts/ovirt-ha-broker.service.in M ovirt-hosted-engine-ha.spec.in 7 files changed, 19 insertions(+), 307 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50714 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9ba6ecce31a9216935f3c67d8a76a99177e525f4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: storage: calling getStorageDomainStats also on FC
by msivak＠redhat.com 18 Dec '15

18 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: storage: calling getStorageDomainStats also on FC ...................................................................... storage: calling getStorageDomainStats also on FC getStorageDomainStats has the side effect of causing a Storage Domain refresh including all its tree under /rhev/data-center/... We need it also for FC! Change-Id: I193a63be1a9de4ef70f5947931e4b631e89fc6db Bug-Url: https://bugzilla.redhat.com/1258465 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_server.py 1 file changed, 6 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Nir Soffer: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/50695 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I193a63be1a9de4ef70f5947931e4b631e89fc6db Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: upgrade: fix upgrade from partially upgraded condition
by msivak＠redhat.com 18 Dec '15

18 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: upgrade: fix upgrade from partially upgraded condition ...................................................................... upgrade: fix upgrade from partially upgraded condition The upgrade procedure has to move the hosted-engine configuration to the shared storage and it has to disconnect the hosted-engine storage domain from its bootstrap storage pool. The two tasks can be performed by different attempts. _move_to_shared_conf was connecting the bootstrap storage pool leaving it connected for further steps. _remove_storage_pool was assuming that it was already connected. But if the configuration volume is OK it skips _move_to_shared_conf and, if the system has been rebooted in the middle, _remove_storage_pool fails cause the storage pool is not connected. Connecting the storage pool if needed also from _remove_storage_pool Change-Id: Ibc5782d2f245129b2bb2f7e70f2558d820eb4267 Bug-Url: https://bugzilla.redhat.com/1292652 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/upgrade.py 1 file changed, 21 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/50699 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc5782d2f245129b2bb2f7e70f2558d820eb4267 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: Remove Sys V support and fix the systemd unit files
by msivak＠redhat.com 18 Dec '15

18 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: Remove Sys V support and fix the systemd unit files ...................................................................... Remove Sys V support and fix the systemd unit files Change-Id: I9ba6ecce31a9216935f3c67d8a76a99177e525f4 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M initscripts/Makefile.am D initscripts/ovirt-ha-agent.init.in R initscripts/ovirt-ha-agent.service.in D initscripts/ovirt-ha-broker.init.in D initscripts/ovirt-ha-broker.service A initscripts/ovirt-ha-broker.service.in M ovirt-hosted-engine-ha.spec.in 7 files changed, 19 insertions(+), 307 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50711 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9ba6ecce31a9216935f3c67d8a76a99177e525f4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Using a nicer groovy summary for the html report
by sbonazzo＠redhat.com 18 Dec '15

18 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Using a nicer groovy summary for the html report ...................................................................... Using a nicer groovy summary for the html report The plugin is uglier than expected, this embeds an iframe only if the index.html file exists Change-Id: I645ff65f7c41b822c5dbaff9ec4c4dbb8fefb2f2 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/publishers/html-reports.yaml 1 file changed, 12 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified -- To view, visit https://gerrit.ovirt.org/50686 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I645ff65f7c41b822c5dbaff9ec4c4dbb8fefb2f2 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: post oVirt Live 3.6.1
by sbonazzo＠redhat.com 18 Dec '15

18 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: post oVirt Live 3.6.1 ...................................................................... post oVirt Live 3.6.1 Change-Id: I001682c37e39bfb8b3bf1e2fee20447560ee26ce Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/centos-7-livecd.cfg M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 3 files changed, 8 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50725 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I001682c37e39bfb8b3bf1e2fee20447560ee26ce Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: master: add rubygem-libxml-ruby
by sbonazzo＠redhat.com 18 Dec '15

18 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: master: add rubygem-libxml-ruby ...................................................................... master: add rubygem-libxml-ruby Change-Id: Ife6d3f9669f9d7184c5db4cb28188075036b56bf Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml M jobs/confs/projects/rubygem-libxml-ruby/rubygem-libxml-ruby.yaml 2 files changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50723 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife6d3f9669f9d7184c5db4cb28188075036b56bf Gerrit-PatchSet: 1 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-engine[ovirt-engine-3.6]: packaging: spec: fix missing shadow-tools deps
by piotr.kliczewski＠gmail.com 18 Dec '15

18 Dec '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: spec: fix missing shadow-tools deps ...................................................................... packaging: spec: fix missing shadow-tools deps Added missing shadow-tools dep to packages creating users. Bug-Url: https://bugzilla.redhat.com/1292009 Change-Id: I40a422eea7164229657fa6e83f60d38872a99a1a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50712 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40a422eea7164229657fa6e83f60d38872a99a1a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: gerrit-hooks <automation(a)ovirt.org>

1 0