Ensure icinga alerts are raised if scheduler journal client service is down
ClosedPublic
Actions

Authored by ardumont on Aug 30 2021, 5:42 PM.

Details

Reviewers

vsellier

Group Reviewers

System administrators

Maniphest Tasks

T3502: Date overflow error in scheduler journal client

Commits

rSPSITE2434b83e98e9: Ensure icinga alerts are raised if scheduler journal client service is down

Summary

Related to T3502

Test Plan

octo-diff cannot be used here (required collection resources).

Vagrant:

Install the checks as collected resources (puppet agent --test on vm scheduler nodes, saatchi, scheduler0-staging).
Then installed on vm pergamon (puppet agent --test).
Checks are appropriately either down or up when modifying their status on scheduler vms in the icinga ui.

Diff Detail

Repository

rSPSITE puppet-swh-site

Branch

staging

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 23275
Build 36316: arc lint + arc unit

Event Timeline

ardumont requested review of this revision.Aug 30 2021, 5:42 PM

ardumont created this revision.

Harbormaster completed remote builds in B23275: Diff 22321.Aug 30 2021, 5:42 PM

ardumont abandoned this revision.Aug 30 2021, 5:42 PM

ardumont mentioned this in D6156: Ensure icinga alerts are raised if the scheduler journal client is down.

ardumont added a subscriber: olasd.Aug 31 2021, 3:57 PM

ardumont added inline comments.

site-modules/profile/manifests/swh/deploy/scheduler/journal_client.pp
46	missing `command_endpoint => $::fqdn,` to tell the check to be executed on the machine! Thanks @olasd.

Apply suggestion! And now the checks are working properly

Harbormaster completed remote builds in B23292: Diff 22336.Aug 31 2021, 4:23 PM

ardumont retitled this revision from wip: Ensure icinga alerts are raised if scheduler journal client service is down to Ensure icinga alerts are raised if scheduler journal client service is down.Aug 31 2021, 4:24 PM

ardumont edited the summary of this revision. (Show Details)

ardumont edited the test plan for this revision. (Show Details)

ardumont edited the test plan for this revision. (Show Details)Aug 31 2021, 4:28 PM