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

Publish 5 minute mqtt msg process count to MQTT #1881

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alandpearson
Copy link

Publish 5 minute mqtt msg process count to mqtt so can be seen as input in emoncms
This allows users to easily see performance and any variances in mqtt volume. Also changed the log level for this to WARN so it can easily be seen in the logfile without drowning in debug data.
This is a very useful stat to see (and log optionally log as a feed) and helped me debug several problems with my installation (run away rfm69 module for example).

input in emoncms
This allows users to easily see performance and any variances in mqtt volume.
Also changed the log level for this so it can easily be seen in the
logfile without drowning in debug data
This is a very useful stat to see (and log optionally log as a feed) and
helped me debug several problems with my installation (run away rfm69
module for example)
@chaveiro
Copy link
Contributor

chaveiro commented May 1, 2024

As its not a warning, why log it as a warning?

@alandpearson
Copy link
Author

There only seems to be 3 levels of logging supported, and 'INFO' drowns the log file in mqtt messages meaning this useful message gets lost.

@TrystanLea
Copy link
Member

Thanks @alandpearson I can see the useful intention of this, perhaps it would be good to have the option to enable this as a setting in settings.ini? I can see some users finding this input and being confused even if useful for those who know.
Cheers

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.

None yet

3 participants