https://bugzilla.redhat.com/show_bug.cgi?id=1530431
--- Comment #2 from Tristan Cacqueray tdecacqu@redhat.com --- Thanks for the review Robert.
Spec URL: https://fedorapeople.org/~tdecacqu/pkgs/nodepool.spec SRPM URL: https://fedorapeople.org/~tdecacqu/pkgs/nodepool-0.4.0-1.20171214gita20d81c....
New version changes:
- Add %{?systemd_requires} - Uses %license instead of %doc - Uses %global gitdate 20171214 - Fixes release and changelog using 1.%{gitdate}git%{shortcommit}%{?dist} - Fixes LICENSE encoding - Add logrotate configuration - Add manpage with help2man - Fixes %file directives for correct ownership - Remove sphinx buildinfo
Regarding the remaining rpmlint errors:
non-readable and zero-length /etc/nodepool/secure.conf is expected and may be removed in the final release. This is a legacy file unused anymore, but the service still tries to load it.
non-standard-dir-perm 750 are expected because those directory should only be available to the nodepool user. Not sure why this is an error.