From 2b385eb0a65ec4012cf3c391c4fd86b4adcc1d80 Mon Sep 17 00:00:00 2001 From: David Schmitt Date: Sun, 26 Aug 2007 22:18:00 +0200 Subject: recognize lsbdistcodename 'n/a' micah reported, that lsb_release can report nonsensical values for lsbdistcodename; assert_lsbdistcodename now recognises "n/a" and acts accordingly --- manifests/classes/lsb_release.pp | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) (limited to 'manifests') diff --git a/manifests/classes/lsb_release.pp b/manifests/classes/lsb_release.pp index 22717b0..fbea24b 100644 --- a/manifests/classes/lsb_release.pp +++ b/manifests/classes/lsb_release.pp @@ -4,6 +4,11 @@ # Copyright (C) 2007 David Schmitt # See LICENSE for the full license granted to you. +# Changelog: +# 2007-08-26: micah reported, that lsb_release can report +# nonsensical values for lsbdistcodename; assert_lsbdistcodename now +# recognises "n/a" and acts accordingly + # This lightweight class only asserts that $lsbdistcodename is set. # If the assertion fails, an error is printed on the server # @@ -16,6 +21,17 @@ class assert_lsbdistcodename { err("Please install lsb_release or set facter_lsbdistcodename in the environment of $fqdn") exec { "/bin/false # assert_lsbdistcodename": alias => assert_lsbdistcodename } } + 'n/a': { + case $operatingsystem { + "Debian": { + err("lsb_release was unable to report your distcodename; This seems to indicate a broken apt/sources.list on $fqdn") + } + default: { + err("lsb_release was unable to report your distcodename; please set facter_lsbdistcodename in the environment of $fqdn") + } + } + exec { "/bin/false # assert_lsbdistcodename": alias => assert_lsbdistcodename } + } default: { exec { "/bin/true # assert_lsbdistcodename": alias => assert_lsbdistcodename } exec { "/bin/true # require_lsbdistcodename": alias => require_lsbdistcodename } -- cgit v1.2.3