| Summary: | disable libvirtd sysv, use libvirtd upstart job | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Linux 6 | Reporter: | Alan Pevec <apevec> |
| Component: | ovirt-node | Assignee: | Alan Pevec <apevec> |
| Status: | CLOSED ERRATA | QA Contact: | Virtualization Bugs <virt-bugs> |
| Severity: | medium | Docs Contact: | |
| Priority: | high | ||
| Version: | 6.2 | CC: | abaron, bazulay, cshao, danken, dnaori, ewarszaw, gouyang, hateya, iheim, leiwang, mgoldboi, moli, ovirt-maint, smehrotr, ycui, ykaul |
| Target Milestone: | rc | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | ovirt-node-2.0.2-0.5.git83340dd.el6 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 694026 | Environment: | |
| Last Closed: | 2011-12-06 19:21:16 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | 678084, 694026 | ||
| Bug Blocks: | |||
|
Description
Alan Pevec
2011-07-27 10:20:47 UTC
I thought that you would not allow vdsm-specific configuration such as this before registration to rhev-m. If I knew that you would, we could have done this configuration on vdsm post script... libvirtd upstart job is useful in general, nothing vdsm specific about it, but since vdsmd was using sysv initscript, we couldn't use it before. How would your post script look like? libvirt refused to make upstart their default in RHEL-6; I guess their reasoning could apply to rhevm-less rhevh.
Anyway, I guess the post script could be not very different from what we currently do on vdsmd startup. Hmmm, but we cannot use it in RHEL where libvirt version (and the location of the upstart script) may change in time.
start_libvirtd() {
local packaged installed target
local startout
packaged=`/bin/rpm -ql libvirt | /bin/grep libvirtd.upstart | \
/usr/bin/tail -1`
target=/etc/init/libvirtd.conf
installed=`/bin/readlink -f "$target"`
if [[ "$packaged" != "$installed" ]];
then
/bin/ln -sf "$packaged" "$target" || return 1
/sbin/initctl reload-configuration
fi
startout=`/sbin/initctl start libvirtd 2>&1`
if [[ "$?" -eq 0 || "$startout" =~ .*already\ running.* ]];
then
return 0
else
echo "$startout" >&2
return 1
fi
}
(In reply to comment #3) > libvirt refused to make upstart their default in RHEL-6; I guess their > reasoning could apply to rhevm-less rhevh. Their reasoning is fine for RHEL-6 in general (it would be unexpected change for the users mid-release) but rhevh is a blackbox (also 6.x wasn't public yet) and I'm confident in using libvirtd upstart job there. Actually in RHEV-H we must configure upstart job during image creation, vdsmd code in comment 3 will create dangling symlink on the running RHEV-H: /etc/init/libvirtd.conf -> /usr/share/doc/libvirt-0.9.2/libvirtd.upstart Files marked %doc are not installed in the RHEV-H image (%packages --excludedocs) *** Bug 726323 has been marked as a duplicate of this bug. *** > start_needed_srv && start_libvirtd
Reopening, the above line should've been patched in rhevh kickstart.
According to comment 16, verified this bug on rhev-hypervisor 6.2-0.14, have removed /etc/rc.d/init.d/libvirtd, and updated /etc/rc.d/init.d/vdsmd with libvirt things. # grep -i start_libvirt /etc/init.d/vdsmd start_libvirtd() { 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-2011-1783.html |