Page MenuHomeSoftware Heritage

icinga checks: Activate the prometheus export on e2e tests
ClosedPublic

Authored by vsellier on Fri, May 6, 5:29 PM.

Details

Summary

Relative to T3129
Depends on D6926
Depends on D7844

Test Plan
  • no testable by octocatalog-diff because it's collected resources
  • no tested in vagrant because of vagrant issues

Diff Detail

Repository
rSPSITE puppet-swh-site
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

olasd added a subscriber: olasd.

🤷🤷🤷 let's try it and see...

This revision is now accepted and ready to land.Mon, May 9, 11:19 AM
ardumont added a subscriber: ardumont.
ardumont added inline comments.
site-modules/profile/manifests/icinga2/objects/e2e_checks_deposit.pp
17

will the icinga check service have the right permission to write into that directory [1]?

[1]

prometheus::node::textfile_directory: /var/lib/prometheus/node-exporter
  • rebase
  • move the parameters after 'icinga_plugins'
  • add an environment parameter