Skip to content

ContainerInfra v1: Fix cluster Get#1968

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
openstacker:fix-magnum-health-status
May 18, 2020
Merged

ContainerInfra v1: Fix cluster Get#1968
jtopjian merged 1 commit intogophercloud:masterfrom
openstacker:fix-magnum-health-status

Conversation

@openstacker
Copy link
Copy Markdown
Contributor

This is a regression issue introduced by[1], which may impact the
cluster Get. The health_status_reason attribute should be a dict
instead of a string.

[1] #1910

For #1909

This is a regression issue introduced by[1], which may impact the
cluster Get. The health_status_reason attribute should be a dict
instead of string.

[1] gophercloud#1910
@openstacker openstacker force-pushed the fix-magnum-health-status branch from 0e847c5 to aea91bb Compare May 18, 2020 06:23
@coveralls
Copy link
Copy Markdown

coveralls commented May 18, 2020

Coverage Status

Coverage remained the same at 79.385% when pulling aea91bb on openstacker:fix-magnum-health-status into 0b03630 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented May 18, 2020

Build succeeded.

@openstacker
Copy link
Copy Markdown
Contributor Author

@jtopjian I'm sorry this is a regression issue. Please review it as it's blocked the very basic cluster Get function. BTW, when the version 0.12 will be released? Thanks.

@jtopjian
Copy link
Copy Markdown
Contributor

BTW, when the version 0.12 will be released?

Probably not until next month.

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!

FixedNetwork string `json:"fixed_network"`
FixedSubnet string `json:"fixed_subnet"`
HealthStatus string `json:"health_status"`
HealthStatusReason map[string]interface{} `json:"health_status_reason"`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@jtopjian
Copy link
Copy Markdown
Contributor

@openstacker We've had a few other small bug fixes come in. If you can confirm this works, I can release version 0.11.1 for you - if that would help?

@openstacker
Copy link
Copy Markdown
Contributor Author

@openstacker We've had a few other small bug fixes come in. If you can confirm this works, I can release version 0.11.1 for you - if that would help?

Thank you very much. I think I can just use the latest commit in CPO. That's OK. Thanks again.

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