voxpupuli / puppet-augeasproviders_grub

Augeas-based grub type and providers
Apache License 2.0
9 stars 33 forks source link

Issue w/Puppet 2016.x #24

Closed acraw1 closed 2 years ago

acraw1 commented 8 years ago

The below puppet code worked fine on puppet 3.8, but it fails on puppet 2016.x. The error message produced is also shown below. I tried setting the timeout value with and without quotes, but I get the same error either way.

Error: no implicit conversion of Fixnum into String
Error: /Stage[main]/Profiles::Grub/Grub_config[timeout]/value: change from 5 to 10 failed: no implicit conversion of Fixnum into String
class profiles::grub (
  $menu_timeout = '10',
) {

  case $::osfamily {
    'RedHat': {
      case $::operatingsystemmajrelease {
        '6': {
          grub_config { 'timeout':
            value => $menu_timeout,
          }
        }
        '7': {
          grub_config { 'GRUB_TIMEOUT':
            value => $menu_timeout,
          }
        }
        default: {
          fail('Need to define the Grub configuration for this version of the OS')
        }
      }
    }
    default: {
      notify { 'Grub configuration not defined for this operating system': }
    }
  }

}
trevor-vaughan commented 8 years ago

@acraw1 Thanks for the report. We'll take a look at this and get back to you.

jhoblitt commented 2 years ago

PE 2016 is long EOL.