-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Added priority attribute for pool members (bigip_pool_member module) #23128
Added priority attribute for pool members (bigip_pool_member module) #23128
Conversation
The test
|
@ossark if you'll be a champ and rename that Also, in the future, you can file issues at this repo for the F5 modules https://github.com/F5Networks/f5-ansible We upstream those modules and changes to those modules during each Ansible release cycle. |
@gundalow shipit This does not need to be cherry picked. |
@@ -384,7 +402,8 @@ def main(): | |||
description=dict(type='str'), | |||
rate_limit=dict(type='int'), | |||
ratio=dict(type='int'), | |||
preserve_node=dict(type='bool', default=False) | |||
preserve_node=dict(type='bool', default=False), | |||
priority_group=dict(type='int') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thus needs adding to the DOCUMENTION block code a the top of the file, just below where preserves node is documented.
This PR will be released in Ansible 2.4, so please add version_added:"2.4"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ossark Could you please address this then we can get this merged. Thanks for your fixes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ossark Thanks for the PR, just one change then I can get this merged. Thanks :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated as requested.
Give me a day or two and I should have this fixed. Thanks
… On 6 juli 2017, at 21:37, John R Barker ***@***.***> wrote:
@gundalow commented on this pull request.
In lib/ansible/modules/network/f5/bigip_pool_member.py:
> @@ -384,7 +402,8 @@ def main():
description=dict(type='str'),
rate_limit=dict(type='int'),
ratio=dict(type='int'),
- preserve_node=dict(type='bool', default=False)
+ preserve_node=dict(type='bool', default=False),
+ priority_group=dict(type='int')
@ossark Thanks for the PR, just one change then I can get this merged. Thanks :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@ossark fyi after this PR goes in, future PR's for F5 "stuff" usually start over here https://github.com/F5Networks/f5-ansible There's still time to get this in before 2.4 is released. |
closing & reopening totrigger CI |
Thanks. Merged into |
…nsible#23128) * Added priority attribute for pool members (bigip_pool_member module) * Line break missing * Change from 'priority' to 'priority_group' * Update bigip_pool_member.py * Update bigip_pool_member.py
SUMMARY
Added the ability to specify priority for pool members.
ISSUE TYPE
Feature pull request
COMPONENT NAME
bigip_pool_member
ANSIBLE VERSION