)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"bd5177ba095af414a8ac20129c9df9d9e0731c8c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     qinqing \u003cqqing0702@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-27 17:30:48 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"subnetpool not supported in tosca neworking"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1864967"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"0fef97b3_b7247566","line":7,"updated":"2022-06-08 09:55:53.000000000","message":"Could you add an appropriate reference to explain this change?","commit_id":"855777c4949f1e49cd8691387ce56bb4d68ff58b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"de6d69d5d9b2324c46b57997093906b7b7c075cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2c19b78f_9eb1159a","updated":"2022-08-26 07:54:51.000000000","message":"Any update?","commit_id":"855777c4949f1e49cd8691387ce56bb4d68ff58b"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"bd5177ba095af414a8ac20129c9df9d9e0731c8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d7f07a25_d0ffd4df","updated":"2022-06-08 09:55:53.000000000","message":"Thank you for the patch.\nPlease kindly find my comments.\n\n\u003e Patch Set 4: Code-Review-1\n\u003e \n\u003e Thanks for the patch!  Can you please add a test to cover the subnetpool scenario?  Something to ensure that the translation occurs properly.  Thanks\n\n+1\nAdding a test case to ``test_tosca_hot_translation`` [1] with a new sample file might be better.\n\n[1] https://github.com/openstack/heat-translator/blob/master/translator/tests/test_tosca_hot_translation.py#L483 ","commit_id":"855777c4949f1e49cd8691387ce56bb4d68ff58b"}],"translator/hot/tosca/tosca_network_network.py":[{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"bd5177ba095af414a8ac20129c9df9d9e0731c8c","unresolved":true,"context_lines":[{"line_number":44,"context_line":"                if key \u003d\u003d \u0027network_name\u0027:"},{"line_number":45,"context_line":"                    # If CIDR or subnetpool is specified network_name should"},{"line_number":46,"context_line":"                    # be used as the name for the new network."},{"line_number":47,"context_line":"                    if \u0027cidr\u0027 in tosca_props.keys() or \\"},{"line_number":48,"context_line":"                            \u0027subnetpool\u0027 in tosca_props.keys():"},{"line_number":49,"context_line":"                        net_props[\u0027name\u0027] \u003d value"},{"line_number":50,"context_line":"                    # If CIDR or subnetpool is not specified network_name"}],"source_content_type":"text/x-python","patch_set":4,"id":"af7f93fd_f12af8ec","line":47,"range":{"start_line":47,"start_character":54,"end_line":47,"end_character":56},"updated":"2022-06-08 09:55:53.000000000","message":"Please use parentheses instead [1].\n\n[1] https://docs.openstack.org/hacking/latest/user/hacking.html#general\n\n\u003e It is preferred to wrap long lines in parentheses and not a backslash for line continuation.","commit_id":"855777c4949f1e49cd8691387ce56bb4d68ff58b"}]}