Skip to content

LBaaSv2: Properly handle update TLS references#1723

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
kayrus:handle-lbaas-tls-update
Oct 3, 2019
Merged

LBaaSv2: Properly handle update TLS references#1723
jtopjian merged 1 commit intogophercloud:masterfrom
kayrus:handle-lbaas-tls-update

Conversation

@kayrus
Copy link
Copy Markdown
Contributor

@kayrus kayrus commented Oct 1, 2019

Resolves #1722

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 1, 2019

Coverage Status

Coverage remained the same at 77.654% when pulling 2999dd1 on kayrus:handle-lbaas-tls-update into 863d540 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Oct 1, 2019

Build failed.

@kayrus kayrus force-pushed the handle-lbaas-tls-update branch from d86ca44 to 2999dd1 Compare October 3, 2019 15:26
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Oct 3, 2019

Build succeeded.

@kayrus kayrus changed the title LBaaSv2: Properly handle update TLS and AllowedCIDRs references LBaaSv2: Properly handle update TLS references Oct 3, 2019
@kayrus
Copy link
Copy Markdown
Contributor Author

kayrus commented Oct 3, 2019

@jtopjian ready for review.

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.

LGTM - thank you!

@jtopjian jtopjian merged commit d231593 into gophercloud:master Oct 3, 2019
@kayrus kayrus deleted the handle-lbaas-tls-update branch October 3, 2019 21:26
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.

Lbaas: Listener's SniContainerRefs cannot be unset

3 participants