Project

General

Profile

Bug #2119

ignored valid puppet modulepath if first configured directory does not exist

Added by Kevin Cormier almost 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Puppet
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

In my master section of my puppet.conf I have the following modulepath:

modulepath = $confdir/environments/$environment/puppet_modules:/home/$environment/puppet_modules

For my personal testing environment (kcormier) I keep all of my modules under /home/kcormier/puppet_modules. /etc/puppet/environments/kcormier/puppet_modules does not exist. Neither does the parent directory /etc/puppet/environments/kcormier. Puppet works fine however foreman fails to import my classes and thinks the environment does not exist. If I create /etc/puppet/environments/kcormier, foreman will load my classes correctly. Just to note, /etc/puppet/environments/kcormier/puppet_modules still DOES NOT exist, however it's parent directory does.

This was found with foreman-proxy 1.1rc1 from the foreman-1.1.rc4 rpms running on centos 6.3 fully patched.

Associated revisions

Revision 722bba2d (diff)
Added by Greg Sutcliffe over 7 years ago

Fixes #2119 - Properly support colon-separated modulepath

Revision d603c3c9 (diff)
Added by Greg Sutcliffe over 7 years ago

Fixes #2119 - Properly support colon-separated modulepath
(cherry picked from commit 722bba2d153d021827d7a1b6eb32748d0ceb7f86)

History

#1 Updated by Sam Kottler almost 8 years ago

  • Target version set to 26

I can reproduce this. Thanks for the report Kevin Cormier!

#2 Updated by Dominic Cleal over 7 years ago

  • Target version changed from 26 to 1.2.0

#3 Updated by Greg Sutcliffe over 7 years ago

  • Assignee set to Greg Sutcliffe

#4 Updated by Greg Sutcliffe over 7 years ago

  • Status changed from New to Ready For Testing

#5 Updated by Anonymous over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#6 Updated by Greg Sutcliffe over 2 years ago

  • Target version deleted (1.2.0)

Also available in: Atom PDF