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

Close #46 #47

Merged
merged 4 commits into from
Oct 20, 2015
Merged

Close #46 #47

merged 4 commits into from
Oct 20, 2015

Conversation

phil-dileo
Copy link
Contributor

  • This commit adds a 'mode' keyword argument to the set_members method. The
    set_members method already looks to get the lacp_mode in use from existing
    members but ultimately uses 'on' if no other members exist. This commit
    allows you to supercede that pass in the desired lacp mode.

- This commit adds a 'mode' keyword argument to the set_members method. The
set_members method already looks to get the lacp_mode in use from existing
members but ultimately uses 'on' if no other members exist. This commit
allows you to supercede that pass in the desired lacp mode.
phil-dileo added a commit that referenced this pull request Oct 20, 2015
Add lacp mode to set_members()
- Close #46
@phil-dileo phil-dileo merged commit c0ce04b into develop Oct 20, 2015
@phil-dileo phil-dileo deleted the feature-lacp_mode branch October 20, 2015 20:05
@phil-dileo phil-dileo added this to the 0.4.0 milestone Oct 27, 2015
@phil-dileo
Copy link
Contributor Author

Release Note:
This enhancement allows you to set the LACP Mode while executing the set_members method. The call would look like node.api('interfaces').set_members(1, [Ethernet1,Ethernet2], mode='active')

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

Successfully merging this pull request may close these issues.

1 participant