Issue #4243 has been updated by Markus Roberts. Status changed from Needs more information to Accepted Assigned to changed from Brice Figureau to Markus Roberts Priority changed from Normal to High
As I worked through my reasoning again I saw that you are correct. This is a race condition and should be observable even under MRI's green threads. Another #4211 instance. I think the simplest fix for the present would be to have the import method in parser support use the type collection rather than its loader to do the deed, but this may require a bit of fudging to pass the name of the current file in (presumably for error reporting). I'll play with it. ---------------------------------------- Bug #4243: Multithread issue with manual manifest import http://projects.puppetlabs.com/issues/4243 Author: Brice Figureau Status: Accepted Priority: High Assigned to: Markus Roberts Category: compiler Target version: 2.6.0 Affected version: 2.6.0rc3 Keywords: Branch: This might be a dup of #4211. Manually importing a manifest with the "import" function ends up calling Puppet::Parser::TypeLoader.import which is not thread-safe (ie it tests and sets @imported[file] and parse non atomically). -- 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.
