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

$settings leaks symbols into manifests

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: PUP 4.6.1
    • Fix Version/s: PUP 4.8.0
    • Component/s: None
    • Labels:
      None
    • Template:
    • Team:
      Puppet Developer Experience
    • Story Points:
      1
    • Sprint:
      PDS 2016-09-21, PDS 2016-10-05
    • Release Notes:
      Bug Fix
    • Release Notes Summary:
      Hide
      A problem was fixed regarding settings values set in variables in the 'settings' namespace where settings that internally used Ruby Symbols leaked those into the Puppet Language making it impossible to compare them with String values. Now, those settings (as well as all others not having a Puppet Language representation) are turned into values that can be used.
      Show
      A problem was fixed regarding settings values set in variables in the 'settings' namespace where settings that internally used Ruby Symbols leaked those into the Puppet Language making it impossible to compare them with String values. Now, those settings (as well as all others not having a Puppet Language representation) are turned into values that can be used.

      Description

      % puppet apply -e '$be = $settings::storeconfigs_backend notice(inline_template("<%= @be.class %>"))'
      Notice: Scope(Class[main]): Symbol
      

      and therefore

      % puppet apply -e 'notice($settings::storeconfigs_backend == "puppetdb")'
      Notice: Scope(Class[main]): false
      

      This stuff should be stringified I guess, and in general $settings is just horrible and should be a hash.

        Attachments

          Issue Links

            Activity

              jsd-sla-details-panel

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  ripienaar R.I.Pienaar
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: