Uploaded image for project: 'Modules'
  1. Modules
  2. MODULES-8346

puppetlabs-iis : iis_application/physicalpath is case sensitive

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: IIS 4.5.0
    • Fix Version/s: None
    • Component/s: iis, windows
    • Labels:
    • Template:
      MODULES Bug Template
    • Acceptance Criteria:
      Hide

      The physicalpath parameter of iis_application should do case insensitive path comparisons.

      Other types in the module that have a physicalpath parameter, or a virtual_directory parameter, should be evaluated to see if this bug also manifests with them and if so, should also be fixed.

      If other types and their parameters need to be fixed, consider spinning off additional tickets for each fix to keep the story points per ticket low enough to fit in a single sprint.

      Puppet should not erroneously report changes made on a run where paths are already correct and no changes are actually getting made.

      Show
      The physicalpath parameter of iis_application should do case insensitive path comparisons. Other types in the module that have a physicalpath parameter, or a virtual_directory parameter, should be evaluated to see if this bug also manifests with them and if so, should also be fixed. If other types and their parameters need to be fixed, consider spinning off additional tickets for each fix to keep the story points per ticket low enough to fit in a single sprint. Puppet should not erroneously report changes made on a run where paths are already correct and no changes are actually getting made.
    • Epic Link:
    • Team:
      Windows
    • Story Points:
      1
    • Sprint:
      Windows 2018-12-19
    • QA Risk Assessment:
      Needs Assessment

      Description

      Basic Info
      Module Version: 4.5.0
      Puppet Version: 5.x
      OS Name/Version: windows 2012 R2

      When attempting to manage the physicalpath of an iis_application, the drive letter for the current value of path is compared case sensitively to the value that Puppet is enforcing.

      Testing will be required to determine if the entire path comparison is case sensitive, or just the drive letter.

      This behavior is unexpected for Windows users who expect case insensitive comparisons for drive paths. Since ruby is case sensitive even on Windows, some additional safeguards will have to put in place to ensure comparisons are done in the manner Windows users expect.

      The result is that even though the path in the application is most likely correct, Puppet thinks the value is different and attempts to make a change. The PowerShell command that results is most likely not actually changing anything as the value is already correct, but the below message is seen with each Puppet run.

      Notice: /Stage[main]/Profiles::Iis/Iis_application[Api]/physicalpath: physicalpath changed 'C:\inetpub\wwwroot\Api' to 'c:\inetpub\wwwroot\Api'
      Notice: Applied catalog in 9.76 seconds
      

      Desired Behavior:

      Puppet should compare path values case insensitively, and not attempt to change a path value that is already correct.

      Actual Behavior:
      Puppet compares case sensitively and erroneously believe a path is incorrect and needs to be updated.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              william.hurt William Hurt
              Reporter:
              romgo Hugo
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Zendesk Support