search

dfarrell07 / ansible-opendaylight

Moved: https://git.opendaylight.org/gerrit/gitweb?p=integration/packaging/ansible-opendaylight.git;a=tree
9 stars 12 forks source link

Update docs for ansible-opendaylight #33

Closed AkshitaJha closed 7 years ago

AkshitaJha commented 7 years ago

Signed-off-by: Akshita Jha zenith158@gmail.com

AkshitaJha commented 7 years ago

Any other information in the README.md that needs to be added/improved?

dfarrell07 commented 7 years ago

This looks perfect, thanks @AkshitaJha!

Any other information in the README.md that needs to be added/improved?

I noticed that the example extra_feautres could use an update. 'odl-ovsdb-openstack' is the old-NetVirt feature, to get the post-refactoring/split feature it should be odl-netvirt-openstack.

Would also be good to update this from a recent version of ODL (or remove ODL-specific version info to require less maintenance?)

opendaylight-user@root>feature:list | grep odl-ovsdb-openstack
odl-ovsdb-openstack | 1.1.0-Lithium | x | ovsdb-1.1.0-Lithium <snip>
dfarrell07 commented 7 years ago

Any other information in the README.md that needs to be added/improved?

Might also be good to include an example of invoking ansible-playbook to do an install?

AkshitaJha commented 7 years ago

Might also be good to include an example of invoking ansible-playbook to do an install?

Already doing this in the 'Example Playbook' section. Maybe it's better to move it to say an 'Install OpenDaylight` section?

dfarrell07 commented 7 years ago

Already doing this in the 'Example Playbook' section.

Ah, yes, so you are, sorry.

Maybe it's better to move it to say an 'Install OpenDaylight` section?

Yes, that might be a bit more clear to quickly scan. Up to you.

AkshitaJha commented 7 years ago

'odl-ovsdb-openstack' is the old-NetVirt feature, to get the post-refactoring/split feature it should be odl-netvirt-openstack.

Updated to odl-netvirt-openstack.

opendaylight-user@root>feature:list | grep odl-ovsdb-openstack
odl-ovsdb-openstack | 1.1.0-Lithium | x | ovsdb-1.1.0-Lithium <snip>

Removed the ODL version info.

Ah, yes, so you are, sorry.

Np. :) Added a new section, 'Installing OpenDaylight`

dfarrell07 commented 7 years ago

Awesome, perfect and ready to merge @AkshitaJha.