Bug 970196 - Properly support colon-separated modulepath
Summary: Properly support colon-separated modulepath
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: Unspecified
Assignee: Ohad Levy
QA Contact: Corey Welton
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-03 16:25 UTC by Sam Kottler
Modified: 2019-09-25 20:45 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-18 18:32:32 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Sam Kottler 2013-06-03 16:25:28 UTC
Description of problem:
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.

Additional info:
Fixed upstream in d603c3c9438b03cfab968d5654c83557d227a217.

Comment 1 Sam Kottler 2013-06-11 19:51:26 UTC
Moved to ON_QA as part of puddle 4.5.

Comment 2 Tazim Kolhar 2013-06-12 09:32:34 UTC
hi

please provide verifications steps

thanks

Comment 3 Corey Welton 2013-06-12 17:59:01 UTC
Test notes:
"Either add a dummy directory to the puppet's module path, or delete a directory specified in the path from the disk."

Still may not be clear.  Perhaps ping dev directly if there are further questions.

Comment 4 Corey Welton 2013-06-12 20:30:45 UTC
clearing needinfo flag.

Comment 6 Dominic Cleal 2013-09-18 18:32:32 UTC
Upstream fix, shipped in MDP1.


Note You need to log in before you can comment on or make changes to this bug.