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

change path or my.cnf for mysqld_exporter #722

Merged
merged 1 commit into from
May 19, 2024
Merged

change path or my.cnf for mysqld_exporter #722

merged 1 commit into from
May 19, 2024

Conversation

TheMeier
Copy link
Collaborator

previously /etc/.my.cnf was used. This is the default mysql config path for many other tools and should not be modified by this module

fixes #226

@TheMeier
Copy link
Collaborator Author

Archlinux test failure is addressed in #719

@TheMeier TheMeier added the enhancement New feature or request label May 19, 2024
previously /etc/.my.cnf was used. This is the default mysql config
path for many other tools and should not be modified by this module.
@TheMeier TheMeier removed the enhancement New feature or request label May 19, 2024
@bastelfreak bastelfreak added the bug Something isn't working label May 19, 2024
@bastelfreak bastelfreak merged commit 5a42167 into master May 19, 2024
33 checks passed
@bastelfreak bastelfreak deleted the issue/226 branch May 19, 2024 14:16
@TheMeier TheMeier added this to the v15.0.0 milestone Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

prometheus::mysqld_exporter::cnf_config_path should not overwrite existing my.cnf file
2 participants