Bug 970196

Summary:	Properly support colon-separated modulepath
Product:	Red Hat Satellite	Reporter:	Sam Kottler <skottler>
Component:	Provisioning	Assignee:	Ohad Levy <ohadlevy>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Corey Welton <cwelton>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	6.0.1	CC:	bkearney, cwelton, dcleal, ohadlevy, tkolhar
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-09-18 18:32:32 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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.