)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8a4b6e94434e1ecfa728f7c65313139f09fac382","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e25d9774_7bb9838a","updated":"2023-05-05 21:33:38.000000000","message":"Rey, this looks good! good work on the functional tests! a comment inline about the subnets attribute...","commit_id":"350b28712860b7205b1d6882802379a096595d42"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c32de6b609d2eb0efe6ea7bf38fc19a5da7b4125","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"71da87a5_7016a7fa","updated":"2023-05-08 16:48:41.000000000","message":"This should work; just a hacking comment around the import","commit_id":"c4ecaeca28f5f24c7ccc21548265a9979026a7ae"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"40922cb7aa98ad669f206f9192a7f0203d8d11e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"96b95503_6dc3ed22","updated":"2023-05-08 17:57:43.000000000","message":"as the pep8 job failure [1] tells you, this repo now has \"black\" enabled.. I just sent you an email about how to address these failures...  \n\n[1] https://zuul.opendev.org/t/openstack/build/dbd64f17fa824fb59cff7dd30f466235/log/job-output.txt","commit_id":"c4ecaeca28f5f24c7ccc21548265a9979026a7ae"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"05294b79279cfa8cc8645c04e1a9a040117fe098","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"84faf0df_6a115eb7","updated":"2023-05-10 03:16:45.000000000","message":"Looks like black would like some changes in test_share_network.py\n\nhttps://zuul.opendev.org/t/openstack/build/142d20ce24af46cca71212887b1ed09a/log/job-output.txt#4191-4243","commit_id":"5e108b38edc54e20633d2fc6e0a111a79071b3ce"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1e3c0977d8d6ff9ff090df055d656b6ed99e3c46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c4de7600_6e40670f","in_reply_to":"84faf0df_6a115eb7","updated":"2023-05-10 09:43:42.000000000","message":"Fixed. FYI if you install pre-commit, it will handle this automatically for you going forward.\n\n    sudo dnf install pre-commit  # or \u0027sudo apt install pre-commit\u0027 or or \u0027pipx install pre-commit\u0027\n    pre-commit install","commit_id":"5e108b38edc54e20633d2fc6e0a111a79071b3ce"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"148c5a5583978d366ff6a98f9c1002423f4c12bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dee8eccb_f91c6928","updated":"2023-05-10 15:17:17.000000000","message":"Thanks Stephen; could you please +W :)","commit_id":"692b7f39e3765b10f3092ea6a2886ed03cbc37b6"}],"openstack/shared_file_system/v2/share_network.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8a4b6e94434e1ecfa728f7c65313139f09fac382","unresolved":true,"context_lines":[{"line_number":42,"context_line":"    project_id \u003d resource.Body(\"project_id\", type\u003dstr)"},{"line_number":43,"context_line":"    #: A list of share network subnets that pertain to the related share"},{"line_number":44,"context_line":"    #: network."},{"line_number":45,"context_line":"    share_network_subnets \u003d resource.Body(\"share_network_subnets\", type\u003dlist)"},{"line_number":46,"context_line":"    #: The UUID of a neutron network when setting up or"},{"line_number":47,"context_line":"    #: updating a share network subnet with neutron."},{"line_number":48,"context_line":"    neutron_net_id \u003d resource.Body(\"neutron_net_id\", type\u003dstr)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bd5c3096_544c960d","line":45,"range":{"start_line":45,"start_character":72,"end_line":45,"end_character":76},"updated":"2023-05-05 21:33:38.000000000","message":"There\u0027s an attribute we could use called \"list_type\" and set that to the ShareNetworkSubnet resource... That way the list isn\u0027t a list of dicts.. \n\nexamples: https://github.com/search?q\u003drepo%3Aopenstack%2Fopenstacksdk%20list_type\u0026type\u003dcode","commit_id":"350b28712860b7205b1d6882802379a096595d42"},{"author":{"_account_id":35745,"name":"Reynaldo","display_name":"Reynaldo","email":"rey.bontje80@gmail.com","username":"plugd-in"},"change_message_id":"06f50c7efe6cb19b797d52da74e71b156720829f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    project_id \u003d resource.Body(\"project_id\", type\u003dstr)"},{"line_number":43,"context_line":"    #: A list of share network subnets that pertain to the related share"},{"line_number":44,"context_line":"    #: network."},{"line_number":45,"context_line":"    share_network_subnets \u003d resource.Body(\"share_network_subnets\", type\u003dlist)"},{"line_number":46,"context_line":"    #: The UUID of a neutron network when setting up or"},{"line_number":47,"context_line":"    #: updating a share network subnet with neutron."},{"line_number":48,"context_line":"    neutron_net_id \u003d resource.Body(\"neutron_net_id\", type\u003dstr)"}],"source_content_type":"text/x-python","patch_set":1,"id":"acc9374d_c26fb04f","line":45,"range":{"start_line":45,"start_character":72,"end_line":45,"end_character":76},"in_reply_to":"bd5c3096_544c960d","updated":"2023-05-08 16:22:18.000000000","message":"Done. Thanks for catching that.","commit_id":"350b28712860b7205b1d6882802379a096595d42"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c32de6b609d2eb0efe6ea7bf38fc19a5da7b4125","unresolved":true,"context_lines":[{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from openstack import resource"},{"line_number":14,"context_line":"from openstack.shared_file_system.v2.share_network_subnet import ("},{"line_number":15,"context_line":"    ShareNetworkSubnet"},{"line_number":16,"context_line":")"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class ShareNetwork(resource.Resource):"},{"line_number":20,"context_line":"    resource_key \u003d \"share_network\""}],"source_content_type":"text/x-python","patch_set":2,"id":"297d259c_8d359fab","line":17,"range":{"start_line":14,"start_character":0,"end_line":17,"end_character":0},"updated":"2023-05-08 16:48:41.000000000","message":"Import the module, not the class directly:\n\n\nhttps://docs.openstack.org/hacking/latest/user/hacking.html#imports","commit_id":"c4ecaeca28f5f24c7ccc21548265a9979026a7ae"},{"author":{"_account_id":35745,"name":"Reynaldo","display_name":"Reynaldo","email":"rey.bontje80@gmail.com","username":"plugd-in"},"change_message_id":"1fec64ae86593d6d8dca480eff77ae9e49e772ca","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from openstack import resource"},{"line_number":14,"context_line":"from openstack.shared_file_system.v2.share_network_subnet import ("},{"line_number":15,"context_line":"    ShareNetworkSubnet"},{"line_number":16,"context_line":")"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class ShareNetwork(resource.Resource):"},{"line_number":20,"context_line":"    resource_key \u003d \"share_network\""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a655ae3_6352ab79","line":17,"range":{"start_line":14,"start_character":0,"end_line":17,"end_character":0},"in_reply_to":"297d259c_8d359fab","updated":"2023-05-10 01:50:15.000000000","message":"Done","commit_id":"c4ecaeca28f5f24c7ccc21548265a9979026a7ae"}]}
