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

scheduled_task : Remove spec folder from shipped module

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Normal
    • Resolution: Fixed
    • None
    • None
    • scheduled_task
    • Hide

      The packaged tarball no longer includes a spec directory

      Show
      The packaged tarball no longer includes a spec directory
    • Windows
    • 1
    • Windows 2018-08-15
    • Needs Assessment
    • Needs Assessment

    Description

      The module contains extraneous code that is not necessary in the spec folder.

       

      In particular, some legacy code that has been removed from Puppet 6 was restored to the module, solely to confirm compatibility between the old provider on the v1 COM APIs and the new providers which use v2 APIs.  That code was migrated in as part of https://github.com/puppetlabs/puppetlabs-scheduled_task/pull/55

       

      I'm not sure if the Puppet loader searches the spec directory when loading code (when module is installed via PMT or pluginsync methods), but we probably don't want the legacy code to still be available in Puppet 6. There shouldn't be any complications to having this code available, but it would be better to have it fully removed.

       

      An easy solution here should be to remove the spec directory from the shipped module tarball as part of the packaging step.

       

      Attachments

        Activity

          People

            Unassigned Unassigned
            ethan Ethan Brown
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Zendesk Support