Issue #947 has been updated by luke.

Status changed from Needs more information to Ready for Checkin
Assigned to changed from community to jamtur01
Target version changed from 0.25.0 to 0.24.6

Okay, I've fixed this in tickets/0.24.x/947 in my repo.

Note that I had to include the fix to the Node environment validation from 
tickets/0.24.x/1614-2 to get the correct failure.
----------------------------------------
Bug #947: pluginsync failure: can't convert nil into String in 
lib/puppet/network/handler/fileserver.rb:797
http://projects.reductivelabs.com/issues/show/947

Author: DavidS
Status: Ready for Checkin
Priority: High
Assigned to: jamtur01
Category: plumbing
Target version: 0.24.6
Complexity: Unknown
Patch: Tests
Affected version: 
Keywords: 


When trying to pluginsync with default parameters, current HEAD 
([8eecbe54c96cec0de492e7ae77211637b65057e8]) puppetmaster fails with this trace:

<pre>
debug: mount[plugins]: Listing /plugins for ic.black.co.at
./lib/puppet/network/handler/fileserver.rb:797:in @join'
./lib/puppet/network/handler/fileserver.rb:797:in @file_path'
./lib/puppet/network/handler/fileserver.rb:807:in @reclist'
./lib/puppet/network/handler/fileserver.rb:730:in @list'
./lib/puppet/network/handler/fileserver.rb:152:in @list'
/usr/share/rails/activerecord/lib/../../activesupport/lib/active_support/dependencies.rb:147:in
 @to_proc'
./lib/puppet/network/xmlrpc/processor.rb:52:in @call'
./lib/puppet/network/xmlrpc/processor.rb:52:in @protect_service'
./lib/puppet/network/xmlrpc/processor.rb:85:in @setup_processor'
/usr/lib/ruby/1.8/xmlrpc/server.rb:336:in @call'
/usr/lib/ruby/1.8/xmlrpc/server.rb:336:in @dispatch'
/usr/lib/ruby/1.8/xmlrpc/server.rb:323:in @each'
/usr/lib/ruby/1.8/xmlrpc/server.rb:323:in @dispatch'
/usr/lib/ruby/1.8/xmlrpc/server.rb:366:in @call_method'
/usr/lib/ruby/1.8/xmlrpc/server.rb:378:in @handle'
./lib/puppet/network/xmlrpc/processor.rb:44:in @process'
./lib/puppet/network/xmlrpc/webrick_servlet.rb:68:in @service'
/usr/lib/ruby/1.8/webrick/httpserver.rb:104:in @service'
/usr/lib/ruby/1.8/webrick/httpserver.rb:65:in @run'
/usr/lib/ruby/1.8/webrick/server.rb:173:in @start_thread'
/usr/lib/ruby/1.8/webrick/server.rb:162:in @start'
/usr/lib/ruby/1.8/webrick/server.rb:162:in @start_thread'
/usr/lib/ruby/1.8/webrick/server.rb:95:in @start'
/usr/lib/ruby/1.8/webrick/server.rb:92:in @each'
/usr/lib/ruby/1.8/webrick/server.rb:92:in @start'
/usr/lib/ruby/1.8/webrick/server.rb:23:in @start'
/usr/lib/ruby/1.8/webrick/server.rb:82:in @start'
./lib/puppet.rb:336:in @start'
./lib/puppet.rb:187:in @newthread'
./lib/puppet.rb:186:in @initialize'
./lib/puppet.rb:186:in @new'
./lib/puppet.rb:186:in @newthread'
./lib/puppet.rb:334:in @start'
./lib/puppet.rb:333:in @each'
./lib/puppet.rb:333:in @start'
./bin/puppetmasterd:281
err: Could not call: can't convert nil into String
</pre>

I have prodded around the <pre>PluginMount</pre> class and I think the basic 
problem is that the data structure returned by <pre>Mount.list</pre> is *beep*. 
This amongst other things causes <pre>relpath = abspath.sub(%r{^#{basepath}}, 
_)</pre> in 
<pre>lib/puppet/network/handler/fileserver.rb:PluginMount:reclist()</pre> to 
not work in all cases, especially when the basepath changes from one module to 
the next.

I have created a little patch that cleans up the [[PluginMount]] class by 
overriding <pre>list</pre> instead of <pre>reclist</pre> and calling the 
original <pre>reclist</pre> for each module in turn. This surely needs more 
love to remove duplicates, but it does work on my puppetmaster now. Please find 
the patch in the <pre>rest/fix-plugins-mount</pre> branch of my repo at 
git://git.black.co.at/puppet-bugfixes



----------------------------------------
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://reductivelabs.com/redmine/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