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

node exporter also installs prometheus server on monitored node #184

Closed
moon-hawk opened this issue Apr 11, 2018 · 2 comments
Closed

node exporter also installs prometheus server on monitored node #184

moon-hawk opened this issue Apr 11, 2018 · 2 comments

Comments

@moon-hawk
Copy link

moon-hawk commented Apr 11, 2018

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 4.10.10
  • Ruby: 2.1.9p490 (2016-03-30 revision 54437) [x86_64-linux]
  • Distribution: Debian 9.4 (Stretch)
  • Module version: v5.0.1-rc0

How to reproduce (e.g Puppet code you use)

When we use

class { 'prometheus::node_exporter':
    version => '0.14.0',
    ...
}

for the monitored nodes, we get also a prometheus server installed on the monitored nodes.

What are you seeing

promethes server gets installed (Version 1.5.2)
and node exporter gets installed (Version 0.14.0)

What behaviour did you expect instead

only node exporter gets installed (Version 0.14.0)

Any additional information you'd like to impart

i think prometheus::node_exporter inherits prometheus
thats why prometheus server gets installed in Version 1,
see your documentation in section "On the server (for prometheus version >= 1.0.0)"

@blupman
Copy link

blupman commented May 3, 2018

I have updated the pull request to remove all inherits from the exports, and replaced the missing data by using lookup(). We're currently running this code on our infra

@bastelfreak
Copy link
Member

I proposed a patch in #194

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

No branches or pull requests

3 participants