Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add hostgroups for all configured roles instead of using the run list. #7

Merged
merged 2 commits into from Dec 5, 2012

Conversation

Projects
None yet
2 participants
Contributor

martinotten commented Dec 3, 2012

Using the run list leaves out role dependencies, like our base role. As we
want to configure specific checks based on them, we have to include them
in the list of hostgroups.

martinotten added some commits Dec 3, 2012

@martinotten martinotten Add hostgroups for all configured roles instead of using the run list.
Using the run list leaves out role dependencies, like our base role. As we
want to configure specific checks based on them, we have to include them
in the list of hostgroups.
497e2ca
@martinotten martinotten fixed available roles check ed0a358

@mdxp mdxp added a commit that referenced this pull request Dec 5, 2012

@mdxp mdxp Merge pull request #7 from adcloud/all_roles_as_hostgroups
Add hostgroups for all configured roles instead of using the run list.
e731fbd

@mdxp mdxp merged commit e731fbd into mdxp:master Dec 5, 2012

1 check passed

default The Travis build passed
Details
Owner

mdxp commented Dec 5, 2012

Merged. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment