Issue #2552 has been updated by Luke Kanies. Status changed from Needs design decision to Needs more information Assigned to changed from Luke Kanies to Markus Roberts
I'm not sure this will cover all the bits you need, but... Unfortunately, our boolean parameters use :true and :false instead of the actual Ruby booleans. It's one of those things I've just never tracked down and is ripe for fixing after this release. So, you need to keep the symbol rather than switching it a boolean. (I just confirmed manually that we get :true when we set hasstatus to 'true'.) However, I don't think that has anything to do with the problem. I think it's all because of the recent changes in the redhat service provider. Run this: <pre> git diff 4f2c066a97e59a89df64af4b25beac6f3f0553c2 lib/puppet/provider/service/redhat.rb </pre> and look at the output with your newfound understanding of this provider structure. Basically, I think the latest fix -- in b1ffffada99ce12b64bed7f93d4352317d1263b9 -- caused this problem. I've asked James to update the ticket with the reasoning for the commit, but the code is certainly wrong now -- the 'restart' method should call execute somehow, but it instead returns an array, which it shouldn't do. I'm leaving this open, but I'm nearly positive an updated fix for #2525 will fix this. ---------------------------------------- Bug #2552: execute fails http://projects.reductivelabs.com/issues/2552 Author: Sam Rowe Status: Needs more information Priority: High Assigned to: Markus Roberts Category: service Target version: 0.25.0 Complexity: Unknown Affected version: 0.25.0rc1 Keywords: Running a 299eadb7d903d250c8340049bd78611f0c705920 RHEL 5 client against a server running with Markus' branch which fixes #2493 (ffe6bb837b56ecaf5a50f26550c0e11c707345dc). We get a lot of these: <pre> /usr/lib/site_ruby/1.8/puppet/util.rb:250:in `execute' /usr/lib/site_ruby/1.8/puppet/provider/service/base.rb:123:in `texecute' /usr/lib/site_ruby/1.8/puppet/provider/service/base.rb:137:in `ucommand' /usr/lib/site_ruby/1.8/puppet/provider/service/base.rb:64:in `status' /usr/lib/site_ruby/1.8/puppet/type/service.rb:68:in `retrieve' /usr/lib/site_ruby/1.8/puppet/type.rb:804:in `currentpropvalues' /usr/lib/site_ruby/1.8/puppet/provider/nameservice.rb:242:in `inject' /usr/lib/site_ruby/1.8/puppet/type.rb:802:in `each' /usr/lib/site_ruby/1.8/puppet/type.rb:802:in `inject' /usr/lib/site_ruby/1.8/puppet/type.rb:802:in `currentpropvalues' /usr/lib/site_ruby/1.8/puppet/type.rb:790:in `retrieve' /usr/lib/site_ruby/1.8/puppet/type.rb:719:in `evaluate' /usr/lib/site_ruby/1.8/puppet/transaction.rb:60:in `apply' /usr/lib/site_ruby/1.8/puppet/transaction.rb:249:in `eval_children_and_apply_resource' /usr/lib/site_ruby/1.8/puppet/util.rb:425:in `thinmark' /usr/lib/ruby/gems/1.8/gems/activesupport-2.1.0/lib/active_support/core_ext/benchmark.rb:8:in `realtime' /usr/lib/site_ruby/1.8/puppet/util.rb:424:in `thinmark' /usr/lib/site_ruby/1.8/puppet/transaction.rb:248:in `eval_children_and_apply_resource' /usr/lib/site_ruby/1.8/puppet/transaction.rb:205:in `eval_resource' /usr/lib/site_ruby/1.8/puppet/transaction.rb:294:in `evaluate' /usr/lib/site_ruby/1.8/puppet/util.rb:425:in `thinmark' /usr/lib/ruby/gems/1.8/gems/activesupport-2.1.0/lib/active_support/core_ext/benchmark.rb:8:in `realtime' /usr/lib/site_ruby/1.8/puppet/util.rb:424:in `thinmark' /usr/lib/site_ruby/1.8/puppet/transaction.rb:293:in `evaluate' /usr/lib/site_ruby/1.8/puppet/transaction.rb:287:in `collect' /usr/lib/site_ruby/1.8/puppet/transaction.rb:287:in `evaluate' /usr/lib/site_ruby/1.8/puppet/resource/catalog.rb:139:in `apply' /usr/lib/site_ruby/1.8/puppet/configurer.rb:142:in `run' /usr/lib/site_ruby/1.8/puppet/util.rb:180:in `benchmark' /usr/lib/ruby/gems/1.8/gems/activesupport-2.1.0/lib/active_support/core_ext/benchmark.rb:8:in `realtime' /usr/lib/site_ruby/1.8/puppet/util.rb:179:in `benchmark' /usr/lib/site_ruby/1.8/puppet/configurer.rb:141:in `run' /usr/lib/site_ruby/1.8/puppet/agent.rb:53:in `run' /usr/lib/site_ruby/1.8/puppet/agent/locker.rb:21:in `lock' /usr/lib/site_ruby/1.8/puppet/agent.rb:53:in `run' /usr/lib/ruby/1.8/sync.rb:229:in `synchronize' /usr/lib/site_ruby/1.8/puppet/agent.rb:53:in `run' /usr/lib/site_ruby/1.8/puppet/agent.rb:130:in `with_client' /usr/lib/site_ruby/1.8/puppet/agent.rb:51:in `run' /usr/lib/site_ruby/1.8/puppet/application/puppetd.rb:103:in `onetime' /usr/lib/site_ruby/1.8/puppet/application.rb:226:in `send' /usr/lib/site_ruby/1.8/puppet/application.rb:226:in `run_command' /usr/lib/site_ruby/1.8/puppet/application.rb:217:in `run' /usr/lib/site_ruby/1.8/puppet/application.rb:306:in `exit_on_fail' /usr/lib/site_ruby/1.8/puppet/application.rb:217:in `run' /usr/sbin/puppetd:159 err: //redhat/Service[gpm]: Failed to retrieve current state of resource: Must pass an array to execute() </pre> -- 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 -~----------~----~----~----~------~----~------~--~---
