Project

General

Profile

Actions

Bug #329

closed

Errors parsing RRDs

Added by Eric Shamow over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Puppet Reports
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

This is apparently related to an issue in Puppet --

http://projects.puppetlabs.com/issues/show/3949

Began getting the following for all reports:

Processing ReportsController#create to yml (for 127.0.0.1 at 2010-06-23 18:19:56) [POST]
Parameters: {"format"=>"yml", "action"=>"create", "controller"=>"reports", "report"=>"[FILTERED]"}
processing report for <puppet_client>
Failed to process report for <puppet_client> due to:undefined method `name' for "RRDtool":String
Completed in 31ms (View: 0, DB: 14) | 500 Internal Server Error [http://&lt;puppet_server&gt;/reports/create?format=yml]

Apparently this is due to RRDTool libraries not being loaded -- adding "require 'RRDTool'" in Foreman's environment.rb clears the problem up.

The fix was suggested by Volcane, it's not mine.

Actions #1

Updated by Ohad Levy over 14 years ago

  • Target version set to 0.1-6

hmm... that's actually a puppet bug (effected version is 0.25.5) and is already fixed at http://projects.reductivelabs.com/issues/3949

I guess a quick workaround if foreman is to require RRD with a rescue block (e.g. for the people who don't have it installed).

Actions #2

Updated by Eric Shamow over 14 years ago

Agreed that it's a Puppet bug, but since Foreman is servicing Puppet and users are more likely to have a current Foreman but a slightly older Puppetmaster, I think a rescue block is a good idea -- and especially for those who are having the problem now with 0.25.5.

Actions #3

Updated by Ohad Levy over 14 years ago

  • Category set to Puppet Reports
  • Assignee set to Ohad Levy
Actions #4

Updated by Ohad Levy over 14 years ago

  • Status changed from New to Ready For Testing
  • % Done changed from 0 to 100
Actions #5

Updated by Robin Bowes over 14 years ago

There's a typo in the changeset - it's RRDtool not RRDTool (small t).

Actions #6

Updated by Robin Bowes over 14 years ago

Let me be clear:

In config/initializers/puppet.rb, it should say:

if Facter.puppetversion == "0.25.5" 
  begin
    require 'RRDtool'
  rescue
    nil
  end
end

Actions #7

Updated by Ohad Levy over 14 years ago

Actions #8

Updated by Ohad Levy over 14 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF