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

Update paho.mqtt.golang #6149

Merged
merged 1 commit into from
Jul 22, 2019
Merged

Update paho.mqtt.golang #6149

merged 1 commit into from
Jul 22, 2019

Conversation

glinton
Copy link
Contributor

@glinton glinton commented Jul 22, 2019

Resolves #6142

@danielnelson danielnelson added this to the 1.12.0 milestone Jul 22, 2019
@danielnelson danielnelson merged commit 6f2e57a into master Jul 22, 2019
@danielnelson danielnelson deleted the bugfix/6142 branch July 22, 2019 21:14
@dogada
Copy link

dogada commented Jul 23, 2019

@glinton thanks for quick fix, do you have Docker image with night builds?

@glinton
Copy link
Contributor Author

glinton commented Jul 23, 2019

I don't believe we do have nightly docker images, but this should be in the official nightly builds now.

It should be trivial to drop the new telegraf binary into a new docker image based on the official telegraf docker image.

@dogada
Copy link

dogada commented Sep 3, 2019

We tested nightly builds in our environment and we don't see stack trace mentioned in #6142 but the problem still persists but with complete silence in logs now. Once per 3-4 days Telegraf just stops to receive data from MQTT broker (we use Mosqitto) and we don't see any diagnostic messages in the logs.

@danielnelson
Copy link
Contributor

It's possible you are now running into #6232, which was an issue with persistent sessions. We just released 1.12.0 which contains fixes for both issues, can you retry your testing with this?

bitcharmer pushed a commit to bitcharmer/telegraf that referenced this pull request Oct 18, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
idohalevi pushed a commit to idohalevi/telegraf that referenced this pull request Sep 29, 2020
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.

Telegraph writes 1 batch from MQTT to to InfluxDB and then restarts with memory error
3 participants