Skip to content

Conversation

@NiccoloTosato
Copy link
Contributor

@NiccoloTosato NiccoloTosato commented Nov 10, 2025

Summary

Interval < 1s

Since turbostat supports intervals smaller than 1s, I've fixed the error that arises in case of intervals less than 1s, which results in truncating every interval to 0. Moreover, I've converted the interval into floating-point seconds, which now correctly affects the sampling of turbostat.

Checklist

Related issues

resolves #17957

@telegraf-tiger telegraf-tiger bot added fix pr to fix corresponding bug plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins labels Nov 10, 2025
Copy link
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @NiccoloTosato!

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Nov 10, 2025
@telegraf-tiger
Copy link
Contributor

Copy link
Contributor

@skartikey skartikey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NiccoloTosato Thanks for the contribution!

@skartikey skartikey merged commit 7d62771 into influxdata:master Nov 11, 2025
29 checks passed
@github-actions github-actions bot added this to the v1.36.4 milestone Nov 11, 2025
srebhan pushed a commit that referenced this pull request Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix pr to fix corresponding bug plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[inputs.turbostat] Interval cannot be less than 1s

4 participants