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

stability file names changed post XA30, fix call to super constructor #1

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

Conversation

tokeefe
Copy link

@tokeefe tokeefe commented Apr 1, 2024

This patch addresses two issues

  1. Following the XA30 cutover, the MRI stability output files have different names. This patch fixes the diamond collector to detect and import files with the new file names.

  2. The way the super class initializer was being called was resulting in the error shown below (repeatedly). This patch also fixes that issue

IOError: [Errno 24] Too many open files: '/etc/diamond/diamond.conf'

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.

1 participant