Uploaded image for project: 'Puppet'
  1. Puppet
  2. PUP-8377

solaris 10 doesn't show raise from custom fact because module expects openssl in the path



    • Bug
    • Status: Closed
    • Normal
    • Resolution: Fixed
    • PUP 5.3.3
    • PUP 5.3.5, PUP 5.4.0
    • None
    • None
      • update the module to always raise the error
      • update the test to run on solaris 10
    • Platform Core
    • Platform Core KANBAN
    • Automated Test
    • Not Needed
    • No Action


      The test tests/i18n/modules/puppet_apply_unsupported_lang.rb is intended to verify that we can get all warning and error messages when we ask for an unsupported language on the system. In this case even if you don't ask for a different language then the default, we don't see the 'raise' message we're expecting from the custom fact:

            pfuqyhk1tw6udul.delivery.puppetlabs.net (solaris10-64-1) 08:49:51$ puppet apply -e "class { 'i18ndemo': filename => '/tmp/provider.LpaqLb' }"
              Warning: This method is deprecated, please use the stdlib validate_legacy function,
                                  with Stdlib::Compat::Absolute_Path. There is further documentation for validate_legacy function in the README. at ["/etc/puppetlabs/code/environments/production/modules/i18ndemo/manifests/init.pp", 15]:
                 (at /etc/puppetlabs/code/environments/production/modules/stdlib/lib/puppet/functions/deprecation.rb:28:in `deprecation')
              Warning: Scope(Class[I18ndemo]): Creating an i18ndemo file
              Notice: Compiled catalog for pfuqyhk1tw6udul.delivery.puppetlabs.net in environment production in 0.06 seconds
              Warning: /Stage[main]/I18ndemo/File[/tmp/provider.LpaqLb]: Ensure set to :present but file type is directory so no content will be synced
              Notice: Applied catalog in 0.02 seconds

      The problem is that module is getting the openssl version available (which there isn't one in the path on the solaris 10 boxes). Since its its not set we don't raise.

      opensslv = Facter::Util::Resolution.exec('openssl version')
          raise _('i18ndemo_fact: this is a raise from a custom fact from eputnam-i18ndemo') if opensslv

      We should change the module to always raise an error and re-enable the test on solaris10




            eric.delaney Eric Delaney
            eric.delaney Eric Delaney
            0 Vote for this issue
            3 Start watching this issue



              Zendesk Support