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

Unify error messages and prepare to adopt wrapping error messages, part 2 #522

Closed
wants to merge 34 commits into from
Closed

Unify error messages and prepare to adopt wrapping error messages, part 2 #522

wants to merge 34 commits into from

Conversation

conallob
Copy link
Contributor

This is part 2 of an implementation for #358

dependabot bot and others added 30 commits May 11, 2023 22:54
Bumps [golang.org/x/sys](https://github.com/golang/sys) from 0.6.0 to 0.7.0.
- [Release notes](https://github.com/golang/sys/releases)
- [Commits](golang/sys@v0.6.0...v0.7.0)

---
updated-dependencies:
- dependency-name: golang.org/x/sys
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: conall <[email protected]>
…riables

- Include original err whenever raising an exception

Signed-off-by: conall <[email protected]>
issue was found by Svace static analyzer (https://www.ispras.ru/en/technologies/svace/)

Signed-off-by: Artem Seleznev <[email protected]>
Signed-off-by: conall <[email protected]>
Can be used by other projects (eg node_exporter) to read a single
sysfs file rather than all of them at once, which can cause
lock contention in the kernel as many sysfs net files grab the
RTNL lock.

Signed-off-by: Dan Williams <[email protected]>
Signed-off-by: conall <[email protected]>
Modernised method of fetching mdraid statistics via machine-readable
sysfs entries, instead of parsing human-readable /proc/mdstat.

Signed-off-by: Daniel Swarbrick <[email protected]>
Signed-off-by: conall <[email protected]>
Signed-off-by: shankeerthan-kasilingam <[email protected]>
Signed-off-by: conall <[email protected]>
Move the file hadling into the parseNetstat() function to only open one
file at a time. This eliminates keeping all files open until all are
read.

Signed-off-by: SuperQ <[email protected]>
Signed-off-by: conall <[email protected]>
fmt.Errorf() atoms when there is no err to wrap

Signed-off-by: conall <[email protected]>
Co-authored-by: Ben Kochie <[email protected]>
Signed-off-by: Conall O'Brien <[email protected]>
@conallob conallob closed this May 16, 2023
@conallob
Copy link
Contributor Author

conallob commented May 16, 2023

This was intended to be compliment #519, but I managed to goof the git branches, so all changes are now in #519

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.

6 participants