Details
-
Bug
-
Status: Closed
-
Normal
-
Resolution: Fixed
-
PUP 5.3.3
-
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
Description
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