Re: Patch for: Issue#6010 Add logrotate to jenkins master

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
--
Sincerely,
Denis Nutiu
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux