Skip to content

Add DefaultPoolID update support#1313

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
kayrus:kayrus/lbaas_update_default_pool_id
Dec 7, 2018
Merged

Add DefaultPoolID update support#1313
jtopjian merged 1 commit intogophercloud:masterfrom
kayrus:kayrus/lbaas_update_default_pool_id

Conversation

@kayrus
Copy link
Copy Markdown
Contributor

@kayrus kayrus commented Oct 30, 2018

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 30, 2018

Coverage Status

Coverage remained the same at 79.608% when pulling a417135 on kayrus:kayrus/lbaas_update_default_pool_id into 3a7818a on gophercloud:master.

@jtopjian
Copy link
Copy Markdown
Contributor

@kayrus Thank you for submitting this.

Per the PR template, can you please review the contributor tutorial? Notably, a corresponding Issue needs opened for this. Additionally, you're referencing the Load Balancer API documentation, so perhaps this change should go under here instead? Or maybe both? The tutorial will detail how to confirm these things.

Please let me know if you have any questions.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Oct 30, 2018

Build succeeded.

@kayrus kayrus force-pushed the kayrus/lbaas_update_default_pool_id branch from 6ff5cd6 to 5cc442b Compare October 30, 2018 22:50
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Oct 31, 2018

Build succeeded.

@kayrus kayrus force-pushed the kayrus/lbaas_update_default_pool_id branch from 6030893 to a417135 Compare November 30, 2018 10:14
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Nov 30, 2018

Build succeeded.

Copy link
Copy Markdown
Contributor

@jtopjian jtopjian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. Let me know if this is ready to be merged.

@kayrus
Copy link
Copy Markdown
Contributor Author

kayrus commented Dec 7, 2018

It is ready.

@jtopjian jtopjian merged commit d3bcea3 into gophercloud:master Dec 7, 2018
@kayrus kayrus deleted the kayrus/lbaas_update_default_pool_id branch December 7, 2018 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants