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.

Sign up for GitHub

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

Jump to bottom

Procstat input plugin should use the same timestamp in all metrics in the same Gather() cycle. #8658

Merged
merged 1 commit into from
Jan 12, 2021
Merged

Procstat input plugin should use the same timestamp in all metrics in the same Gather() cycle. #8658

merged 1 commit into from
Jan 12, 2021

Conversation

ivorybilled
Copy link
Contributor

resolves #8648

@sspaink sspaink self-requested a review January 7, 2021 19:42
sspaink
sspaink approved these changes Jan 7, 2021
View reviewed changes
Copy link
Contributor

@sspaink sspaink left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@sjwang90 sjwang90 added area/procstat feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin labels Jan 12, 2021
@ivorybilled ivorybilled merged commit 70d2b1f into master Jan 12, 2021
@ivorybilled ivorybilled deleted the sameTimestampProcstatGather branch January 12, 2021 22:59
@adrianlzt adrianlzt mentioned this pull request Jan 27, 2021
Closed
2 tasks
ssoroka pushed a commit that referenced this pull request Jan 27, 2021
@ivorybilled @ssoroka
Procstat input plugin should use the same timestamp in all metrics in…
98e6bdc 
… the same Gather() cycle. (#8658)

(cherry picked from commit 70d2b1f)
@sspaink sspaink mentioned this pull request Jun 1, 2022
Closed
2 tasks
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
@ivorybilled
Procstat input plugin should use the same timestamp in all metrics in…
a337b37 
… the same Gather() cycle. (influxdata#8658)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sspaink sspaink sspaink approved these changes

Assignees
No one assigned
Labels
area/procstat feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

procstat plugin create metrics with different timestamps in the same gather cycle
3 participants
@ivorybilled @sspaink @sjwang90