Skip to content

Support merge_labels when creating Magnum cluster#1985

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
openstacker:support-merge-labels
Jun 25, 2020
Merged

Support merge_labels when creating Magnum cluster#1985
jtopjian merged 1 commit intogophercloud:masterfrom
openstacker:support-merge-labels

Conversation

@openstacker
Copy link
Copy Markdown
Contributor

For #1984

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

The feature was introduced in OpenStack Magnum by https://review.opendev.org/#/c/720221/

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 79.379% when pulling fcbea8d on openstacker:support-merge-labels into 1f9e326 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jun 23, 2020

Build failed.

@jtopjian
Copy link
Copy Markdown
Contributor

@openstacker The OpenLab failure is ignorable - it was a temporary failure. This looks good to me - is this ready for review/merge?

@openstacker
Copy link
Copy Markdown
Contributor Author

@openstacker The OpenLab failure is ignorable - it was a temporary failure. This looks good to me - is this ready for review/merge?

Yep, it's ready for review. I tested locally. Thanks.

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!

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