Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

README.md and bootstrap.sh changes #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jtktam
Copy link

@jtktam jtktam commented Apr 7, 2015

I am using a 5 node coreos cluster and ran into some issues, proposed changes:

  1. the ansible_python_interpreter variable setting in the README.md didn't work for me, I had to change it (tested working)
  2. I use an offline mirror of the pypy tar file, I made the download URL a variable to easily repoint the location
  3. Added some comments to bootstrap.sh

"PATH=/home/core/bin:$PATH python" didn't work for me, I had to change it to "/home/core/bin/python" for ansible to pick up the location of the python file
@defunctzombie
Copy link
Owner

the only change I like from this is the PYPY_URL variable. The rest isn't needed I think.

@jtktam
Copy link
Author

jtktam commented Apr 7, 2015

ansible_python_interpreter="PATH=/home/core/bin:$PATH python"

I am not sure if this was intentional to have a PATH= line inside the quotes, but the sample didn't work for me til i changed this to

ansible_python_interpreter="/home/core/bin/python"

i agree that the PYPY_URL change and added comments is not needed, but for me it made it easier to understand

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants