Refactor #1195
let the user specify the dirs to scan for import
| Status: | New | Start: | 09/29/2011 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned to: | - | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - | |||
| Backlog: | No | Difficulity: | ||
| Votes: | 0 |
Description
Let the user specify the import dirs to scan from the task to import classes through the settings.yaml file.
When using roles to assign hosts instead of directories using classes it is common to declare roles (classes) in /etc/puppet/manifests dir and then include the needed modules classes.
Example
class role_abc {
$role = role_abc
include http
include database
}
then one could assign role role_abc to a host named test.example.com. (roles is mentioned in puppetlabs best practices guide)
Patch:
https://github.com/gustavosoares/foreman/commit/e40249ad0180d6817226e2112af30888d45953b1
History
Updated by Ohad Levy over 1 year ago
hmm.. maybe I'm missing something here, but isnt hostgroup == roles in you example?
Updated by Gustavo Soares over 1 year ago
Ohad Levy wrote:
hmm.. maybe I'm missing something here, but isnt hostgroup == roles in you example?
more or less.. I think. Take a look at this design guideline: http://projects.puppetlabs.com/projects/1/wiki/Infrastructure_Design_Guidelines
IMHO I see hostgroups as a way to implement inheritance between nodes. It's subtle difference, I believe.
For instance:
node 'ntp.example42.com' inherits basenode {
$my_ntp_server = "0.pool.ntp.org"
include role_general
}then I could declare the role_general like this
class role_general {
include yum
include hosts
include puppet
include iptables
include sysctl
include nrpe
include ntp
include syslog
}
This file file would be located outside the modules dir...
Wouldn't "basenode" be a hostgroup in Foreman? Or there is another way to do this?
The provided patch would allow both ways to coexist.
Updated by Ohad Levy over 1 year ago
Gustavo Soares wrote:
Ohad Levy wrote:
hmm.. maybe I'm missing something here, but isnt hostgroup == roles in you example?
more or less.. I think. Take a look at this design guideline: http://projects.puppetlabs.com/projects/1/wiki/Infrastructure_Design_Guidelines
IMHO I see hostgroups as a way to implement inheritance between nodes. It's subtle difference, I believe.
For instance:
node 'ntp.example42.com' inherits basenode { $my_ntp_server = "0.pool.ntp.org"
include role_general }
then I could declare the role_general like this
class role_general { include yum include hosts include puppet include iptables include sysctl include nrpe include ntp include syslog }
This file file would be located outside the modules dir...
Wouldn't "basenode" be a hostgroup in Foreman? Or there is another way to do this?
It seems like you are not using foreman as your ENC, if you do, then all of this is not required....
The provided patch would allow both ways to coexist.
You patch force the user to enter that value afaik
Updated by Gustavo Soares over 1 year ago
Ohad Levy wrote:
Gustavo Soares wrote:
Ohad Levy wrote:
hmm.. maybe I'm missing something here, but isnt hostgroup == roles in you example?
more or less.. I think. Take a look at this design guideline: http://projects.puppetlabs.com/projects/1/wiki/Infrastructure_Design_Guidelines
IMHO I see hostgroups as a way to implement inheritance between nodes. It's subtle difference, I believe.
For instance:
node 'ntp.example42.com' inherits basenode { $my_ntp_server = "0.pool.ntp.org"
include role_general }
then I could declare the role_general like this
class role_general { include yum include hosts include puppet include iptables include sysctl include nrpe include ntp include syslog }
This file file would be located outside the modules dir...
Wouldn't "basenode" be a hostgroup in Foreman? Or there is another way to do this?
It seems like you are not using foreman as your ENC, if you do, then all of this is not required....
The node example was just to (try) to illustrate my point. I do use foremam as my ENC. Maybe i am missing something here...
The provided patch would allow both ways to coexist.
You patch force the user to enter that value afaik
