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

Skip empty jobstats #71

Merged
merged 1 commit into from
May 16, 2017
Merged

Skip empty jobstats #71

merged 1 commit into from
May 16, 2017

Conversation

roclark
Copy link
Contributor

@roclark roclark commented May 16, 2017

In some situations, a metric that is being queried by the lustre_exporter
might not be tracked in the job_stats file and can cause GO to panic.
This update skips over any metrics that don't provide valid responses.

Signed-Off-By: Robert Clark robert.d.clark@hpe.com

In some situations, a metric that is being queried by the lustre_exporter
might not be tracked in the job_stats file and can cause GO to panic.
This update skips over any metrics that don't provide valid responses.

Signed-Off-By: Robert Clark <robert.d.clark@hpe.com>
@roclark roclark requested a review from joehandzik May 16, 2017 16:37
@joehandzik
Copy link
Contributor

Rodger, Rodger.

@joehandzik joehandzik merged commit 8fa85a6 into master May 16, 2017
@roclark roclark deleted the wip-fix-empty-metric branch May 16, 2017 16:40
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