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

logging: Log no action messages at debug level #2442

Closed
wants to merge 1 commit into from
Closed

logging: Log no action messages at debug level #2442

wants to merge 1 commit into from

Conversation

mcassaniti
Copy link

There's a few issues floating around about the verbosity of logged messages that start with 'no action'. PR #2161 attempts to resolve this by reducing how often the messages are printed. This PR instead changes the severity level which is easy enough to reconfigure.

This should fix #1154, #1800 and #621. It would also help with #1203.

@CyberDem0n
Copy link
Member

Sorry, but just silencing log messages will not be accepted.

@mcassaniti
Copy link
Author

Sorry, but just silencing log messages will not be accepted.

Correct me if I am wrong, but the logging level can be changed from info to debug as part of the configuration. When changed the output will be logged. I certainly don't want to drop these log messages always, but instead to configure when they are produced

@CyberDem0n
Copy link
Member

The DEBUG level is too verbose.
The number of people who don't have any issues with seeing heartbeat messages every 10 seconds is certainly much higher than people who would like to run Patroni with log.level=DEBUG.

@mcassaniti
Copy link
Author

OK, when should the logs actually show heartbeat messages? From the issues I linked and the number of comments on each it seems logging these by default is not what users wish to see.

How many users wish to see heartbeat messages every 10 seconds under the default configuration? That is a different question than how many users are indifferent.

@CyberDem0n
Copy link
Member

From the issues I linked and the number of comments on each it seems logging these by default is not what users wish to see.

Issues don't always reflect the real world. The total amount of Patroni users is much higher than the number of users opening/commenting issues.

@mcassaniti
Copy link
Author

OK, what would be accepted so that a user could drop any 'no action' messages from the log. Is a simple configuration option sufficient?

@mcassaniti
Copy link
Author

@CyberDem0n Just nudging this one again. Would a non-default configuration option be accepted that silenced any 'no action' messages? If so I'm happy to write a PR and we can nut it out.

@mcassaniti mcassaniti closed this by deleting the head repository Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Excessive log messages
2 participants