Add missing fields in Objectstorage and compute API #2387
Conversation
There was a problem hiding this comment.
Thank you for submitting your first PR! Be sure that we will be looking at it but keep in mind
this sometimes takes a while.
Please let the maintainers know if your PR has not got enough attention after a few days.
If any doubt, please consult our PR tutorial.
|
bonjour @pdesgarets :-) Thanks for your pull request. For this PR, I see it has unit tests, thanks. It'll be great to also have acceptance coverage. Let me know if you need help for that. Thanks! |
|
OK, thanks for the review, I'll try to add acceptance tests, that may require some additional time |
|
Hello, may I ask if you are willing to further improve this PR? Now I have some requirements for Server Lock Status @pdesgarets |
|
Hi, I currently lack of free time, feel free to start from by branch and add the needed acceptance tests |
f8b21a4 to
cec507c
Compare
Fixes #2386 and #1958
Links to the line numbers/files in the OpenStack source code that support the
code in this PR:
https://docs.openstack.org/api-ref/compute/?expanded=show-server-details-detail#:~:text=security%20group%20name.-,locked,-body
https://docs.openstack.org/api-ref/object-store/?expanded=create-container-detail#:~:text=X%2DContainer%2DSync%2DTo%20(Optional)