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.

Sign up for GitHub

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

Jump to bottom

prometheus::mysqld_exporter::cnf_config_path should not overwrite existing my.cnf file #226

Closed
dotSource-GmbH opened this issue Jun 29, 2018 · 1 comment · Fixed by #722
Closed

prometheus::mysqld_exporter::cnf_config_path should not overwrite existing my.cnf file #226

dotSource-GmbH opened this issue Jun 29, 2018 · 1 comment · Fixed by #722
Assignees
@TheMeier

Comments

@dotSource-GmbH
Copy link

The prometheus::mysqld_exporter manifest overwrites an existing my.cnf file with the template.
I think if file exists then do not overwrite.
When i configure my prometheus exporters with puppet, i have all my configuration under version control.
So i don't want any passwords in version control systems. But there is actualy no possibility to change the my.cnf password.
@TheMeier
Copy link
Contributor

TheMeier commented May 5, 2024

I is a bit unlucky to have the default for prometheus::mysqld_exporter::cnf_config_path set to /etc/.my.cnf

@TheMeier TheMeier self-assigned this May 5, 2024
@TheMeier TheMeier mentioned this issue May 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheMeier TheMeier

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

change path or my.cnf for mysqld_exporter voxpupuli/puppet-prometheus
2 participants
@TheMeier @dotSource-GmbH