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

Log files are not rotated properly on remote pollers #3629

Closed
bmfmancini opened this issue Jun 19, 2020 · 6 comments
Closed

Log files are not rotated properly on remote pollers #3629

bmfmancini opened this issue Jun 19, 2020 · 6 comments
Labels
bug Undesired behaviour resolved A fixed issue
Milestone

Comments

@bmfmancini
Copy link
Member

Hey, everyone!

I notice that on 1.2.12 log files are not rotating but only on the remote pollers
the GUI is able to access the log and purge it so all permissions seem to be fine

I know this worked up until 1.2.11

on the main poller, the logs rotate just fine

I went to each remote poller and re-saved the log settings just in case but no difference

@bmfmancini bmfmancini added bug Undesired behaviour unverified Some days we don't have a clue labels Jun 19, 2020
@netniV
Copy link
Member

netniV commented Jun 19, 2020

The rotation logic itself hasn't changed in a long while but does make me wonder if the log rotation date has been syncrhonised as it's in the settings table.

@bmfmancini
Copy link
Member Author

bmfmancini commented Jun 20, 2020 via email

@TheWitness
Copy link
Member

There are just few settings not replicated, mostly '%path%' related. When you are in 'online' mode, anything you save with the exception of '%path%' for the most part is saved on the main data collector. In offline mode, you can override only until the next full sync though pretty sure.

@netniV
Copy link
Member

netniV commented Jun 20, 2020

If log rotation dates are synced we should do that

@TheWitness
Copy link
Member

Well, looking at poller_maintenance.php, it looks like some things should first be bypassed, second, we should force the database to use a local connection when it's running on the remote collector.

I'm okay doing this for the 1.2.x branch. How about you?

@TheWitness TheWitness removed the unverified Some days we don't have a clue label Jun 20, 2020
@TheWitness TheWitness added this to the 1.2.13 milestone Jun 20, 2020
TheWitness added a commit that referenced this issue Jun 21, 2020
- Some database conditions lead to warnings in the Cacti log
- Restructuring the poller maintenances into function blocks to simplify code flow.
@TheWitness
Copy link
Member

I did a little restructuring of the maintenance script and nothing jumped out at me. I'll keep looking.

@netniV netniV changed the title Cacti v1.2.12 log files not rotating on Remote pollers Log files are not rotated properly on remote pollers Jun 28, 2020
@netniV netniV added the resolved A fixed issue label Jul 13, 2020
@netniV netniV closed this as completed Jul 13, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Oct 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour resolved A fixed issue
Projects
None yet
Development

No branches or pull requests

3 participants