Giuseppe Lavagetto has submitted this change and it was merged. Change subject: labstore: ignore the replication snapshots with check_disk ......................................................................
labstore: ignore the replication snapshots with check_disk Creation of the temporary snapshots currently cause check_disk to try to statvfs() them, which fails. Since the check is completely vacuous on a readonly snapshot anyways, simply exclude them explicitly. I hate that this is hidden away in a hiera datapoint rather than in plain view in the manifest, but I can't think of a better way atm. Change-Id: I4c4ba8fba17db8b6e80052fe50a97eb63475a926 --- M hieradata/role/common/labs/nfs/fileserver.yaml 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Giuseppe Lavagetto: Looks good to me, approved jenkins-bot: Verified diff --git a/hieradata/role/common/labs/nfs/fileserver.yaml b/hieradata/role/common/labs/nfs/fileserver.yaml index e85071f..fe5ab57 100644 --- a/hieradata/role/common/labs/nfs/fileserver.yaml +++ b/hieradata/role/common/labs/nfs/fileserver.yaml @@ -7,3 +7,6 @@ # Paramiko needs to ssh into these for replication/backups "ssh::server::disable_nist_kex": false "ssh::server::explicit_macs": false +# the snapshots created by the backup process cannot be checked +# by the check_disk nrpe plugin so need to be excluded. +base::monitoring::host::nrpe_check_disk_options: -w 10% -c 5% -l -e -A -i /run/lock/storage-replicate-*/snapshot -- To view, visit https://gerrit.wikimedia.org/r/231043 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c4ba8fba17db8b6e80052fe50a97eb63475a926 Gerrit-PatchSet: 2 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: coren <[email protected]> Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
