Modules PRs that require review

30 Days 60 Days 90 Days All
Minimum age:
Maximum age:
repo pr age owner title last_comment by age_comment num_comments no_comment_from_puppet last_comment_mentions_puppet
device_manager 86 70 tkishel Update for Puppet 7 compability Hello Friend! This was in response to a post in the puppet Slack channel. It should close Issue 85 (once someone builds and uploads this module to the Forge!) tkishel 70 3 false false
puppetlabs-apache 2207 85 hahnx117 Add multiple directories to userdir.conf [apache::mod::userdir](https://github.com/puppetlabs/puppetlabs-apache/blob/5952b47d529d8eaf952ced2ea66fb999abd5931e/manifests/mod/userdir.pp) is a _class_ ---------------------------------- that may have no external impact to Forge modules. #### This module is declared in 172 of 578 indexed public `Puppetfiles`. --------------------------------- These results were generated with [Rangefinder](https://github.com/puppetlabs/puppet-community-rangefinder), a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report. Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only. puppet-community-rangefinder[bot] 85 2 true false
puppetlabs-apache 2210 64 Tamerz Allow adding 'ProxyPreserveHost' to vhost location [apache::vhost](https://github.com/puppetlabs/puppetlabs-apache/blob/d1200028427e09349704ca003ac3e5f70c876ead/manifests/vhost.pp) is a _type_ ---------------------------------- that may have no external impact to Forge modules. #### This module is declared in 172 of 578 indexed public `Puppetfiles`. --------------------------------- These results were generated with [Rangefinder](https://github.com/puppetlabs/puppet-community-rangefinder), a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report. Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only. puppet-community-rangefinder[bot] 64 1 true false
puppetlabs-docker 788 69 PolaricEntropy Make RedHat version check respect acknowledge_unsupported_os [docker](https://github.com/puppetlabs/puppetlabs-docker/blob/248cbeb41152515e9a993cda64ae24b0877f22d9/manifests/init.pp) is a _class_ ---------------------------------- that may have no external impact to Forge modules. #### This module is declared in 6 of 578 indexed public `Puppetfiles`. --------------------------------- These results were generated with [Rangefinder](https://github.com/puppetlabs/puppet-community-rangefinder), a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report. Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only. puppet-community-rangefinder[bot] 69 2 true false
puppetlabs-docker 787 73 Vincevrp Fix missing comma in docker::image example [![CLA assistant check](https://cla-assistant.io/pull/badge/signed)](https://cla-assistant.io/puppetlabs/puppetlabs-docker?pullRequest=787) All committers have signed the CLA. CLAassistant 73 1 true false
puppetlabs-firewall 1023 83 kjetilho fix service port number lookup to use protocol Hrm, looks like the test case "'040 partial invert" needs adjustment? It now bombs since "http/udp" does not exist in the test harness (it does exist on my Fedora!). This error was hidden earlier since it looked up "http/tcp". kjetilho 83 2 true false
puppetlabs-firewall 1013 170 onyxmaster Fix rpfilter parameter Well, it looks like I fixed the problem with CentOS 6 by adding proper version checks. @adrianiurca, could you please take a look? onyxmaster 70 3 true false
puppetlabs-kubernetes 540 69 treydock Remove cgroup-driver arg to avoid deprecation warnings [![CLA assistant check](https://cla-assistant.io/pull/badge/signed)](https://cla-assistant.io/puppetlabs/puppetlabs-kubernetes?pullRequest=540) All committers have signed the CLA. CLAassistant 69 1 true false
puppetlabs-kubernetes 542 67 nickperry Fix #541 > Hi @nickperry, can you sign the CLA, please? Thank you! I have done previously. I signed it again a couple of days ago but it keeps resetting. Not sure what I can do. nickperry 63 4 true false
puppetlabs-postgresql 1315 83 vaol Fix table grant with schema [![CLA assistant check](https://cla-assistant.io/pull/badge/signed)](https://cla-assistant.io/puppetlabs/puppetlabs-postgresql?pullRequest=1315) All committers have signed the CLA. CLAassistant 75 2 true false
puppetlabs-stdlib 1205 180 smortex vox tc Fix serialization of undef in to_python() @hlindberg may I ask you to check the last commit to confirm it is what you expected? It looks a bit useless to me to have support for this so I might have missed something? Maybe accepting `Data` instead of `Any` would make more sense? ```sh-session romain@zappy ~/Projects/puppetlabs/puppetlabs-stdlib % cat tset.pp type Car = Object[attributes => {regnbr => String}] $x = [default, Timestamp(), /.*/, Car(regnbr => 'abc123'), Integer[0,10], Deferred('f', [1,2,3]), Resource['file', '/tmp/tmp.txt'],3] warning($x.to_python) warning($x.to_ruby) romain@zappy ~/Projects/puppetlabs/puppetlabs-stdlib % puppet apply -t --modulepath spec/fixtures/modules tset.pp Info: Loading facts Warning: Scope(Class[main]): [{"__ptype": "Default"}, {"__ptype": "Timestamp", "__pvalue": "2021-10-11T20:15:28.626940753 UTC"}, {"__ptype": "Regexp", "__pvalue": ".*"}, {"__ptype": "Car", "regnbr": "abc123"}, {"__ptype": "Pcore::IntegerType", "from": 0, "to": 10}, {"__ptype": "Deferred", "name": "f", "arguments": [1, 2, 3]}, {"__ptype": "Pcore::ResourceType", "type_name": "File", "title": "/tmp/tmp.txt"}, 3] Warning: Scope(Class[main]): [{"__ptype" => "Default"}, {"__ptype" => "Timestamp", "__pvalue" => "2021-10-11T20:15:28.626940753 UTC"}, {"__ptype" => "Regexp", "__pvalue" => ".*"}, {"__ptype" => "Car", "regnbr" => "abc123"}, {"__ptype" => "Pcore::IntegerType", "from" => 0, "to" => 10}, {"__ptype" => "Deferred", "name" => "f", "arguments" => [1, 2, 3]}, {"__ptype" => "Pcore::ResourceType", "type_name" => "File", "title" => "/tmp/tmp.txt"}, 3] Notice: Compiled catalog for zappy.blogreen.org in environment production in 0.02 seconds Info: Using environment 'production' Info: Applying configuration version '1633983328' Notice: Applied catalog in 0.02 seconds romain@zappy ~/Projects/puppetlabs/puppetlabs-stdlib % ``` I will then rework the branch to take your feedback into account and split the `to_ruby` changes in another PR. Thanks! smortex 87 6 true false