Skip to content

Neutron LB V2: use UpdateOptsBuilder#1706

Merged
ozerovandrei merged 2 commits intogophercloud:masterfrom
ozerovandrei:fix-neutron-lbaas-lb-updateopts
Sep 19, 2019
Merged

Neutron LB V2: use UpdateOptsBuilder#1706
ozerovandrei merged 2 commits intogophercloud:masterfrom
ozerovandrei:fix-neutron-lbaas-lb-updateopts

Conversation

@ozerovandrei
Copy link
Copy Markdown
Contributor

Use the UpdateOptsBuilder instead of the specific UpdateOpts struct
type in extensions/lbaas_v2/loadbalancers/requests.Update.

For #1704

Use the UpdateOptsBuilder instead of the specific UpdateOpts struct
type in extensions/lbaas_v2/listeners/requests.Update.
Use the UpdateOptsBuilder instead of the specific UpdateOpts struct
type in extensions/lbaas_v2/loadbalancers/requests.Update.
@coveralls
Copy link
Copy Markdown

coveralls commented Sep 17, 2019

Coverage Status

Coverage increased (+0.5%) to 78.12% when pulling 41e4978 on ozerovandrei:fix-neutron-lbaas-lb-updateopts into c1ae5ed on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Sep 17, 2019

Build failed.

@ozerovandrei
Copy link
Copy Markdown
Contributor Author

I'm going to merge this because here are no issues with loadbalancers tests and those changes doesn't break existing API.

@ozerovandrei ozerovandrei merged commit 34bae44 into gophercloud:master Sep 19, 2019
@ozerovandrei ozerovandrei deleted the fix-neutron-lbaas-lb-updateopts branch September 19, 2019 16:00
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.

2 participants