Patch for: Issue#6010 Add logrotate to jenkins master

[Denis Nuțiu]

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.

[Kevin Fenzi]

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

[Denis Cosmin]

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  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 -- [e-mail]
> To unsubscribe send an email to
> [e-mail]