Bug 716970

Summary: Provide native systemd unit file
Product: [Fedora] Fedora Reporter: Jóhann B. Guðmundsson <johannbg>
Component: apmdAssignee: Jiri Skala <jskala>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: aglotov, jskala
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-07-18 07:11:42 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 713562    
Attachments:
Description Flags
Native systemd service file apmd none

Description Jóhann B. Guðmundsson 2011-06-27 11:37:12 EDT
Description of problem:

https://fedoraproject.org/wiki/Features/SysVtoSystemd

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


How reproducible:


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


Expected results:


Additional info:
Comment 1 Jóhann B. Guðmundsson 2011-07-11 11:48:29 EDT
Created attachment 512249 [details]
Native systemd service file apmd

What's the current status on this?

I've submitted totally untested service file based on the init script and note that faik modules should be loaded via autoloading based on
bus information, or via /etc/modules-load.d/*.conf. and unloading a module from
the kernel should not be done except for debugging purposes. 

Thus you should propably rop a conf file into /etc/modules-load.d for the module

If you dont have the time to package this make note of that on this bug and I
can see if I can find a proven packager you can package this for you...

https://fedoraproject.org/wiki/Packaging:Guidelines:Systemd
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Systemd
https://fedoraproject.org/wiki/Packaging:Tmpfiles.d
Comment 2 Jóhann B. Guðmundsson 2011-07-15 10:52:29 EDT
Ping you component is part of the Base group hence we need to get some move on this..
Comment 3 Jiri Skala 2011-07-18 03:56:33 EDT
Hi,
APM is quite historical. So I was checking for a support of this service firstly.

Well, I'm searching for hw to test otherwise I'd use your attached unit file. But as you mentioned loading kernel module should be separated into /etc/module-load.d/. This would be only change against attached file. Any comments?
Comment 4 Jóhann B. Guðmundsson 2011-07-18 04:10:46 EDT
Nope not really you would just create a file called apm.conf containing something like..

# Load apm.ko at boot
apm

And remove the line

ExecStartPre=-/sbin/modprobe -qab apm