Project

General

Profile

Bug #2770

Import of Environments and Classes Fail

Added by Andy Edmonds over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Importers
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

On installing foreman 1.2 using the foreman installer (vagrant project here [1]) the import environment and classes functionality does not work. This issue has also been discussed on the foreman-user mailing list [2]. In summary:

Andy

[1] https://github.com/dizz/vagrant-foreman
[2] https://groups.google.com/d/topic/foreman-users/H2WIGMfXIAI/discussion

Associated revisions

Revision 83c98163 (diff)
Added by Dominic Cleal over 6 years ago

fixes #2770 - create modules/ subdir for each environment

History

#1 Updated by Dominic Cleal over 6 years ago

  • Target version set to 1.2.1

#2 Updated by Przemyslaw Bak over 6 years ago

The Foreman 1.2 + Puppet 3.2.2 with simple config - the same behaviour: cannot import puppet environments.
The Foreman just after installation is at the moment unusable.

#3 Updated by Dominic Cleal over 6 years ago

  • Status changed from New to Assigned
  • Assignee set to Dominic Cleal

I think I see what's happening, the correct environment directory structure isn't being created by the installer (we were trying to standardise the paths it uses in different configs).

[production]
modulepath = /etc/puppet/environments/production/modules:/etc/puppet/environments/common:/usr/share/puppet/modules

Note the path is /etc/puppet/environments/production/modules but the modules are installed under /etc/puppet/environments/production (missing the last modules/). The installer is only creating up to /etc/puppet/environments/production without the modules/ subdir, which is likely to throw you (it just did me too).

#4 Updated by Dominic Cleal over 6 years ago

  • Status changed from Assigned to Ready For Testing

#5 Updated by Andy Edmonds over 6 years ago

I've just tested this and it works! Thanks very much Dominic!

Dominic Cleal wrote:

https://github.com/theforeman/puppet-puppet/pull/80

#6 Updated by Sam Kottler over 6 years ago

  • Status changed from Ready For Testing to Closed

The pull request got merged.

Also available in: Atom PDF