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

Bump telegraf to 1.28.5 #141

Open
wants to merge 3 commits into
base: 1.28.5-aiven
Choose a base branch
from
Open

Conversation

nicois
Copy link

@nicois nicois commented Apr 29, 2024

Based on the upstream v1.28.5 release, cherry-pick the two Aiven commits used previously with v1.26

mhoffm-aiven and others added 2 commits April 29, 2024 09:56
see AIVEN_CHANGES.md for details
The extra metrics give a better view of what is happening
in the cluster.

The description of the too_many_tries_replicas metrics
was not accurate, this is fixed. Ideally the name should
be changed to, but that would break compatibility.
@nicois nicois marked this pull request as ready for review April 29, 2024 02:16
@nicois nicois changed the title Nicois/bump telegraf to 1.28 Bump telegraf to 1.28.5 Apr 29, 2024
@nicois nicois force-pushed the nicois/bump-telegraf-to-1.28 branch 4 times, most recently from f566185 to 982256b Compare May 7, 2024 01:23
…#15311)

Some badly-behaved sources (such as dragonfly) may send error messages
which are not in the expected format. If errorstat lines are found
which do not include an `=` sign, silently discard them.

Somewhere between 1.26 and 1.28.5, a change was made which caused such
missing '=' symbols to cause a panic.

In addition to ignoring such entries, add debug-level logging for this
and a similar problem, where the errorstat values are not base-10.
@nicois nicois force-pushed the nicois/bump-telegraf-to-1.28 branch from 982256b to 6400904 Compare May 7, 2024 02:56
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.

3 participants