Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1312018 - [systemd] Make changes to start opal-prd daemon automatically (fedora-release)
Summary: [systemd] Make changes to start opal-prd daemon automatically (fedora-release)
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: fedora-release
Version: 24
Hardware: ppc64le
OS: Linux
unspecified
high
Target Milestone: ---
Assignee: Stephen Gallagher
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1284527
Blocks: PPCTracker F-ExcludeArch-ppc64le, PPC64LETracker
TreeView+ depends on / blocked
 
Reported: 2016-02-25 14:26 UTC by Vasant Hegde
Modified: 2016-06-16 17:41 UTC (History)
13 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-06-16 17:41:28 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
IBM Linux Technology Center 137829 0 None None None 2019-07-16 10:29:17 UTC

Description Vasant Hegde 2016-02-25 14:26:45 UTC 
Description of problem:
  We have new package opal-prd which contains "OPAL Processor Recovery Diagnostics daemon" (#1284527). This is ppc64le specific package. We want to start daemon automatically.


Please include this daemon in 90-systemd.preset script.


Vasant
Comment 1 IBM Bug Proxy 2016-02-25 16:17:18 UTC 
------- Comment From hannsj_uhl.com 2016-02-25 10:46 EDT-------
.
Comment 2 Jason McDonald 2016-02-26 08:58:36 UTC 
This bug has been incorrectly added to the "Edit Fedora Components" group, which should not be used for bugs.  As it is not clear whether this bug is intended to be public or private, I'm going to add the bug to the Fedora Contrib Private group so that the other group can be removed from this bug and disabled.

If it's ok for the bug to be public, the Fedora Contrib Private group can be cleared.
Comment 3 Vasant Hegde 2016-02-26 09:18:57 UTC 
(In reply to Jason McDonald from comment #2)
> This bug has been incorrectly added to the "Edit Fedora Components" group,
> which should not be used for bugs. 

Sorry. That was my mistake.


> As it is not clear whether this bug is
> intended to be public or private, I'm going to add the bug to the Fedora
> Contrib Private group so that the other group can be removed from this bug
> and disabled.

I'm fine to keep it under Private group.

-Vasant
Comment 4 Jan Synacek 2016-02-26 11:53:58 UTC 
If you want your service to be enabled by default, see https://fedoraproject.org/wiki/Packaging:DefaultServices, section "Approved Exceptions".

Feel free to reopen this bugzilla when you have the necessary exception approved.
Comment 5 Vasant Hegde 2016-03-03 12:47:00 UTC 
I've raised https://fedorahosted.org/fesco/ticket/1558 to get exception approval from FESCO.

-Vasant
Comment 6 Vasant Hegde 2016-03-04 06:34:01 UTC 
As disu(In reply to Vasant Hegde from comment #5)
> I've raised https://fedorahosted.org/fesco/ticket/1558 to get exception
> approval from FESCO.

As suggested by sgallagh in above ticket I'm REOPENING this bug and changing component to fedora-release.

-Vasant
Comment 7 Stephen Gallagher 2016-03-04 14:06:00 UTC 
(In reply to Vasant Hegde from comment #6)
> As disu(In reply to Vasant Hegde from comment #5)
> > I've raised https://fedorahosted.org/fesco/ticket/1558 to get exception
> > approval from FESCO.
> 
> As suggested by sgallagh in above ticket I'm REOPENING this bug and changing
> component to fedora-release.
> 
> -Vasant

Thanks. Can you please specify the exact systemd units that you want enabled by default? e.g. foo.service, bar.socket or baz.timer
Comment 8 Vasant Hegde 2016-03-07 04:40:46 UTC 
(In reply to Stephen Gallagher from comment #7)
> (In reply to Vasant Hegde from comment #6)
> > As disu(In reply to Vasant Hegde from comment #5)
> > > I've raised https://fedorahosted.org/fesco/ticket/1558 to get exception
> > > approval from FESCO.
> > 
> > As suggested by sgallagh in above ticket I'm REOPENING this bug and changing
> > component to fedora-release.
> > 
> > -Vasant

Stephen,

> 
> Thanks. Can you please specify the exact systemd units that you want enabled
> by default? e.g. foo.service, bar.socket or baz.timer

Exact service name is :  opal-prd.service

-Vasant
Comment 9 Zbigniew Jędrzejewski-Szmek 2016-03-07 12:10:51 UTC 
What about the socket unit?
Comment 10 Vasant Hegde 2016-03-07 12:58:51 UTC 
(In reply to Zbigniew Jędrzejewski-Szmek from comment #9)
> What about the socket unit?

opal-prd.socket

-Vasant
Comment 11 Zbigniew Jędrzejewski-Szmek 2016-03-07 14:42:33 UTC 
Sure, I know what the socket is called. My question (implicit, but I thought it would be obvious) is whether you want it enabled by default, and if yes why, and if no, why not.

And if yes, why do you want the service started by default instead of being socket activated.
Comment 12 Vasant Hegde 2016-03-11 05:41:27 UTC 
(In reply to Zbigniew Jędrzejewski-Szmek from comment #11)
> Sure, I know what the socket is called. My question (implicit, but I thought
> it would be obvious) is whether you want it enabled by default, and if yes
> why, and if no, why not.

Yes. We want to enable this by default.

opal-prd daemon handles hardware-specific recovery processes, and should be run as a background system process after boot. It interacts with OPAL firmware to capture hardware error causes, log events to the management processor, and handles recoverable errors where suitable. Without this daemon we will not be able to handle hardware errors (like processor checkstop). Hence we would like to enable this daemon by default on ppc64le architecture. 

> 
> And if yes, why do you want the service started by default instead of being
> socket activated.

No particular reason. Either way works for me as long as daemon starts by default.


-Vasant
Comment 13 Zbigniew Jędrzejewski-Szmek 2016-03-11 12:51:42 UTC 
OK, after looking at the two unit files, .service Requires the .socket unit, so it is enough to only put the .service in presets.
Comment 14 Vasant Hegde 2016-03-14 04:36:24 UTC 
(In reply to Zbigniew Jędrzejewski-Szmek from comment #13)
> OK, after looking at the two unit files, .service Requires the .socket unit,
> so it is enough to only put the .service in presets.

Thanks!

-Vasant
Comment 15 Stephen Gallagher 2016-03-14 12:58:10 UTC 
https://pagure.io/fedora-release/pull-request/38 (F24)
https://pagure.io/fedora-release/pull-request/37 (Rawhide/F25)
Comment 16 Vasant Hegde 2016-03-28 04:32:26 UTC 
Looks like its fixed in rawhide branch.. But not yet included in Fedora24 branch.


changelog from rawhide branch.

fedora-release-25-0.7.noarch.rpm
* Fri Mar 18 2016 Dennis Gilmor <dennis> - 25-0.7
- drop Requires(post): sed
- Fork to execute systemctl calls

* Tue Mar 15 2016 Dennis Gilmore <dennis> - 25-0.6
- Properly handle systemd presets in Lua scripts
- enable opal-prd.service
- Remove call to grub2-mkconfig


-Vasant
Comment 17 Dennis Gilmore 2016-03-28 14:49:32 UTC 
It was not pulled into f24 yet as we were in alpha freeze and it did not fix anything that was proposed as a blocker or freeze exception. As such it was inappropriate to pull in.
Comment 18 Mike McCune 2016-03-28 23:33:48 UTC 
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions
Comment 19 Peter Robinson 2016-05-16 11:26:53 UTC 
This will be in fedora-release-24-0.18 (or the next one)
Comment 20 Vasant Hegde 2016-06-16 16:56:47 UTC 
Verified on fedora-release-24-1.noarch.rpm and its fixed.
We can close this bug.

-Vasant
Note You need to log in before you can comment on or make changes to this bug.