Issue #14986 has been updated by Brice Figureau.
Definitely +1 The Thread.current was a kludge I added a long time ago to be able to at least survive to multiple threads compiling at the same time. IMHO, there should be one set of Environment instances per compiler instances in the master, as those are difficult to share. ---------------------------------------- Refactor #14986: Stop using Thread.current for tracking the environment https://projects.puppetlabs.com/issues/14986#change-66211 Author: Andrew Parker Status: Unreviewed Priority: Normal Assignee: Category: Target version: 3.x Affected Puppet version: Keywords: Branch: Puppet::Node::Environment uses Thread.current for tracking the environment. This causes code that is hard to follow and tests that are very fragile. By removing this we should have a more understandable way of dealing with environments. -- 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.
