Module triage summary

[Ashley Penney]

Just wanted to send out our notes for the week, it's a bit messy in terms of formatting, you can see the original at https://etherpad.openstack.org/p/module_hangout_2014-04-3

Puppet Module Hangout   2014-04-03

Attended: ashp, hunner, igalic, blkperl, lauren, michelle, ryan, miguel, phil

puppetlabs_spec_helper

      - New release happening today

      

      #46 Add support for "forge_modules" in fixtures - merged

Concat:  - https://github.com/puppetlabs/puppetlabs-concat/pulls

     - New released wanted by ashp

    

• #152 ignore mode and owner on windows                                                    -   waiting for tests

• #150 added double quotes to paths in concat command                            -   merged

• #123 ensure that numbers are also valid as prefix                                       -  rejected

• #121 [Most of] variables quoted to allow multi-word warning messages  - merged

     #119 Adds a noop_mode to concat::setup                                                       - no tests / closed

     #99 [Issue_69] Prevent cat globbing if filenames contains wildcards         - no tests / closed

     

Corosync  - https://github.com/puppetlabs/puppetlabs-corosync/pulls

   - rename module to pacemaker

   - document parameters

    #62 provide param to overwrite the list of packages which should be installed - merged

    #60 Consider <instance_attributes/> within primitive operations                          - merged

    #59 fix depreciation warning in latest puppet versions                                            -  merged

   #56 more than two primitives per cs_colocation                                                        - merged

   #51 Added caching on self.ready                                                                                 - needs rebase

   #50 Allow ordering of cs_groups                                                                                  -  merged

   #49 Added symmetrical parameter on cs_order                                                        - merged

   #47 Reworked cs_shadow (fixes #15)                                                                         - needs work

   Remaining PRs need rebase

   

   DHCP  - https://github.com/puppetlabs/puppetlabs-dhcp/pulls

  - skipped Hunner did a lot of the work earlier

Firewall - https://github.com/puppetlabs/puppetlabs-firewall/pulls

   - rename module to iptables

   - have a firewall module which does common tasks such as  port forwarding and calls the iptables module, but also ipfw, pf, or other firewall modules.

  #340   --reap flag is not added to iptables command                            - merged

  #339   Fix typo in SNAT error message                                                   - merged

  #338   Treat RHEL 7 and later like Fedora w/r/t iptables                      - merged

 #337   Add support for mac address source rules pt2                           - needs tests

 #335  WIP Firewallchain cache fw rules                                                  - needs work / closed

 #334 Add support for mac address source rules                                    - needs work / closed

 #333 cache retrieved firewall rules                                                           - needs work / closed

 #295 (MODULES-443) Deprecate unset proto                                       - needs work

 #279 Fedora commands needed changed to support the systemctl  - closed / fixed in #338

 #274 Include additional valid ctstate parameters.                                  - needs tests

 #179 WIP: New iptables_ng provider with better parsing support       - skipped

 #176 UNIT_TESTS_FAIL: Fedora 18 support                                          - closed

 

 Git - https://github.com/puppetlabs/puppetlabs-git/pulls

#22 Get source_dir from html_path                - merged

#21 Install git-subtree documentation           - merged

#20 Fix git-subtree's vcsrepo source             - merged

#19 Install git-subtree documentation           - needs work / no docs / closed

#18 Add user support for git configuration   - needs docs / comments left

#14 Check package git is defined                 - closed

#9 add version for vcsrepo dependency       - merged

#5 Spec tests                                                     - needs rebase

inifile - https://github.com/puppetlabs/puppetlabs-inifile/pulls

#92 Added support for keep_secret parameter  - comments left

#71 Create resources                                              - needs work

vcsrepo - https://github.com/puppetlabs/puppetlabs-vcsrepo/pulls

#129 Ensure the repository does not differ from the provided revision            - needs work / changes default behavior

#127 Avoid checkout failure due to untracked files                                             - same as above

#124 Fixing two bugs with git and svn                                                                    - needs tests

#123 Rebased b11326d - tmp_dir param for systems with noexec on /tmp   - needs rebase / docs

#121 Only add ssh options to commands that actually talk to the network.    - merged

#120 Use `git rev-parse` to get tag canonical revision                                        - merged

apt  - https://github.com/puppetlabs/puppetlabs-apt/pulls

#281 unattended-upgrades: Fix origins for Squeeze  - merged

#280 Fix pin comments                                                      - merged

postgres - https://github.com/puppetlabs/puppetlabs-postgresql/pulls

#297- quoting fix

--

Ashley Penney

ashley...@puppetlabs.com

Module Engineer

Join us at PuppetConf 2014, September 23-24 in San Francisco - http://puppetconf.com