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

Update README.md to include ansible version #6

Merged
merged 1 commit into from
Nov 7, 2014

Conversation

adammeghji
Copy link
Contributor

Ansible version needs to be >= 1.7, otherwise mesos_install_mode="slave" does not get set correctly, and slaves get set up as master nodes.

👎 ANSIBLE 1.6:

TASK: [ansible-mesos | debug msg="Install Mode is slave"] *********************
ok: [ec2-54-87-91-235.compute-1.amazonaws.com] => {
    "item": "",
    "msg": "Install Mode is master"
}
ok: [ec2-54-235-237-69.compute-1.amazonaws.com] => {
    "item": "",
    "msg": "Install Mode is master"
}
ok: [ec2-54-234-24-109.compute-1.amazonaws.com] => {
    "item": "",
    "msg": "Install Mode is master"
}

👍 ANSIBLE 1.7:

TASK: [ansible-mesos | debug msg="Install Mode is slave"] *********************
ok: [ec2-54-87-91-235.compute-1.amazonaws.com] => {
    "msg": "Install Mode is slave"
}
ok: [ec2-54-235-237-69.compute-1.amazonaws.com] => {
    "msg": "Install Mode is slave"
}
ok: [ec2-54-234-24-109.compute-1.amazonaws.com] => {
    "msg": "Install Mode is slave"
}

Ansible version needs to be >= 1.7, otherwise mesos_install_mode="slave" does not get set correctly, and slaves get set up as master nodes.
mhamrah added a commit that referenced this pull request Nov 7, 2014
Update README.md to include ansible version
@mhamrah mhamrah merged commit 0ef0f25 into mhamrah:master Nov 7, 2014
@mhamrah
Copy link
Owner

mhamrah commented Nov 7, 2014

Thanks for the FYI!

@adammeghji
Copy link
Contributor Author

No problem! I really really appreciate the work you've put into this -- it's been very inspiring and helpful. 🙇

I've actually got a number of other supplementary scripts / Makefiles / etc. which I'm using to implement a complete solution on EC2, hopefully I'll have some more contributions to share back with you. 👍

Cheers!

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