Bug 2265387
Summary: | Review Request: ulogd - Userspace logging daemon for netfilter | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Thomas Haller <thaller> | ||||
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> | ||||
Status: | NEW --- | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | egarver, package-review, pgnd | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
URL: | https://www.netfilter.org/projects/ulogd/ | ||||||
Whiteboard: | Unretirement | ||||||
Fixed In Version: | Doc Type: | If docs needed, set a value | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | Type: | --- | |||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | 2024579 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Thomas Haller
2024-02-21 19:46:51 UTC
Copr build: https://copr.fedorainfracloud.org/coprs/build/7047104 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2265387-ulogd/fedora-rawhide-x86_64/07047104-ulogd/builder-live.log.gz Please make sure the package builds successfully at least for Fedora Rawhide. - If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field --- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string. + # SIGHUP would reopen files (for logrotate). However, that may not work with + # "ULOGD_ARGS=-u ulog", as we won't have permissions to reopen the files. + ExecReload=/bin/kill -HUP $MAINPID ExecReload is optional. If we expect this to not work then maybe we simply don't define it. Otherwise this looks good to me. Thanks! (In reply to Eric Garver from comment #2) > ExecReload is optional. If we expect this to not work then maybe we simply > don't define it. I dropped `ExecReload`. Created attachment 2019097 [details]
rpmlint output 1 (rpmlint-for-ulogd-1.txt)
generated via
$ ( for f in $(ls -1 *spec *src.rpm x86_64/*rpm); do echo ">>>> rpmlint $f"; rpmlint "$f"; done ; for f in $(rpm -qa |grep '^ulogd' | sed 's/-2\..*//') ; do echo ">>>> rpmlint -i $f"; rpmlint -i "$f"; done ) 2>&1
|