Bug 694403
Summary: | should use systemctl instead of chkconfig --list | |||
---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Rahul Sundaram <metherid> | |
Component: | libvirt | Assignee: | Eric Blake <eblake> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | |
Severity: | unspecified | Docs Contact: | ||
Priority: | unspecified | |||
Version: | 15 | CC: | aquini, berrange, clalance, crobinso, eblake, itamar, jforbes, laine, notting, sven, veillard, virt-maint | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 761329 (view as bug list) | Environment: | ||
Last Closed: | 2012-05-24 10:03:42 UTC | Type: | --- | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 761329 |
Description
Rahul Sundaram
2011-04-07 08:51:26 UTC
Really, it should just do 'chkconfig --level $level <service>' to check if a service is enabled. The scriptlet as is today will already fail if you're running in a non-english locale, for example. This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component. This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component. This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component. This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component. This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component. This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component. bug 738725 tracks the change to silence the update; however, it missed out on your observation in comment 1 that non-C locale may break the behavior of the snippet. Patch for comment 1 proposed: https://www.redhat.com/archives/libvir-list/2011-December/msg00296.html This should be backported to F15, and cloned to F16. commit fea83dde7b7e0725e4353a2cc5b6e5c7c61cb294 Author: Eric Blake <eblake> Date: Tue Dec 6 16:16:34 2011 -0700 spec: don't use chkconfig --list https://bugzilla.redhat.com/show_bug.cgi?id=694403 reports that the specfile is incorrectly checking for a running libvirt-guests service. For example, $ LC_ALL=es_ES chkconfig --list libvirt-guests libvirt-guests 0:desactivado 1:desactivado 2:desactivado 3:activo 5:activo 6:desactivado will fail to find 5:on, even though it is active. But chkconfig already has a mode where you can silently use the exit status to check for an active service. * libvirt.spec.in (%post): Use simpler chkconfig options, to avoid issues with localization. This is fixed for F16+ - as F15 is nearing EOL and libvirt hasn't been updated in 10 Months I guess it's save to close this bug now. |