Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable customized log4j.properties files #200

Closed
elnormeo opened this issue Aug 28, 2017 · 4 comments · Fixed by #328
Closed

Enable customized log4j.properties files #200

elnormeo opened this issue Aug 28, 2017 · 4 comments · Fixed by #328
Labels
enhancement New feature or request

Comments

@elnormeo
Copy link

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: current
  • Ruby: current
  • Distribution: current
  • Module version: current

How to reproduce (e.g Puppet code you use)

Let Kafka run for several days.

What are you seeing

Logfiles are rotated but never deleted. Even logrotate does not work in this case.

What behaviour did you expect instead

Possibility to configure the logfiles retention in this puppet module

Output log

n/a

@juniorsysadmin juniorsysadmin added the enhancement New feature or request label Sep 29, 2017
@ffrank
Copy link

ffrank commented Oct 3, 2017

Hi, what do you mean by "even logrotate does not work"?

Have you implemented logrotate to remove log files, but it won't work for some reason? What is the reason?

@AlexWerz
Copy link

We have the same problem. Our Kafka runs for several days, logs are created and rotated daily but never actually deleted. In rare cases they fill the harddisk until free space runs out.

@witchbutter
Copy link

The recommended way to do this is detailed here -> https://community.hortonworks.com/content/supportkb/49706/kafka-log-file-rotation-based-on-file-size.html
It would be simple to add with a log4j.properties template.

@bastelfreak
Copy link
Member

@witchbutter thanks for pointing this out. Could you provide a patch for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants