]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commitdiff
Do not rely on deprecated argument signature
authorRomain Tartière <romain@blogreen.org>
Fri, 26 Jan 2018 15:18:47 +0000 (16:18 +0100)
committerRomain Tartière <romain@blogreen.org>
Fri, 26 Jan 2018 16:07:24 +0000 (17:07 +0100)
This commit fixes the following warning in the puppet master's logs:

Jan 26 16:09:26 <daemon.warn> marvin puppet-master[68168]: The argument signature (String format, [String timezone]) is deprecated for #strfime. See #strftime documentation and Timespan type for more info
Jan 26 16:09:26 <daemon.warn> marvin puppet-master[68168]:    (at /usr/local/etc/puppet/environments/production/modules/apt/manifests/update.pp:17)

manifests/update.pp

index f3871c09d2a26bc42032aec3cfe61847b5abe32a..4b05c21e069061c297bb79b1cbf8d8ecbce72a99 100644 (file)
@@ -14,7 +14,7 @@ class apt::update {
     'daily': {
       #compare current date with the apt_update_last_success fact to determine
       #if we should kick apt_update.
-      $daily_threshold = (Integer(strftime('%s')) - 86400)
+      $daily_threshold = (Integer(Timestamp().strftime('%s')) - 86400)
       if $::apt_update_last_success {
         if $::apt_update_last_success + 0 < $daily_threshold {
           $_kick_apt = true
@@ -29,7 +29,7 @@ class apt::update {
     'weekly':{
       #compare current date with the apt_update_last_success fact to determine
       #if we should kick apt_update.
-      $weekly_threshold = (Integer(strftime('%s')) - 604800)
+      $weekly_threshold = (Integer(Timestamp().strftime('%s')) - 604800)
       if $::apt_update_last_success {
         if ( $::apt_update_last_success + 0 < $weekly_threshold ) {
           $_kick_apt = true