As part of the Puppet dev team's commitment to better communication on the dev list, each week a developer (I'm starting this week) will be responsible for making sure that all patch submissions are responded to in a timely manner. Note I didn't say each patch would be accepted, but there will be a response and the patch won't get ignored. Community contributions are extremely valuable to us and we want to encourage them and make it as easy as possible. Let us know if you've sent a patch that has gone unnoticed for more than a week. At the bottom of this email I've got a small list of recent patches on the list that need review and I'll be starting there in addition to making sure new patches are looked at.
By way of a feeble excuse as to why we've let some patches languish on the list for so long, the most common reason was that nobody felt knowledgeable about the code the patches touched, so the default response was to do nothing and hope someone else knew more. That turned out not to be a good default. Also, to make it easier for people to contribute we'll be updating and cleaning up our development lifecycle documentation (http://projects.puppetlabs.com/projects/puppet/wiki/Development_Development_Lifecycle) based on feedback from the development community. That project has just begun, but if you find something in there that's confusing or hindering your ability to get us some awesome code you've written for Puppet, let us know. Here's the community patches I came across in the recent archives that haven't seen any response or been resolved. I'm going to start looking through these. Again, let us know if there's something else we've missed. If you sent something and nobody's responded in over a week, we messed up. Mailing list: James Cammarata - [Puppet-dev] [PATCH 1/1] Fix for bug #2043 - up2date provider should split off architec... Sep 13 Dean Wilson - Puppet-dev] Possible fix for #4418. Aug 25 Daniel Pittman - [Puppet-dev] [PATCH/puppet 2/2] file: set propertieslike mode before we rename. Aug 22 Daniel Pittman - [Puppet-dev] [PATCH/puppet 1/2] file: use tempfilerather than reinventing that particu... Aug 22 Daniel Pittman - [Puppet-dev] [PATCH/puppet] Improve safety of thefile type against disasters. Aug 22 Bryan Kearney - Puppet-dev] Support for external certificate keys in the agent. July 12 Thomas S Hatch - [Puppet-dev] [PATCH/puppet 1/1] Add new type andprovider for sysctl, ticket #4068 Jun 27 Stéphan Gorge - Features #2198 - Combine package installation. June 22 Another area where we may have received patches that we haven't reviewed is the ticketing system. Here's query results Markus got to show tickets that have patches that may need further examination. We'll start working through this backlog as well, but after the more recent mailing list submissions. Ticket System: 110 Accepted 0001-Fix-to-allow-nested-arrays-in-resource-parameters.patch 219 Accepted locking.patch 370 Accepted 0001-Fixed-370.patch 511 Accepted fix-511.patch 738 Needs more information puppet-0.23.0-gentoo-init-try1.diff 745 Code Insufficient basic-syslog-tests.patch 772 Accepted privkeypatch 913 Accepted up2date.patch 1061 Tests Insufficient diff 1100 Tests Insufficient patch 1203 Accepted leopard_puppet_fix_042308.patch 1220 Accepted authstore.rb.patch 1276 Code Insufficient 0001-Default-service-pattern-match-with-word-boundaries.patch 1361 Accepted puppet_autosign_allow_deny.diff 1519 Accepted 3371-If-lite-is-not-specified-we-hand-the-reading-of-the.patch 1590 Rejected puppet.patch 1597 Accepted gem_install_source_hack.diff 1658 Code Insufficient gentoo_package_provider.patch 1728 Accepted cron.patch 1732 Tests Insufficient 0004-Reduce-P-U-Settings-surface-area-and-remove-sett.patch 1798 Needs design decision puppet-0.24.6-purgeservices.patch 1856 Needs design decision 0001-Fix-service-shutdown-issue-in-redhat.patch 1858 Rejected puppet_hash_resources.patch 2043 Accepted up2date.rb.udiff 2120 Re-opened virtual.vmware.patch 2135 Tests Insufficient global_expire.patch 2156 Accepted patch 2160 Accepted 0001-implement-Smart-package-provider.patch 2193 Accepted exec_list.patch 2211 Accepted autoload-regression-diff 2255 Code Insufficient diff 2331 Accepted darwinport.rb.patch 2350 Accepted 0001-Aptitude-provider-makes-a-dry-run-to-ensure-version.patch 2359 Accepted 0001-Fixes-2359-Capture-stderr-from-exec.patch 2547 Rejected manifest.pp.patch 2721 Needs more information facter-1.5.7.patch 2722 Accepted puppet-0.25.0-slapt-get.patch 2747 Accepted facter-1.5.7-1-virtual.patch 2749 Accepted puppet-netbsd-support.diff 2804 Investigating 0001-added-uninstall-capabilities.patch 2819 Code Insufficient 0003-Fixed-a-bug-where-sometimes-Apps-are-not-installed-b.patch 2820 Code Insufficient 0001-added-quicklook-provider.patch 2891 Code Insufficient base.rb.diff 2917 Rejected puppet-checksums-speedup.patch 3004 Code Insufficient puppet_3004.diff 3033 Investigating 0001-Use-a-pipe-instead-of-a-temp-file-for-command-output.patch 3087 Needs more information facter-esx.diff 3136 Needs more information 0001-Load-the-plugins-type-first.patch 3168 Investigating puppet-use_crl.patch 3323 Needs design decision puppet_hasreload.diff 3420 Investigating naginator.rb.patch 3713 Needs design decision ipaddress.patch 3730 Needs design decision puppet-yumhelper-plugins.patch 3962 Accepted Add_Varparent_to_create_vardir_parent.patch 3984 Investigating selinux-warnings.patch 3986 Needs more information puppet-provider-apt-patch 4508 Accepted virtual.rb.diff 4630 Rejected file.rb_environment.patch 4729 Needs design decision reporting_patch 4746 Accepted freebsd_fix_newlines_in_rc.conf.patch -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
