Hello,

I'm glad I could help. :)

I've created the issue here: https://bugzilla.redhat.com/show_bug.cgi?id=1451202

Have a great day! Denis

On Mon, May 15, 2017 at 8:56 PM Kevin Fenzi kevin@scrye.com wrote:

On 05/06/2017 06:42 AM, Denis Nuțiu wrote:

...

Hello,

I have created a patch for the following issue:

https://pagure.io/fedora-infrastructure/issue/6010

Please let me know if something is not okay.

Looks great. Only issue I see is that we don't really need to restart jenkins when just installing a new logrotate.d file.. it should use that next time logrotate runs.

I've pushed this change and will close the ticket.

One final last thing: could you file a bug on the fedora jenkins package and ask them to include the logrotate there? Then we can drop our local copy once they do.

Thanks a bunch for working on this, and sorry for the delays in applying it.

kevin

---

infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org