Skip to content

Containerinfra: Allow setting floating_ip_enabled for creating cluster#1677

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
lingxiankong:magnum-floating_ip_enabled
Aug 13, 2019
Merged

Containerinfra: Allow setting floating_ip_enabled for creating cluster#1677
jtopjian merged 1 commit intogophercloud:masterfrom
lingxiankong:magnum-floating_ip_enabled

Conversation

@lingxiankong
Copy link
Copy Markdown
Contributor

For #1673

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

https://github.com/openstack/magnum/blob/04fd0470ad3b35c4af5212eee82431b1be43d64c/magnum/api/controllers/v1/cluster.py#L183

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Aug 12, 2019

Build failed.

@lingxiankong
Copy link
Copy Markdown
Contributor Author

/test openlab/check

@lingxiankong
Copy link
Copy Markdown
Contributor Author

@jtopjian How can I retrigger the job?

@jtopjian
Copy link
Copy Markdown
Contributor

recheck

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Aug 13, 2019

Build succeeded.

@lingxiankong lingxiankong force-pushed the magnum-floating_ip_enabled branch from 9658af3 to 079fc42 Compare August 13, 2019 01:47
@jtopjian
Copy link
Copy Markdown
Contributor

@lingxiankong It looks like some git diff text was accidentally merged in. For example: https://github.com/gophercloud/gophercloud/pull/1677/files#diff-58f1d20ea9e591802d022af1b5956176R150

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Aug 13, 2019

Build succeeded.

@lingxiankong lingxiankong force-pushed the magnum-floating_ip_enabled branch from 079fc42 to 3f59acb Compare August 13, 2019 03:50
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Aug 13, 2019

Build succeeded.

@lingxiankong
Copy link
Copy Markdown
Contributor Author

@jtopjian thanks for the review, diff text was removed.

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 15741a8 into gophercloud:master Aug 13, 2019
@lingxiankong lingxiankong deleted the magnum-floating_ip_enabled branch August 13, 2019 21:23
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 76.79% when pulling 9658af3 on lingxiankong:magnum-floating_ip_enabled into 33168d2 on gophercloud:master.

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