Bug #1869

Not all providers loaded because of ImportErrors

Added by Patrice Neff about 3 years ago. Updated almost 3 years ago.

Status:Closed Start date:01/15/2009
Priority:Normal Due date:
Assignee:James Turnbull % Done:

0%

Category:plumbing
Target version:0.24.8
Affected Puppet version:0.24.7 Branch:
Keywords:
Votes: 0

Description

On one of our systems a recipe didn’t work which did work on another host with the exact same configuration. Turns out that the Dir.glob() from the util/autoload.rb method loadall returned files in a different order. In the failure case it tried to load the “directoryservice” group provider before all the others. On other servers that provider cam in the end.

The fix is:

--- autoload.rb.old    2009-01-15 17:59:32.000000000 +0100
+++ autoload.rb 2009-01-15 17:59:48.000000000 +0100
@@ -115,7 +115,7 @@
                 begin
                     Kernel.require file
                     loaded(name, file)
-                rescue => detail
+                rescue LoadError => detail
                     if Puppet[:trace]
                         puts detail.backtrace
                     end

The same happened in bug #312 but I don’t know enough about Ruby to know why this happens.


Related issues

duplicated by Puppet - Bug #1958: directoryservice provider sometimes seems to break the pr... Duplicate 02/11/2009
duplicated by Puppet - Bug #2030: launchd service provider getting set to Nil Duplicate 02/26/2009

History

Updated by James Turnbull about 3 years ago

  • Category set to plumbing
  • Status changed from Unreviewed to Needs More Information
  • Assignee set to Luke Kanies

Luke?

Updated by Luke Kanies about 3 years ago

  • Status changed from Needs More Information to Accepted
  • Target version set to 0.25.0

It’s way past time to replace autoload.rb, I think.

Updated by Luke Kanies almost 3 years ago

  • Target version changed from 0.25.0 to 0.24.8

Updated by Luke Kanies almost 3 years ago

  • Status changed from Accepted to Ready For Checkin
  • Assignee changed from Luke Kanies to James Turnbull

Fixed in tickets/0.24.x/1869.

Updated by James Turnbull almost 3 years ago

  • Status changed from Ready For Checkin to Closed

Pushed in commit:319822af6d58c3e0c391e86cfd836ec31de43c67 in branch 0.24.x

Also available in: Atom PDF