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

Add MSMQ collector #159

Merged
merged 5 commits into from
Feb 14, 2018
Merged

Add MSMQ collector #159

merged 5 commits into from
Feb 14, 2018

Conversation

Teriand
Copy link
Contributor

@Teriand Teriand commented Feb 14, 2018

Add MSMQ collector - Win32_PerfRawData_MSMQ_MSMQQueue
Count message and size for both queueu and journal.
metrics:

# HELP wmi_msmq_messagesin_journal_queue Count messages in queue journal
# TYPE wmi_msmq_messagesin_journal_queue gauge
wmi_msmq_messagesin_journal_queue{name="msk-fna\\am_q"} 435

# HELP wmi_msmq_messagesin_queue Count messages in queue
# TYPE wmi_msmq_messagesin_queue gauge
wmi_msmq_messagesin_queue{name="msk-fna\\am_mirror_queue"} 2303

# HELP wmi_msmq_bytesin_journal_queue Size of queue journal in bytes
# TYPE wmi_msmq_bytesin_journal_queue gauge
wmi_msmq_bytesin_journal_queue{name="msk-fna\\am_q"} 1.712848e+06

# HELP wmi_msmq_bytesin_queue Size of queue in bytes
# TYPE wmi_msmq_bytesin_queue gauge
wmi_msmq_bytesin_queue{name="msk-fna\\am_mirror_queue"} 4.44604e+06

@carlpett carlpett mentioned this pull request Feb 14, 2018
@carlpett
Copy link
Collaborator

@Teriand Thanks for the contribution, looks really good!
My only wish would be to add the missing _ before "in" in the metric names so they are easier to read (wmi_msmq_bytesin_queue -> wmi_msmq_bytes_in_queue, etc).

@Teriand
Copy link
Contributor Author

Teriand commented Feb 14, 2018

Good wish, added missing "_"

@carlpett carlpett merged commit 6b98771 into prometheus-community:master Feb 14, 2018
@carlpett
Copy link
Collaborator

Great work, thanks! 🎉

anubhavg-icpl pushed a commit to anubhavg-icpl/windows_exporter that referenced this pull request Sep 22, 2024
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