X-Git-Url: https://review.fuel-infra.org/gitweb?a=blobdiff_plain;f=manifests%2Fupdate.pp;h=44f6c6e301ca37f4d48ca4357a2301928db5cc79;hb=a71dfd168296f12207749131d4f734ae671622a4;hp=86f068200c7890d4e59d77ba250ff1cc9096b4bb;hpb=dc746734f17faa996e8441d176143945e8347330;p=puppet-modules%2Fpuppetlabs-apt.git diff --git a/manifests/update.pp b/manifests/update.pp index 86f0682..44f6c6e 100644 --- a/manifests/update.pp +++ b/manifests/update.pp @@ -1,4 +1,6 @@ class apt::update { + assert_private() + #TODO: to catch if $::apt_update_last_success has the value of -1 here. If we #opt to do this, a info/warn would likely be all you'd need likely to happen #on the first run, but if it's not run in awhile something is likely borked @@ -13,7 +15,7 @@ class apt::update { #if we should kick apt_update. $daily_threshold = (strftime('%s') - 86400) if $::apt_update_last_success { - if $::apt_update_last_success < $daily_threshold { + if $::apt_update_last_success + 0 < $daily_threshold { $_kick_apt = true } else { $_kick_apt = false @@ -28,7 +30,7 @@ class apt::update { #if we should kick apt_update. $weekly_threshold = (strftime('%s') - 604800) if $::apt_update_last_success { - if ( $::apt_update_last_success < $weekly_threshold ) { + if ( $::apt_update_last_success + 0 < $weekly_threshold ) { $_kick_apt = true } else { $_kick_apt = false @@ -39,7 +41,7 @@ class apt::update { } } default: { - #catches 'recluctantly', and any other value (which should not occur). + #catches 'reluctantly', and any other value (which should not occur). #do nothing. $_kick_apt = false }