search

contiv-experimental / demo

Easy cut demos to try contiv [DEPRECATED]
Other
6 stars 25 forks source link

Update README.md #55

Closed gaurav-dalvi closed 8 years ago

gaurav-dalvi commented 8 years ago

Added 2 steps in prerequisite section.

gaurav-dalvi commented 8 years ago

Thanks Madhav. I was running Centos 7 which came with Ansible 1.x, So I bumped up the version of Ansible. So I feel its appropriate to write this step here. If not in installer script should take care of displaying this message or bail from the installation with some error message.

mapuri commented 8 years ago

@gaurav-dalvi

So I feel its appropriate to write this step here.

may be I wasn't clear, what I meant was that since installer takes care of installing it it shall take care of updating it and we should have a need to document this as a pre-requisite.

gaurav-dalvi commented 8 years ago

Hey Madhav,

You are right. I have other PR with the fix. (via EPEL testing repo, installing ansible 2.0) Closing this PR now as Its not required.