)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"3906090fc51d66fb2eaa2f0ddb41c9c0090a3136","unresolved":true,"context_lines":[{"line_number":11,"context_line":"response from the connected server, it will not be able to close"},{"line_number":12,"context_line":"its own socket even if it terminates abnormally."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"To resolve the problem, we add vnf_lcm.timeout to tacker.conf."},{"line_number":15,"context_line":"When submitting the API, the timeout value is set to the requests"},{"line_number":16,"context_line":"api. We set the default value to 10 seconds (with 3 tries)."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"f36c963c_689c39d9","line":14,"range":{"start_line":14,"start_character":31,"end_line":14,"end_character":46},"updated":"2021-03-22 02:29:31.000000000","message":"It\u0027s not `timeout` but `retry_timeout` correctly?","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"6c14eff9fa61bfd0d5ba98268ec6d9dc75679d48","unresolved":false,"context_lines":[{"line_number":11,"context_line":"response from the connected server, it will not be able to close"},{"line_number":12,"context_line":"its own socket even if it terminates abnormally."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"To resolve the problem, we add vnf_lcm.timeout to tacker.conf."},{"line_number":15,"context_line":"When submitting the API, the timeout value is set to the requests"},{"line_number":16,"context_line":"api. We set the default value to 10 seconds (with 3 tries)."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"088974a2_30d7b7eb","line":14,"range":{"start_line":14,"start_character":31,"end_line":14,"end_character":46},"in_reply_to":"ec37673d_28b8552f","updated":"2021-03-24 08:50:30.000000000","message":"Done","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"29221f940149ea0c3875543467717906be465505","unresolved":true,"context_lines":[{"line_number":11,"context_line":"response from the connected server, it will not be able to close"},{"line_number":12,"context_line":"its own socket even if it terminates abnormally."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"To resolve the problem, we add vnf_lcm.timeout to tacker.conf."},{"line_number":15,"context_line":"When submitting the API, the timeout value is set to the requests"},{"line_number":16,"context_line":"api. We set the default value to 10 seconds (with 3 tries)."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ec37673d_28b8552f","line":14,"range":{"start_line":14,"start_character":31,"end_line":14,"end_character":46},"in_reply_to":"f36c963c_689c39d9","updated":"2021-03-23 08:37:35.000000000","message":"\u003e It\u0027s not `timeout` but `retry_timeout` correctly?\n\nChanged to vnf_lcm.retry_timeout","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"3906090fc51d66fb2eaa2f0ddb41c9c0090a3136","unresolved":true,"context_lines":[{"line_number":16,"context_line":"api. We set the default value to 10 seconds (with 3 tries)."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1911124"},{"line_number":19,"context_line":"Implements: https://bugs.launchpad.net/tacker/+bug/1911124"},{"line_number":20,"context_line":"Change-Id: I63078635a8db5669dbd94ecf8c69bb3946ae0347"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"e3ba1e86_777c417e","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":58},"updated":"2021-03-22 02:29:31.000000000","message":"`Implements` tag is used for blueprint. You have mentioned your fix with `Closes-Bug`, so no need to use `Implements`.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"6c14eff9fa61bfd0d5ba98268ec6d9dc75679d48","unresolved":false,"context_lines":[{"line_number":16,"context_line":"api. We set the default value to 10 seconds (with 3 tries)."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1911124"},{"line_number":19,"context_line":"Implements: https://bugs.launchpad.net/tacker/+bug/1911124"},{"line_number":20,"context_line":"Change-Id: I63078635a8db5669dbd94ecf8c69bb3946ae0347"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ee3c61b9_2b6c055f","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":58},"in_reply_to":"7dc73a24_bd94acd7","updated":"2021-03-24 08:50:30.000000000","message":"Done","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"29221f940149ea0c3875543467717906be465505","unresolved":true,"context_lines":[{"line_number":16,"context_line":"api. We set the default value to 10 seconds (with 3 tries)."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1911124"},{"line_number":19,"context_line":"Implements: https://bugs.launchpad.net/tacker/+bug/1911124"},{"line_number":20,"context_line":"Change-Id: I63078635a8db5669dbd94ecf8c69bb3946ae0347"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7dc73a24_bd94acd7","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":58},"in_reply_to":"e3ba1e86_777c417e","updated":"2021-03-23 08:37:35.000000000","message":"\u003e `Implements` tag is used for blueprint. You have mentioned your fix with `Closes-Bug`, so no need to use `Implements`.\n\nRemoved \"Implements\" line.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"}],"tacker/conductor/conductor_server.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"9598b7b9a8a90e5645a0ea7b8d2083c2eb43ac72","unresolved":true,"context_lines":[{"line_number":1463,"context_line":"        if retry_count \u003d\u003d CONF.vnf_lcm.retry_num:"},{"line_number":1464,"context_line":"            LOG.warn(\"Number of retries exceeded retry count\")"},{"line_number":1465,"context_line":""},{"line_number":1466,"context_line":"    def send_notification(self, context, notification):"},{"line_number":1467,"context_line":"        try:"},{"line_number":1468,"context_line":"            LOG.debug(\"send_notification start notification[%s]\""},{"line_number":1469,"context_line":"                      % notification)"}],"source_content_type":"text/x-python","patch_set":3,"id":"f4ac7687_df207032","line":1466,"updated":"2021-03-22 06:46:55.000000000","message":"I think it\u0027s better to include docstrings because the returned value is special.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"0ce34702d98bea0118223769624aeb383b09bc4b","unresolved":false,"context_lines":[{"line_number":1463,"context_line":"        if retry_count \u003d\u003d CONF.vnf_lcm.retry_num:"},{"line_number":1464,"context_line":"            LOG.warn(\"Number of retries exceeded retry count\")"},{"line_number":1465,"context_line":""},{"line_number":1466,"context_line":"    def send_notification(self, context, notification):"},{"line_number":1467,"context_line":"        try:"},{"line_number":1468,"context_line":"            LOG.debug(\"send_notification start notification[%s]\""},{"line_number":1469,"context_line":"                      % notification)"}],"source_content_type":"text/x-python","patch_set":3,"id":"424fc6a3_90dda4b1","line":1466,"in_reply_to":"cd2a28df_e4a6601c","updated":"2021-03-24 00:47:28.000000000","message":"I\u0027ve confirmed.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"29221f940149ea0c3875543467717906be465505","unresolved":true,"context_lines":[{"line_number":1463,"context_line":"        if retry_count \u003d\u003d CONF.vnf_lcm.retry_num:"},{"line_number":1464,"context_line":"            LOG.warn(\"Number of retries exceeded retry count\")"},{"line_number":1465,"context_line":""},{"line_number":1466,"context_line":"    def send_notification(self, context, notification):"},{"line_number":1467,"context_line":"        try:"},{"line_number":1468,"context_line":"            LOG.debug(\"send_notification start notification[%s]\""},{"line_number":1469,"context_line":"                      % notification)"}],"source_content_type":"text/x-python","patch_set":3,"id":"cd2a28df_e4a6601c","line":1466,"in_reply_to":"f4ac7687_df207032","updated":"2021-03-23 08:37:35.000000000","message":"\u003e I think it\u0027s better to include docstrings because the returned value is special.\n\nAdded docstrings","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"9598b7b9a8a90e5645a0ea7b8d2083c2eb43ac72","unresolved":true,"context_lines":[{"line_number":1513,"context_line":""},{"line_number":1514,"context_line":"                    for num in range(CONF.vnf_lcm.retry_num):"},{"line_number":1515,"context_line":"                        try:"},{"line_number":1516,"context_line":"                            LOG.warn(\"send notify[%s]\" %"},{"line_number":1517,"context_line":"                                json.dumps(notification))"},{"line_number":1518,"context_line":"                            auth_client \u003d auth.auth_manager.get_auth_client("},{"line_number":1519,"context_line":"                                notification[\u0027subscriptionId\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"53398873_1df6090b","line":1516,"range":{"start_line":1516,"start_character":28,"end_line":1516,"end_character":36},"updated":"2021-03-22 06:46:55.000000000","message":"Isn\u0027t \u0027LOG.info\u0027 more appropriate?","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"29221f940149ea0c3875543467717906be465505","unresolved":true,"context_lines":[{"line_number":1513,"context_line":""},{"line_number":1514,"context_line":"                    for num in range(CONF.vnf_lcm.retry_num):"},{"line_number":1515,"context_line":"                        try:"},{"line_number":1516,"context_line":"                            LOG.warn(\"send notify[%s]\" %"},{"line_number":1517,"context_line":"                                json.dumps(notification))"},{"line_number":1518,"context_line":"                            auth_client \u003d auth.auth_manager.get_auth_client("},{"line_number":1519,"context_line":"                                notification[\u0027subscriptionId\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"b29182e0_904091de","line":1516,"range":{"start_line":1516,"start_character":28,"end_line":1516,"end_character":36},"in_reply_to":"53398873_1df6090b","updated":"2021-03-23 08:37:35.000000000","message":"\u003e Isn\u0027t \u0027LOG.info\u0027 more appropriate?\n\nChanged to LOG.info","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"0ce34702d98bea0118223769624aeb383b09bc4b","unresolved":false,"context_lines":[{"line_number":1513,"context_line":""},{"line_number":1514,"context_line":"                    for num in range(CONF.vnf_lcm.retry_num):"},{"line_number":1515,"context_line":"                        try:"},{"line_number":1516,"context_line":"                            LOG.warn(\"send notify[%s]\" %"},{"line_number":1517,"context_line":"                                json.dumps(notification))"},{"line_number":1518,"context_line":"                            auth_client \u003d auth.auth_manager.get_auth_client("},{"line_number":1519,"context_line":"                                notification[\u0027subscriptionId\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba5ab298_1346d3f5","line":1516,"range":{"start_line":1516,"start_character":28,"end_line":1516,"end_character":36},"in_reply_to":"b29182e0_904091de","updated":"2021-03-24 00:47:28.000000000","message":"I\u0027ve confirmed.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"3906090fc51d66fb2eaa2f0ddb41c9c0090a3136","unresolved":true,"context_lines":[{"line_number":1522,"context_line":"                                data\u003djson.dumps(notification),"},{"line_number":1523,"context_line":"                                timeout\u003dCONF.vnf_lcm.retry_timeout)"},{"line_number":1524,"context_line":"                            if response.status_code \u003d\u003d 204:"},{"line_number":1525,"context_line":"                                LOG.info("},{"line_number":1526,"context_line":"                                    \"send success notify[%s]\" %"},{"line_number":1527,"context_line":"                                    json.dumps(notification))"},{"line_number":1528,"context_line":"                                break"},{"line_number":1529,"context_line":"                            else:"},{"line_number":1530,"context_line":"                                LOG.warning("},{"line_number":1531,"context_line":"                                    \"Notification failed id[%s] status[%s] \\"},{"line_number":1532,"context_line":"                                        callback_uri[%s]\" %"},{"line_number":1533,"context_line":"                                    (notification[\u0027id\u0027],"},{"line_number":1534,"context_line":"                                    response.status_code,"},{"line_number":1535,"context_line":"                                    line.callback_uri))"},{"line_number":1536,"context_line":"                                LOG.debug("},{"line_number":1537,"context_line":"                                    \"retry_wait %s\" %"},{"line_number":1538,"context_line":"                                    CONF.vnf_lcm.retry_wait)"},{"line_number":1539,"context_line":"                                self._retry_check(num)"},{"line_number":1540,"context_line":""},{"line_number":1541,"context_line":"                                continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"21b59618_8b4d0e5a","line":1538,"range":{"start_line":1525,"start_character":0,"end_line":1538,"end_character":60},"updated":"2021-03-22 02:29:31.000000000","message":"Could you revise log messages?\n* Unify style of messages.\n* No need to use `%` as `format()` for LOG.*.\n* Is backslash required?","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"29221f940149ea0c3875543467717906be465505","unresolved":true,"context_lines":[{"line_number":1522,"context_line":"                                data\u003djson.dumps(notification),"},{"line_number":1523,"context_line":"                                timeout\u003dCONF.vnf_lcm.retry_timeout)"},{"line_number":1524,"context_line":"                            if response.status_code \u003d\u003d 204:"},{"line_number":1525,"context_line":"                                LOG.info("},{"line_number":1526,"context_line":"                                    \"send success notify[%s]\" %"},{"line_number":1527,"context_line":"                                    json.dumps(notification))"},{"line_number":1528,"context_line":"                                break"},{"line_number":1529,"context_line":"                            else:"},{"line_number":1530,"context_line":"                                LOG.warning("},{"line_number":1531,"context_line":"                                    \"Notification failed id[%s] status[%s] \\"},{"line_number":1532,"context_line":"                                        callback_uri[%s]\" %"},{"line_number":1533,"context_line":"                                    (notification[\u0027id\u0027],"},{"line_number":1534,"context_line":"                                    response.status_code,"},{"line_number":1535,"context_line":"                                    line.callback_uri))"},{"line_number":1536,"context_line":"                                LOG.debug("},{"line_number":1537,"context_line":"                                    \"retry_wait %s\" %"},{"line_number":1538,"context_line":"                                    CONF.vnf_lcm.retry_wait)"},{"line_number":1539,"context_line":"                                self._retry_check(num)"},{"line_number":1540,"context_line":""},{"line_number":1541,"context_line":"                                continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"77006bbc_207beb0c","line":1538,"range":{"start_line":1525,"start_character":0,"end_line":1538,"end_character":60},"in_reply_to":"21b59618_8b4d0e5a","updated":"2021-03-23 08:37:35.000000000","message":"\u003e Could you revise log messages?\n\u003e * Unify style of messages.\n\u003e * No need to use `%` as `format()` for LOG.*.\n\u003e * Is backslash required?\n\nUpdated log messages to not use \"%\" as formatter and \"\\\"","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"3906090fc51d66fb2eaa2f0ddb41c9c0090a3136","unresolved":true,"context_lines":[{"line_number":1546,"context_line":"                                    \"id\": notification[\u0027id\u0027],"},{"line_number":1547,"context_line":"                                    \"uri\": line.callback_uri,"},{"line_number":1548,"context_line":"                                    \"reason\": str(e)})"},{"line_number":1549,"context_line":"                            self._retry_check(num)"},{"line_number":1550,"context_line":"                except Exception as e:"},{"line_number":1551,"context_line":"                    LOG.warn(\"send error[%s]\" % str(e))"},{"line_number":1552,"context_line":"                    LOG.warn(traceback.format_exc())"}],"source_content_type":"text/x-python","patch_set":3,"id":"2d11c82b_08fef4dd","line":1549,"range":{"start_line":1549,"start_character":28,"end_line":1549,"end_character":50},"updated":"2021-03-22 02:29:31.000000000","message":"Is it required here even though it\u0027s warned timeout before?","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"29221f940149ea0c3875543467717906be465505","unresolved":true,"context_lines":[{"line_number":1546,"context_line":"                                    \"id\": notification[\u0027id\u0027],"},{"line_number":1547,"context_line":"                                    \"uri\": line.callback_uri,"},{"line_number":1548,"context_line":"                                    \"reason\": str(e)})"},{"line_number":1549,"context_line":"                            self._retry_check(num)"},{"line_number":1550,"context_line":"                except Exception as e:"},{"line_number":1551,"context_line":"                    LOG.warn(\"send error[%s]\" % str(e))"},{"line_number":1552,"context_line":"                    LOG.warn(traceback.format_exc())"}],"source_content_type":"text/x-python","patch_set":3,"id":"4641b9b0_da97eabe","line":1549,"range":{"start_line":1549,"start_character":28,"end_line":1549,"end_character":50},"in_reply_to":"2d11c82b_08fef4dd","updated":"2021-03-23 08:37:35.000000000","message":"\u003e Is it required here even though it\u0027s warned timeout before?\n\nYes, this is required here. Same pattern with test_notification, when a timeout exception occurs there will be a sleep then a checking if the retry count exceeds the number of retries possible.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"6c14eff9fa61bfd0d5ba98268ec6d9dc75679d48","unresolved":false,"context_lines":[{"line_number":1546,"context_line":"                                    \"id\": notification[\u0027id\u0027],"},{"line_number":1547,"context_line":"                                    \"uri\": line.callback_uri,"},{"line_number":1548,"context_line":"                                    \"reason\": str(e)})"},{"line_number":1549,"context_line":"                            self._retry_check(num)"},{"line_number":1550,"context_line":"                except Exception as e:"},{"line_number":1551,"context_line":"                    LOG.warn(\"send error[%s]\" % str(e))"},{"line_number":1552,"context_line":"                    LOG.warn(traceback.format_exc())"}],"source_content_type":"text/x-python","patch_set":3,"id":"d375c0ba_6633fc83","line":1549,"range":{"start_line":1549,"start_character":28,"end_line":1549,"end_character":50},"in_reply_to":"4641b9b0_da97eabe","updated":"2021-03-24 08:50:30.000000000","message":"Ack","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"6b34575eb848a2d088cfd9a1ed181d5458e1d428","unresolved":false,"context_lines":[{"line_number":1546,"context_line":"                                    \"id\": notification[\u0027id\u0027],"},{"line_number":1547,"context_line":"                                    \"uri\": line.callback_uri,"},{"line_number":1548,"context_line":"                                    \"reason\": str(e)})"},{"line_number":1549,"context_line":"                            self._retry_check(num)"},{"line_number":1550,"context_line":"                except Exception as e:"},{"line_number":1551,"context_line":"                    LOG.warn(\"send error[%s]\" % str(e))"},{"line_number":1552,"context_line":"                    LOG.warn(traceback.format_exc())"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ea645ef_7f98cf2a","line":1549,"range":{"start_line":1549,"start_character":28,"end_line":1549,"end_character":50},"in_reply_to":"622918dd_aeb3ea45","updated":"2021-03-25 10:11:53.000000000","message":"Are you asking about `ACK`? I\u0027m using `DONE` if something is done and fixed, or using `ACK` if I just understand something for which nothing is done. Anyway, this comment is resolved.","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"55ac51778f2c5fd627ecdf81c9b255bba35ed929","unresolved":false,"context_lines":[{"line_number":1546,"context_line":"                                    \"id\": notification[\u0027id\u0027],"},{"line_number":1547,"context_line":"                                    \"uri\": line.callback_uri,"},{"line_number":1548,"context_line":"                                    \"reason\": str(e)})"},{"line_number":1549,"context_line":"                            self._retry_check(num)"},{"line_number":1550,"context_line":"                except Exception as e:"},{"line_number":1551,"context_line":"                    LOG.warn(\"send error[%s]\" % str(e))"},{"line_number":1552,"context_line":"                    LOG.warn(traceback.format_exc())"}],"source_content_type":"text/x-python","patch_set":3,"id":"622918dd_aeb3ea45","line":1549,"range":{"start_line":1549,"start_character":28,"end_line":1549,"end_character":50},"in_reply_to":"d375c0ba_6633fc83","updated":"2021-03-24 09:18:10.000000000","message":"Sorry, but may I know what you mean here?","commit_id":"0ada2b6950386a362b5d3b2eccde3d6880ae2627"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"6c14eff9fa61bfd0d5ba98268ec6d9dc75679d48","unresolved":true,"context_lines":[{"line_number":1538,"context_line":"                                timeout\u003dCONF.vnf_lcm.retry_timeout)"},{"line_number":1539,"context_line":"                            if response.status_code \u003d\u003d 204:"},{"line_number":1540,"context_line":"                                LOG.info("},{"line_number":1541,"context_line":"                                    \"send success notify[%s]\" %"},{"line_number":1542,"context_line":"                                    json.dumps(notification))"},{"line_number":1543,"context_line":"                                break"},{"line_number":1544,"context_line":"                            else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"975000c5_27d6b59e","line":1541,"range":{"start_line":1541,"start_character":61,"end_line":1541,"end_character":63},"updated":"2021-03-24 08:50:30.000000000","message":"It\u0027s still remained.","commit_id":"40e6c878d3293eae654ee2c275d9f212e7a1850a"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"5858ee87ae75d1d7a9d74220e2d602b42ff2b6e1","unresolved":false,"context_lines":[{"line_number":1538,"context_line":"                                timeout\u003dCONF.vnf_lcm.retry_timeout)"},{"line_number":1539,"context_line":"                            if response.status_code \u003d\u003d 204:"},{"line_number":1540,"context_line":"                                LOG.info("},{"line_number":1541,"context_line":"                                    \"send success notify[%s]\" %"},{"line_number":1542,"context_line":"                                    json.dumps(notification))"},{"line_number":1543,"context_line":"                                break"},{"line_number":1544,"context_line":"                            else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"cb0ca607_334fefbf","line":1541,"range":{"start_line":1541,"start_character":61,"end_line":1541,"end_character":63},"in_reply_to":"91be3767_6a166137","updated":"2021-03-25 10:15:20.000000000","message":"Done","commit_id":"40e6c878d3293eae654ee2c275d9f212e7a1850a"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"8f4dcd4778a192c71f211642116dcfed3b10b93d","unresolved":true,"context_lines":[{"line_number":1538,"context_line":"                                timeout\u003dCONF.vnf_lcm.retry_timeout)"},{"line_number":1539,"context_line":"                            if response.status_code \u003d\u003d 204:"},{"line_number":1540,"context_line":"                                LOG.info("},{"line_number":1541,"context_line":"                                    \"send success notify[%s]\" %"},{"line_number":1542,"context_line":"                                    json.dumps(notification))"},{"line_number":1543,"context_line":"                                break"},{"line_number":1544,"context_line":"                            else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"91be3767_6a166137","line":1541,"range":{"start_line":1541,"start_character":61,"end_line":1541,"end_character":63},"in_reply_to":"975000c5_27d6b59e","updated":"2021-03-25 07:56:56.000000000","message":"\u003e It\u0027s still remained.\n\nUpdated log messages to not use \"%\" as formatter.","commit_id":"40e6c878d3293eae654ee2c275d9f212e7a1850a"}]}