HewlettPackard / lustre_exporter

Prometheus exporter for use with the Lustre parallel filesystem
Apache License 2.0
34 stars 51 forks source link

Fix job id missing process name #137

Closed wutz closed 5 years ago

wutz commented 5 years ago

When enable jobid_var = procname_uid in lustre, the lustre client also collect job id like <process_name>.<uid> .

The current lustre_exporter implement has missing to handle process name in job id, which also cause the issue #135

wutz commented 5 years ago

https://github.com/HewlettPackard/lustre_exporter/blob/6d69c7d8c2a1791826ec6dccd8c61da95fc607c5/sources/procfs.go#L665 This also need to be fixed, it does not work with complex process name like abc .0123 .-_+ AB

wutz commented 5 years ago

@joehandzik

joehandzik commented 5 years ago

Clearly we should have merged this long ago. We appreciate the support, and sincere apologies for they ridiculous delay here.