-
-
Notifications
You must be signed in to change notification settings - Fork 236
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
Systemd dependencies cycle with rabbitmq and rabbitmq-exporter #434
Comments
I've noticed the same error with other services that use the class |
I did a quick workaround in https://github.com/egnyte/puppet-prometheus/tree/v10.0.0_workaround_434 but I am not sure if it doesn't have side effects yet, so it's definitely not ready to create a PR. |
You could be less strict with your relationships. You probably don't actually need every resource declared in the exporter (and specifically the unit_file) to come after every resource in rabbitmq. Something like class { 'prometheus::rabbitmq_exporter':
...
require => Class['rabbitmq::service'],
} will probably break the dependency cycle. |
I think that less ordering won't help, @alexjfisher . Because in our code we do not have any ordering between the class that contains |
I think this is obsolete and only affects outdated puppet versions. Pleas re-open the issue if I am wrong. |
Hello,
I am not sure if this is a puppet-prometheus or a puppet-rabbitmq issue but I am having a dependency cycle using them both:
puppet-server: 2.8.1
puppet: 4.10.12
puppet-rabbitmq: 8.5.0
puppet-prometheus: 6.4.0
system: centos7
My manifest is has followed (The prometheus exporter should be added after Rabbitmq, that is why I have a require in the 'prometheus::rabbitmq_exporter' class)
The error that I have is:
Any ideas ?
The text was updated successfully, but these errors were encountered: