Attending: JoshP, Michael, Ethan, Felix Short triage this week while working on 3.7 release.
New: Facter 763: (CPR-5) Move dmidecode from dependency to recommends - Just needs to reference FACT-467\ - Michael updated commit message and merged 751: (FACT-651) added KVM detection for SunOS - Will is working with the contributor; waiting on merge until after Facter release Puppet: 2997: (PUP-3094) - Adding separate binaries for SELinux - JoshP commenting that this seems like more of a packaging issue; pinged Haus. 2986: Added yaml output to puppet resource #2986 - Pinged Henrik and Eric to get their opinions; Eric is discussing it with contributor in the PR. - If we move forward, would need a ticket and specs. 2978: (PUP-3066) Services are started before flags are set #2978 - Asked for a spec Held Over Facter: 748: Fact-639 Allow any kind of hypervisor for ec2 facts - Kylo and Adrien are talking with the contributor - Some concerns that the specs are incomplete (not checking enough cases) Felix commenting - Still under discussion 731: Display facts for aliases in *BSD - It looks like the code was written by a different contributor from the GitHub submitter, we need to work out the status of CLAs for the pull request - Code could use some refactoring, cleanup, and tests, but we can work with the contributor on that - Needs a JIRA issue - Still needs work; pending contributor response; check in next week. (2x) - Check in with Adrien about status and whether this should be kept open - Hasn't seen an update in 4 weeks and still needs a fair amount of refactoring and spec work; closing. Puppet: 2866 PUP-1244 puppet4 - Switches the yum provider to use RPM version comparison instead of semantic version comparison - This might be breaking but is more correct and has been requested since Puppet 0.24, document as a breaking change in Puppet 4 - Commit messages need to be fixed up, Andy to handle - Has an error; @jantman will be looking into it this week; check back next week. - Still waiting on contributor for changes; Joshua to ping - Jason has updated the failing case JoshP saw; Joshua to test again - Tests out fine, and has been squashed, but seeing a failure on 1.8.7; JoshP will look at it 2961: PUP-2790 Use telnet to detect upstart version - Some questions for the contributor about how the change fixes PUP-2790 from Felix - Also needs commit fix up and spec and CLA 2952: PUP-3038 Cache search_directories - Ongoing discussion about unexpected impact from this change - Breaking specs - Commit message neds updating to provide ticket # and information about the change - Felix also wondered how this related to PUP-1592 and has updated PUP-3038 to link that 2951: PUP-3038 Cache typeloader misses - Same ticket as 2952, similar concerns 2933: (PUP-1527) After upgrade from 3.3.2-1 to 3.4.2-1 naginator fails to crea... - Commits need squashing - Contributor is having issues signing the CLA, will sort that out. - Waiting to hear back from Jeff about the CLA issue. 2919: Proposal for processing an external trusted OID yaml file - JIRA issue has been filed for this pull request - Needs spec tests, needs refining of the implementation. We'll continue work on this with the contributor. - Check in with Adrien tomorrow 8/14 about picking this up now that he's rotated off of community role 2915: Set accept-encoding to identity when http_compression is false - Looks fine; waiting for one spec name fix from Contributor, and pinged Josh Cooper for any comments, as he was last to mention this code change in the ticket. - Josh commented on this indicating that the puppet module tool will need a similar fix; we don't have enough information on this to proceed, waiting for feedback from Andy. - Need to look into how to handle this for puppet module tool as well 2879: Clarify possible exit codes and behavior with negative exit codes on Windows - Pinging Josh Cooper for additional feedback on doc changes. - Rob to review the doc changes and merge if they look good 2649: Add slot support for portage package provider. - Duplicates GH-2779, decision is to ping both contributors to see if they can coalesce their individual PRs into a single branch. Finch commenting this week. - GH-2779 hasn't seen updates, closing that one in favor of this one. There's some concern about possibly using `:` as a special value in package resource titles but we don't want to go down that route. - Pinged Adrien for a final call on this now that the specs are updated 2683: Add more config manipulation options - Needs a JIRA issue and tests are failing - Would be a good fix to have, if contributor doesn't get back to us might be worthwhile to fixup ourselves. - Adrien asking for a Ticket so we can pull it onto community board and tackle it. - No response/no ticket, Joshua to create ticket - Discussion that we should preserve the current behavior of `puppet config set prop ''` and you should use `puppet config del` to remove the setting and restore the default. JoshP to continue work. 2770: Allow version and held states of a package to specified as separate properties - This is a very big and sticky issue, and might be worth promoting into the platform team because of the wide ranging ramifications - Plan is to discuss how we promote issues like this, and respond to the contributor with our next steps forward - Pending contributor response, willing to wait on this since we're asking a lot of the contributor - Will need to talk with Adrien tomorrow and figure out how to hand this off. 2870 (PUP-19708) Make the puppet agent retry failed requests. - Issue in commit message references projects.puppetlabs.com ticket, actual issue is https://tickets.puppetlabs.com/browse/PUP-2526 - There are a lot of cases where HTTP retries aren't meaningful (catalog retrieval, report submission) - We haven't been able to come to an agreement with the contributor about what requests should be retried and how, and we might not have the resources to make the changes ourselves. - Felix commented in the ticket, some discussion is happening there - Waiting to see if we can come to some agreement in the ticket as to the direction of the change. 2869 Allow Upstart jobs on Amazon Linux - Ticket filed, issues pending on the pull request - Waiting on contributor feedback - Needs a spec and an updated commit message 2820: (PUP-2867) Support ~ in file paths, allows to deploy to ~<user> directories - Already in discussion on puppet-dev - Tests are failing, commits need squashing; waiting for contributor response - What happens when the path starts with `~/` - do we expand to the current user? - Felix clarified the above issue; waiting on contributor 2614: (PUP-2452) Refresh support for Package type - Looks reasonable. - There is a question from the contributor about if there needs to be something changed to use this. - The answer to the question is, "yes, having a use of this is ideal". - Adrien noted that using the :methods system for the feature is hard to control. Better is to require that the provider just explicitly declare the feature, which it already does. - Another concern came up about getting refresh events that will cause unintended reinstalls. Such a case is when a class contains a package and is the target of a notify. - This is a larger change to functionality that we need to bring up on puppet-dev to figure out how to deal with the unintended consequences of adding this functionality. - Adrien to file a ticket on finer grained refresh handling and start a thread on puppet-dev to come to a decision about how to better control refresh events. - Some discussion taking place in the PR, waiting on the contributor for an updated opt-in approach -- Josh Partlow [email protected] Developer, Puppet Labs Join us at PuppetConf 2014, September 20-24 in San Francisco Register by September 8th to take advantage of the Final Countdown —save $149! -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/puppet-dev/CADxAQ5pQwkrz%3DCsKyjqgCdXMH-d--oH0ExsaD1zaFw48vmyVww%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
