Skip to content
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.

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

Made cluster bootable #3

Merged
merged 1 commit into from
Feb 7, 2016
Merged

Conversation

Rodjers
Copy link
Contributor

@Rodjers Rodjers commented Feb 7, 2016

Removed start_join from bootstrap template. Removed own node from
start_join in server template. Added ui flag and fixed ui_dir in both
server and bootstrap template files.

Added service task to site.yml to start the consul service on all nodes
after finishing provisioning Consul.

Fixed CentOS.yml and Debian.yml to follow Ansible best practices.

Removed start_join from bootstrap template. Removed own node from
start_join in server template. Added ui flag and fixed ui_dir in both
server and bootstrap template files.

Added service task to site.yml to start the consul service on all nodes
after finishing provisioning Consul.

Fixed CentOS.yml and Debian.yml to follow Ansible best practices.
@Rodjers
Copy link
Contributor Author

Rodjers commented Feb 7, 2016

I have tested this using the preinstall script and vagrant up. It works for both Debian and CentOS. For CentOS i used the box image "centos/7".

@brianshumate
Copy link
Contributor

This is great, thank you!

brianshumate added a commit that referenced this pull request Feb 7, 2016
@brianshumate brianshumate merged commit 1300c9a into ansible-collections:master Feb 7, 2016
@Rodjers
Copy link
Contributor Author

Rodjers commented Feb 7, 2016

My pleasure. I should be thanking you. This role was just what i needed.

2016-02-07 22:36 GMT+01:00 Brian Shumate notifications@github.com:

Merged #3 #3.


Reply to this email directly or view it on GitHub
#3 (comment).

Mvh
Oddgeir Gitlestad

@brianshumate
Copy link
Contributor

👍 Glad that you found it useful!

brianshumate pushed a commit that referenced this pull request Nov 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants