Page MenuHomeSoftware Heritage

icinga checks: Activate the prometheus export on e2e tests
ClosedPublic

Authored by vsellier on May 6 2022, 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
Branch
staging
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 29442
Build 46016: arc lint + arc unit

Event Timeline

olasd added a subscriber: olasd.

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

This revision is now accepted and ready to land.May 9 2022, 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