)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2cdabcdfb8926f75f5bf02fadf70b1bb77c60029","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2019-05-21 17:39:49 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Default securit group may be in-use by resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie8e7e8f462011fb13f780892c4541bc1e79f7845"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_e3f7f7f4","line":8,"updated":"2019-06-05 07:05:17.000000000","message":"hmm, this can be happened in theory but not sure in pratical. So, could you provide more information such as gate logs, a logstash URL or some other ?","commit_id":"4a7c43a85ba12f59a4a8ddfca9417b9a66323837"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"4d97d1a91c2811ea607e8e9dbbb52150086c5d8f","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2019-05-21 17:39:49 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Default securit group may be in-use by resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie8e7e8f462011fb13f780892c4541bc1e79f7845"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bfb3d3c7_c695fa41","line":8,"updated":"2019-05-23 02:20:45.000000000","message":"nit: better to add commit message body","commit_id":"4a7c43a85ba12f59a4a8ddfca9417b9a66323837"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"cc6f14855d5f541238c3ccbd96ce6f762eee696a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2019-05-21 17:39:49 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Default securit group may be in-use by resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie8e7e8f462011fb13f780892c4541bc1e79f7845"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_f01b7619","line":8,"in_reply_to":"9fb8cfa7_168350ce","updated":"2019-07-02 16:20:23.000000000","message":"The case is here:\nhttps://review.opendev.org/#/c/659009/","commit_id":"4a7c43a85ba12f59a4a8ddfca9417b9a66323837"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"b9edc857a9319255a767ce50243113997d5bfe8a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2019-05-21 17:39:49 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Default securit group may be in-use by resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie8e7e8f462011fb13f780892c4541bc1e79f7845"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_168350ce","line":8,"in_reply_to":"9fb8cfa7_e3f7f7f4","updated":"2019-06-06 07:58:07.000000000","message":"Seems when using SDN this often happens","commit_id":"4a7c43a85ba12f59a4a8ddfca9417b9a66323837"}],"tempest/lib/common/dynamic_creds.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"4d97d1a91c2811ea607e8e9dbbb52150086c5d8f","unresolved":false,"context_lines":[{"line_number":462,"context_line":"                            creds.tenant_name)"},{"line_number":463,"context_line":"            except lib_exc.Conflict as e_conf:"},{"line_number":464,"context_line":"                LOG.warning(\"failed to cleanup tenant %s\u0027s secgroup, \""},{"line_number":465,"context_line":"                            \"it may still in use by other resources, \""},{"line_number":466,"context_line":"                            \"detail: %s.\","},{"line_number":467,"context_line":"                            creds.tenant_name,"},{"line_number":468,"context_line":"                            str(e_conf))"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_06855208","line":465,"range":{"start_line":465,"start_character":29,"end_line":465,"end_character":67},"updated":"2019-05-23 02:20:45.000000000","message":"I hesitate whether we should try again after sleep some time to ensure the secgroup is deleted finally.","commit_id":"4a7c43a85ba12f59a4a8ddfca9417b9a66323837"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2cdabcdfb8926f75f5bf02fadf70b1bb77c60029","unresolved":false,"context_lines":[{"line_number":462,"context_line":"                            creds.tenant_name)"},{"line_number":463,"context_line":"            except lib_exc.Conflict as e_conf:"},{"line_number":464,"context_line":"                LOG.warning(\"failed to cleanup tenant %s\u0027s secgroup, \""},{"line_number":465,"context_line":"                            \"it may still in use by other resources, \""},{"line_number":466,"context_line":"                            \"detail: %s.\","},{"line_number":467,"context_line":"                            creds.tenant_name,"},{"line_number":468,"context_line":"                            str(e_conf))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_83dcfb74","line":465,"updated":"2019-06-05 07:05:17.000000000","message":"I\u0027m not sure where it already sleeps. But I agree with zhufl. I think this should be done really carefully when we catch an error and just log it. So, I think adding a NOTE or comment is preferaeble.","commit_id":"4a7c43a85ba12f59a4a8ddfca9417b9a66323837"}]}
