search

arista-eosplus / ansible-eos-interfaces

Prebuilt Ansible Role to manage interface configuration on your Arista EOS Device
BSD 3-Clause "New" or "Revised" License
5 stars 1 forks source link

Enable eapi for transport #12

Closed glennsb closed 7 years ago

glennsb commented 7 years ago

This changes all the actions to use only the provider dictionary settings for connection, thus removing hardcoded entries of cli transport only

This fixes #11

All stand alone connection/transport settings are removed and must come in via the provider dictionary only

grybak-arista commented 7 years ago

Hi Stuart. Thanks so much for the PR.

We have modified your suggested changes slightly in order to maintain the global setting availability. This will allow you and other users to specify everything within the individual provider configuration for each device (as your changes do), as well as using a globally specified value for values that are common to all providers.

The changes should be merged early next week. We are also applying the changes to our other ansible-eos roles. In the meantime, we'll leave this PR open. You can test the changes if you like by pulling down the 'transport' branch from this repo.

Gary Rybak

glennsb commented 7 years ago

Sounds great