Bug 606763

Summary:	Old smolt configuration file still listed
Product:	[Fedora] Fedora	Reporter:	Michel Lind <michel>
Component:	smolt	Assignee:	Mike McGrath <mmcgrath>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	13	CC:	mike.fleetwood, mmcgrath
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-09-19 16:57:05 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:

Description Michel Lind 2010-06-22 12:41:02 UTC

Description of problem:
The smolt RPM owns (as a ghost file) %{_sysconfigdir}/sysconfig/hw-uuid -- but this file is never created. From looking at

/usr/share/smolt/client/sendProfile.py

it is evident that the actual file used is now /etc/smolt/hw-uuid.pub

Version-Release number of selected component (if applicable):
smolt-1.4.2.2-1.fc13.noarch

How reproducible:
Always

Steps to Reproduce:
1. Run smoltSendProfile as root
2. ls /etc/sysconfig/hw-uuid
3. rpm -ql smolt | grep etc
  
Actual results:
#2 not found, but listed in #3

Expected results:
/etc/sysconfig/hw-uuid should not be claimed by the smolt package anymore.

Additional info:

Comment 1 Mike Fleetwood 2010-09-19 11:07:53 UTC

Suggested fix to RPM spec file.

--- smolt.spec.orig	2010-01-28 17:43:40.000000000 +0000
+++ smolt.spec	2010-09-19 12:02:11.777100447 +0100
@@ -189,7 +189,7 @@
 %{_sysconfdir}/cron.d/%{name}
 %{_mandir}/man1/*gz
 %{_initrddir}/%{name}
-%ghost %config(noreplace) %{_sysconfdir}/sysconfig/hw-uuid
+%ghost %config(noreplace) %{_sysconfdir}/%{name}/hw-uuid
 
 %files server
 %defattr(-,root,root,-)

Comment 2 Mike McGrath 2010-09-19 16:57:05 UTC

Thanks, I've just committed this to the repo but I don't think a rebuild/update is warranted enough just for this, but it'll go out the next time we build and push.  I'm hoping to have a bulk of other changes fairly soon.