Issue #5789 has been updated by Jeff McCune.

Status changed from Tests Insufficient to Merged - Pending Release
Assignee changed from Jeff McCune to Michael Stahnke

# Merged #

Talking with Gary, this code and tests have already been merged into the 
integration branch as per <https://github.com/puppetlabs/puppet/pull/139>

The merge commit is: 
<https://github.com/puppetlabs/puppet/commit/db403ddc51d83e5e0001a3e62b0989e8e97d9f96>

Jacob, Nigel, Haus and Gary worked together on this pull request so I believe 
Gary got the engineering help he needed on the tests.

I'm assigning this to Mike in case it already got released and should instead 
be marked as closed rather than merged - pending release.

Jason, James, I think this covers the items mentioned in this mornings 
engineering standup.

-Jeff
----------------------------------------
Bug #5789: launchd provider parses all plists twice, and uses too much CPU
https://projects.puppetlabs.com/issues/5789

Author: Nigel Kersten
Status: Merged - Pending Release
Priority: Normal
Assignee: Michael Stahnke
Category: provider
Target version: 2.7.x
Affected Puppet version: 
Keywords: mac launchd service provider
Branch: 
https://github.com/glarizza/puppet-1/tree/feature/2.7.x/launchd_optimization


We shouldn't be parsing everything twice, and it appears that piping to 
/dev/stdout is particularly slow on OS X.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en.

Reply via email to