Skip to content

Role-based setup #30

New issue

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

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
mraerino opened this issue Jan 12, 2017 · 2 comments
Closed

Role-based setup #30

mraerino opened this issue Jan 12, 2017 · 2 comments

Comments

@mraerino
Copy link

Would you consider accepting a pull-request, that would refactor the playbook to use roles instead of task includes?

Right now, if I want to use just parts of your playbook, I need to copy most of the vagrant.yml playbook, instead of just including some roles.
The advantage would be having handlers, templates and tasks bundled together in the role definition.
@dirkaholic
Copy link
Owner

Sure, feel free to open a PR. I think using roles is the better way anyways.

dirkaholic added a commit that referenced this issue Feb 5, 2017
@dirkaholic
Transfer ansible playbooks to roles, closes #30
6cfab6b
@dirkaholic
Copy link
Owner

Hi @mraerino, I opened #31 in order to refactor the playbooks into roles. Maybe you can test it ?

dirkaholic added a commit that referenced this issue Feb 6, 2017
@dirkaholic
Transfer ansible playbooks to roles, closes #30
05c3a17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dirkaholic @mraerino