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

fix(inputs.docker): keep field type of tasks_desired the same #10711

Merged
merged 1 commit into from
Mar 8, 2022

Conversation

sspaink
Copy link
Contributor

@sspaink sspaink commented Feb 22, 2022

Resolve: #8570

The field tasks_desired could change between int and uint64, which caused the issue already exists as type in #8570. Keeping it as uint64 seems like the best option because tasksNoShutdown which was an int that only increments up and should never be negative.

keep field type of tasks_desired the same
@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Feb 22, 2022
@telegraf-tiger
Copy link
Contributor

@sspaink sspaink 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 Mar 7, 2022
@sspaink sspaink merged commit 80d66df into master Mar 8, 2022
@sspaink sspaink deleted the dockerfix branch March 8, 2022 15:00
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 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.

Docker plugin + InfluxDB 2.0: type mismatch
2 participants