Summary: Bump local prometheus retention down to 1 month
ClosedPublic
Actions

Authored by olasd on Sep 13 2022, 7:26 PM.

Details

Reviewers

vsellier
ardumont

Group Reviewers

System administrators

Commits

rSPSITE781960c26298: Bump local prometheus retention down to 1 month
rSPSITE009710953dfe: Add icinga2 warning for thanos sidecar shipper

Summary

To do so, add an icinga2 warning for thanos sidecar shipper

Depends on D8470

Test Plan

Tested the icinga2 service check without exporting it, and the generation went fine.

Diff Detail

Repository

rSPSITE puppet-swh-site

Branch

detached-head

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 31520
Build 49308: arc lint + arc unit

Event Timeline

olasd created this revision.Sep 13 2022, 7:26 PM

Herald added a reviewer: System administrators. · View Herald TranscriptSep 13 2022, 7:26 PM

Harbormaster completed remote builds in B31520: Diff 30512.Sep 13 2022, 7:26 PM

olasd requested review of this revision.Sep 13 2022, 7:26 PM

vsellier accepted this revision.Sep 13 2022, 7:29 PM

This revision is now accepted and ready to land.Sep 13 2022, 7:29 PM

Wait, now we can make icinga check with prometheus/thanos metrics?
How awesome!

Thanks.

Closed by commit rSPSITE009710953dfe: Add icinga2 warning for thanos sidecar shipper (authored by olasd). · Explain WhySep 14 2022, 1:44 PM

This revision was automatically updated to reflect the committed changes.

olasd added a commit: rSPSITE009710953dfe: Add icinga2 warning for thanos sidecar shipper.

olasd added a commit: rSPSITE781960c26298: Bump local prometheus retention down to 1 month.

Revision Contents
Changeset List

Path

Size

data/

common/

common.yaml

2 lines

site-modules/

profile/

manifests/

thanos/

prometheus_sidecar.pp

23 lines

Diff 30512

View Options

data/common/common.yaml