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

class_infiniband: continue on syscall EINVAL #705

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mfuller-lambda
Copy link

Addresses #704, in a very naive way.

In some very bleeding-edge configurations, syscalls against some IB counters will return invalid argument. This bubbles back to callers (e.g. node_exporter) in a very bad way -- e.g. all IB metrics collection failing when a single or group of IB ports returns this.

Suspect this has always been an error case and possible to experience (the syscall code path in procfs has been in place for ages), but it's the first time we're seeing it in a very new hardware deployment.

prometheus#704

In some very bleeding-edge configurations, syscalls against some IB
counters will return `invalid argument`. This bubbles back to callers (e.g. node_exporter) in
a very bad way -- e.g. all IB metrics collection failing when a single
or group of IB ports returns this.

Suspect this has always been an error case and possible to experience,
but it's the first time we're seeing it in a very new hardware deployment.

Signed-off-by: Michael Fuller <[email protected]>
@valner
Copy link

valner commented Mar 26, 2025

@SuperQ @pgier @discordianfish Could you please take a look?
We’ve encountered a similar issue, and this PR fixes it. We would appreciate it if we could get it merged.

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.

2 participants