Skip to content

Fix rollover bug in mountstats collector#1364

Merged
SuperQ merged 1 commit intomasterfrom
superq/fix-mountstats-rollover
May 31, 2019
Merged

Fix rollover bug in mountstats collector#1364
SuperQ merged 1 commit intomasterfrom
superq/fix-mountstats-rollover

Conversation

@SuperQ
Copy link
Member

@SuperQ SuperQ commented May 30, 2019

  • Update procfs vendor to pull in github.com/Fix overflow in mounstats procfs#165
  • Update mountstats collector to use new types.
  • Rollover counter automatically to avoid float64 accuracy issues.
  • Update e2e test.

Signed-off-by: Ben Kochie superq@gmail.com

Fixes: #1363

@SuperQ SuperQ requested a review from discordianfish May 30, 2019 10:53
@SuperQ
Copy link
Member Author

SuperQ commented May 30, 2019

cc: @pgier

@SuperQ SuperQ force-pushed the superq/fix-mountstats-rollover branch from d62e545 to 888a7c0 Compare May 30, 2019 18:44
Copy link
Contributor

@pgier pgier 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, I think it just needs a go mod tidy to update go.sum

* Update procfs vendor to pull in github.com/prometheus/procfs/pull/165
* Update mountstats collector to use new types.
* Rollover counter automatically to avoid float64 accuracy issues.
* Update e2e test.

Signed-off-by: Ben Kochie <superq@gmail.com>
@SuperQ SuperQ force-pushed the superq/fix-mountstats-rollover branch from 888a7c0 to f5a2b9c Compare May 31, 2019 14:18
@SuperQ
Copy link
Member Author

SuperQ commented May 31, 2019

@pgier Whups, fixed go.mod.

Copy link
Contributor

@pgier pgier left a comment

Choose a reason for hiding this comment

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

LGTM

@SuperQ SuperQ merged commit 8146998 into master May 31, 2019
@SuperQ SuperQ deleted the superq/fix-mountstats-rollover branch May 31, 2019 16:31
SuperQ added a commit that referenced this pull request Jun 3, 2019
Signed-off-by: Ben Kochie <superq@gmail.com>
@SuperQ SuperQ mentioned this pull request Jun 4, 2019
SuperQ added a commit that referenced this pull request Jun 4, 2019
* Update procfs vendor to pull in github.com/prometheus/procfs/pull/165
* Update mountstats collector to use new types.
* Rollover counter automatically to avoid float64 accuracy issues.
* Update e2e test.

Signed-off-by: Ben Kochie <superq@gmail.com>
SuperQ added a commit that referenced this pull request Jun 4, 2019
Signed-off-by: Ben Kochie <superq@gmail.com>
SuperQ added a commit that referenced this pull request Jun 4, 2019
* Update procfs vendor to pull in github.com/prometheus/procfs/pull/165
* Update mountstats collector to use new types.
* Rollover counter automatically to avoid float64 accuracy issues.
* Update e2e test.

Signed-off-by: Ben Kochie <superq@gmail.com>
SuperQ added a commit that referenced this pull request Jun 4, 2019
Signed-off-by: Ben Kochie <superq@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Update procfs vendor to pull in github.com/prometheus/procfs/pull/165
* Update mountstats collector to use new types.
* Rollover counter automatically to avoid float64 accuracy issues.
* Update e2e test.

Signed-off-by: Ben Kochie <superq@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
Signed-off-by: Ben Kochie <superq@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Update procfs vendor to pull in github.com/prometheus/procfs/pull/165
* Update mountstats collector to use new types.
* Rollover counter automatically to avoid float64 accuracy issues.
* Update e2e test.

Signed-off-by: Ben Kochie <superq@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
Signed-off-by: Ben Kochie <superq@gmail.com>
cce added a commit to algorand/node_exporter that referenced this pull request Feb 18, 2025
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.

Mounstats rollover with older kernels

2 participants