)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34041,"name":"Diwei Zhu","email":"zhu.diw@northeastern.edu","username":"DiweiZhu"},"change_message_id":"e10ad771d767c08cb94dbfb151821a8e9a933879","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3d82873f_b027a1dc","updated":"2021-11-02 05:00:45.000000000","message":"Hi, I could not find a good way for the unit test.\nBecause fake servers are all sdk Server objects, its method _action is not a mock.\nI am thinking making the server dump method return a list of return values from _action, and cheking these values. Not sure it\u0027s a good way.\nCould you guys give me some advise? Thanks a lot.","commit_id":"72e0e990d2c54be7e3deb6af67b92c07c386694c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"11e2972c6ef0e281502e1f679af92b19e185b749","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2823b95e_08b45435","in_reply_to":"3886df7d_ba475367","updated":"2022-12-15 17:21:14.000000000","message":"Done","commit_id":"72e0e990d2c54be7e3deb6af67b92c07c386694c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0cd93c4d06692ef1aaea5a46805b230801066fae","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3886df7d_ba475367","in_reply_to":"3d82873f_b027a1dc","updated":"2021-11-03 12:09:26.000000000","message":"My gut feeling here would be that we should instead add the missing feature to openstacksdk and then wait for that feature to merge and be release. This would be a relatively easy feature addition, if you wanted to give it a go. I have a number of recently merged patches to openstacksdk that you could use as blueprints:\n\n  $ cd openstacksdk\n  $ git checkout feature/r1\n  $ git log --author\u003dFinucane -- openstacksdk/compute\n\nIf you do this, remember to base your work on the \u0027feature/r1\u0027 branch, which exists to allow us prepare for the upcoming v1.0 release of openstacksdk.\n\nIf you don\u0027t want to do this, I\u0027d personally skip this command for now. Doing things this way leaves a TODO and it seems silly to work on the same command twice when the version based on novaclient works just fine for now.\n\nLet me know what you think","commit_id":"72e0e990d2c54be7e3deb6af67b92c07c386694c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0e6a7d44975f9e99ed17ddee9a249bfb31e1b82d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"501ebb3d_2509635a","updated":"2021-12-09 17:21:44.000000000","message":"recheck","commit_id":"2450bc7fa812339bcf37be8250e9c5a4b613e251"}]}