Bug 1043524 - Restart libvirtd every deploy
Summary: Restart libvirtd every deploy
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: ovirt-host-deploy
Classification: oVirt
Component: Plugins.VDSM
Version: 1.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.1.3
Assignee: Alon Bar-Lev
QA Contact: Tareq Alayan
URL:
Whiteboard: infra
Depends On:
Blocks: 3.3rc1
TreeView+ depends on / blocked
 
Reported: 2013-12-16 14:42 UTC by Alon Bar-Lev
Modified: 2016-02-10 19:32 UTC (History)
11 users (show)

Fixed In Version: is28
Doc Type: Enhancement
Doc Text:
libvirtd is now restarted after each host-deploy process, in order to pick up VDSM and pki configuration changes.
Clone Of:
Environment:
Last Closed: 2014-01-21 15:57:16 UTC
oVirt Team: Infra
Embargoed:
pm-rhel: blocker+
alonbl: devel_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2014:0074 0 normal SHIPPED_LIVE ovirt-host-deploy bug fix and enhancement update 2014-01-21 20:15:57 UTC
oVirt gerrit 22435 0 None None None Never
oVirt gerrit 22478 0 None None None Never

Description Alon Bar-Lev 2013-12-16 14:42:19 UTC
Due to vdsm changes and reports that at end of host-deploy process libvirt is still running and unconfigured, it will be easiest to restart libvirt every host-deploy.

Comment 1 Alon Bar-Lev 2013-12-16 14:43:00 UTC
Barak,
Do you see potential issue in doing so?
Thanks,

Comment 2 Yaniv Bronhaim 2013-12-16 23:17:24 UTC
what reports? can you be more specific? In ovirt-3.3 we changed the code to restart libvirt after each successful configure, and in master we should handle it right unless there is an issue that i'm not familiar with

Comment 3 Alon Bar-Lev 2013-12-16 23:20:56 UTC
(In reply to Yaniv Bronhaim from comment #2)
> what reports? can you be more specific? In ovirt-3.3 we changed the code to
> restart libvirt after each successful configure, and in master we should
> handle it right unless there is an issue that i'm not familiar with

The pki configuration was not applied in all cases, as the recent vdsm considered it already configured.

Comment 4 Yaniv Bronhaim 2013-12-16 23:43:08 UTC
the ca,cert and key files? is this being set again by the host-deploy or only by libvirt configure with the defaults? if host deploy overwrites the defaults it must restart libvirt manually (isconfigured shouldn't recognize it), but if its the defaults configure, libvirt service should be restarted after the configure unless there is a bug

Comment 5 Alon Bar-Lev 2013-12-16 23:45:26 UTC
(In reply to Yaniv Bronhaim from comment #4)
> the ca,cert and key files? is this being set again by the host-deploy or
> only by libvirt configure with the defaults? if host deploy overwrites the
> defaults it must restart libvirt manually (isconfigured shouldn't recognize
> it), but if its the defaults configure, libvirt service should be restarted
> after the configure unless there is a bug

Once again... the --force should have handled anything... now that you removed this we need to take care of it at host-deploy. The opened question in this bug is simple: what are the implications [if any].

Comment 7 Tareq Alayan 2013-12-29 15:48:07 UTC
verified on ovirt-host-deploy-1.1.3-1.el6ev.noarch

from log:
========
2013-12-29 17:37:30 DEBUG otopi.context context._executeMethod:123 Stage closeup METHOD otopi.plugins.ovirt_host_deploy.vdsm.packages.Plugin._start
2013-12-29 17:37:30 INFO otopi.plugins.ovirt_host_deploy.vdsm.packages packages._start:173 Stopping libvirtd
2013-12-29 17:37:30 DEBUG otopi.plugins.otopi.services.rhel rhel.exists:121 check if service libvirtd exists
2013-12-29 17:37:30 DEBUG otopi.plugins.otopi.services.rhel plugin.executeRaw:366 execute: ('/sbin/initctl', 'status', 'libvirtd'), executable='None', cwd='None', env=None
2013-12-29 17:37:30 DEBUG otopi.plugins.otopi.services.rhel plugin.executeRaw:383 execute-result: ('/sbin/initctl', 'status', 'libvirtd'), rc=0
2013-12-29 17:37:30 DEBUG otopi.plugins.otopi.services.rhel plugin.execute:441 execute-output: ('/sbin/initctl', 'status', 'libvirtd') stdout:
libvirtd start/running, process 2323

Comment 8 errata-xmlrpc 2014-01-21 15:57:16 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2014-0074.html


Note You need to log in before you can comment on or make changes to this bug.