Bug 1285672 - Bird should start in foreground in the systemd unit
Bird should start in foreground in the systemd unit
Status: ASSIGNED
Product: Fedora EPEL
Classification: Fedora
Component: bird (Show other bugs)
epel7
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Stanislav Kozina
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2015-11-26 03:54 EST by Warren Turkal
Modified: 2017-04-14 07 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Run bird in foreground (301 bytes, patch)
2015-11-26 03:54 EST, Warren Turkal
no flags Details | Diff
another patch (3.10 KB, patch)
2017-04-14 07:27 EDT, Marcel Haerry
no flags Details | Diff

  None (edit)
Description Warren Turkal 2015-11-26 03:54:34 EST
Created attachment 1099170 [details]
Run bird in foreground

Description of problem:
The systemd unit file doesn't run bird in a good way to integrate with the systemd journal.


Version-Release number of selected component (if applicable):
bird-1.4.5-1.el7.x86_64

How reproducible:
Always able to reproduce.


Steps to Reproduce:
1. install bird package
2. run "systemctl start bird.service"

Actual results:
The command used to exec the service is "/usr/sbin/bird". This results in bird daemonizing itself. Also, when I have "log stderr all", I see no logs from bird after start.


Expected results:
The command used should be "/usr/sbin/bird -f" so that bird runs in the foreground and integrates better with the systemd journal.


Additional info:
I added an attachment with a patch for the unit file at /usr/lib/systemd/system/bird.service.
Comment 1 Warren Turkal 2015-12-08 19:14:56 EST
I haven't seen any feedback on this request. I was wondering, does this seem like a reasonable change to make?
Comment 2 Stanislav Kozina 2016-05-05 07:06:55 EDT
Hi Warren,
Thank you for your request and sorry for reaaaaly long delay in my reply. Yes, it seems perfectly reasonable, I'll take a look at it. Thanks!
Comment 3 Warren Turkal 2016-05-05 13:29:43 EDT
Sweet, thanks for the reply.
Comment 4 Marcel Haerry 2017-04-14 07:27 EDT
Created attachment 1271651 [details]
another patch

Attached is a patch against the current master of the fedpkg repository.

This also addresses things raised in #1397574

I'm happy to help with the package if you need more support.

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