Bug 253438 - devel/jenkins & devel/jenkins-lts: Add newsyslog configuration
Summary: devel/jenkins & devel/jenkins-lts: Add newsyslog configuration
Status: In Progress
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Li-Wen Hsu
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-11 17:08 UTC by Brad Davis
Modified: 2021-06-20 03:29 UTC (History)
2 users (show)

See Also:


Attachments
patch (2.74 KB, patch)
2021-02-11 17:08 UTC, Brad Davis
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brad Davis freebsd_committer freebsd_triage 2021-02-11 17:08:32 UTC
Created attachment 222371 [details]
patch

This follows the path of what the Jenkins packages for other OSes do[1].

1: https://github.com/jenkinsci/jenkins/commit/8b09a585243287b6cb5270a0a1447995f1fc3dff
Comment 1 Li-Wen Hsu freebsd_committer freebsd_triage 2021-02-11 17:20:49 UTC
Looks good to me. Wait for reply from swills.
Comment 2 Brad Davis freebsd_committer freebsd_triage 2021-03-30 15:23:23 UTC
Can I commit now that you own both?
Comment 3 Li-Wen Hsu freebsd_committer freebsd_triage 2021-03-30 15:45:06 UTC
I'm fine with it. OTOH, I'm thinking maybe it's also good to install as a sample file and notice user in the pkg-message. What do you think about this?
Comment 4 Olli Hauer freebsd_committer freebsd_triage 2021-06-16 12:31:09 UTC
I agree with Li-Wen-Hsu to install as sample file, e.g. I'm haveing a different rotate scedule and it would be nice to not overwrite it all the time jenkins is updated

Additional it would be nice to use the official --logfile parameter, see bug #256644
Comment 5 Li-Wen Hsu freebsd_committer freebsd_triage 2021-06-20 03:29:50 UTC
I'm testing this, however, I found that it seems we cannot just use this newsyslog setting.  If we don't specify pid_file, the process will not be aware of fd changed and there will be no new log appended until restart.  If we add pid_file, the default HUP signal will stop the Jenkins process.  I'm still finding if there is other solution.

Note that the patch in comment #0 is using "copytruncate" as workaround.