Bug 498302

Summary: /usr/share/smolt/client/os_detect.py:103: DeprecationWarning:
Product: [Fedora] Fedora Reporter: Jóhann B. Guðmundsson <johannbg>
Component: smoltAssignee: Mike McGrath <mmcgrath>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 11CC: jeff, mike.cloaked, mmcgrath
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.3-1.fc11 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-06-22 20:26:55 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 Jóhann B. Guðmundsson 2009-04-29 19:33:00 UTC
Description of problem:

/usr/share/smolt/client/os_detect.py:103: DeprecationWarning: os.popen2 is deprecated.  Use the subprocess module.
  child_stdin, child_stdout = os.popen2(command)


Version-Release number of selected component (if applicable):

smolt-1.2-4.2.fc11.noarch

How reproducible:

Always

Steps to Reproduce:
1.SmoltSendProfile
2.
3.
  
Actual results:

The above

Expected results:

No deprecation warning.. 

Additional info:

Minor nuance does not stop to eat babies..

Comment 1 Bug Zapper 2009-06-09 14:48:55 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 11 development cycle.
Changing version to '11'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 2 Mike McGrath 2009-06-22 20:26:55 UTC
Fixed upstream, will be released with the version we're shipping in a week or two.

Comment 3 Mike C 2009-08-25 21:21:24 UTC
Has this update been abandoned?  It is two months since comment #2 !

Comment 4 Fedora Update System 2009-08-26 14:55:37 UTC
smolt-1.3-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/smolt-1.3-1.fc11

Comment 5 Mike C 2009-08-26 20:56:13 UTC
Closing this bug was perhaps premature.
Trying the new build smolt-1.3-1.fc11
[mike@samsung2 ~]$ smoltSendProfile 
Unable to save UUID, continuing...
Your UUID file could not be created: [Errno 13] Permission denied: '/etc/smolt/hw-uuid'

The file referenced does not exist in the /etc/smolt directory.

Comment 6 Mike McGrath 2009-08-26 21:05:36 UTC
... that has nothing to do with the bug you reported.  I'll fix it in a bit though.

Comment 7 Mike McGrath 2009-08-26 21:16:26 UTC
Oh wait, this is from you not running smolt as root.  It doesn't have permission to write there.

Comment 8 Mike C 2009-08-27 07:11:56 UTC
But I have never had to run smolt as root before F11?  Has policy changed?

Comment 9 Mike C 2009-08-27 07:14:18 UTC
In F10 I can run smoltSendProfile as a normal user and there is no hw-uuid created in /etc/smolt/

Comment 10 Mike C 2009-08-27 09:04:01 UTC
Confirmed that the new version does work as root but not as user - which is more secure and probably better than previous versions.

Comment 11 Mike C 2009-08-27 13:59:27 UTC
The comments I added above refer to running on a Samsung NC10 with smolt data
http://www.smolts.org/client/show/pub_bf13b7f1-dd7f-4f20-a181-8ec47e7a6818

However strangely I tried the original smolt running in F11 on a Dell Dimension 2400 which allowed me to run smoltSendProfile as a normal user though with the originally reported bug symptoms. However it did generate the profile correctly and produced it in 
http://www.smolts.org/client/show/pub_eddb6cf9-b933-4c1b-9eb2-23cb0d984056

Then I updated from smolt-1.2-4.2.fc11 to smolt-1.3-1.fc11
Then it gives the same behaviour as in comments #9 and #10

So it seems that the need to run as root is new behaviour - which I do endorse!

Comment 12 Mike McGrath 2009-08-27 14:12:39 UTC
You can still run smoltSendProfile as a non-root user.  The error you are seeing is a non-fatal error.  It just indicates that normal users can't write to /etc/ because they can't :)

Comment 13 Fedora Update System 2009-08-27 22:56:02 UTC
smolt-1.3-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.