)]}'
{"tempest/cmd/cleanup_service.py":[{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"66b97889ae867d58559f38727b30e959a869d70b","unresolved":false,"context_lines":[{"line_number":696,"context_line":"            try:"},{"line_number":697,"context_line":"                client.delete_subnetpool(pool[\u0027id\u0027])"},{"line_number":698,"context_line":"            except Exception:"},{"line_number":699,"context_line":"                LOG.exception(\"Delete Subnet Pool exception.\")"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"    def dry_run(self):"},{"line_number":702,"context_line":"        pools \u003d self.list()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_cd94f7ea","line":699,"range":{"start_line":699,"start_character":31,"end_line":699,"end_character":60},"updated":"2019-02-27 04:59:44.000000000","message":"I think it\u0027s better to add pool value, it will help in debugging which pool is throwing exception.","commit_id":"e681998023d9714da346d117e18ab673b5657f71"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5a5cb24a1ce9858ceafe9e4eddcee2cacfbc2038","unresolved":false,"context_lines":[{"line_number":696,"context_line":"            try:"},{"line_number":697,"context_line":"                client.delete_subnetpool(pool[\u0027id\u0027])"},{"line_number":698,"context_line":"            except Exception:"},{"line_number":699,"context_line":"                LOG.exception(\"Delete Subnet Pool exception.\")"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"    def dry_run(self):"},{"line_number":702,"context_line":"        pools \u003d self.list()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_81947837","line":699,"range":{"start_line":699,"start_character":31,"end_line":699,"end_character":60},"in_reply_to":"9fdfeff1_5603b442","updated":"2019-02-27 08:57:07.000000000","message":"https://review.openstack.org/#/c/639600/","commit_id":"e681998023d9714da346d117e18ab673b5657f71"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3312638028bfb5062c7667a0e17fba19f1252932","unresolved":false,"context_lines":[{"line_number":696,"context_line":"            try:"},{"line_number":697,"context_line":"                client.delete_subnetpool(pool[\u0027id\u0027])"},{"line_number":698,"context_line":"            except Exception:"},{"line_number":699,"context_line":"                LOG.exception(\"Delete Subnet Pool exception.\")"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"    def dry_run(self):"},{"line_number":702,"context_line":"        pools \u003d self.list()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_c6d57b44","line":699,"range":{"start_line":699,"start_character":31,"end_line":699,"end_character":60},"in_reply_to":"9fdfeff1_cd94f7ea","updated":"2019-03-01 04:24:01.000000000","message":"+1, nice idea. let\u0027s do it as separate change as martin started in 639600","commit_id":"e681998023d9714da346d117e18ab673b5657f71"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"c45947a885cfe1d575497e42f62da475882cb1f7","unresolved":false,"context_lines":[{"line_number":696,"context_line":"            try:"},{"line_number":697,"context_line":"                client.delete_subnetpool(pool[\u0027id\u0027])"},{"line_number":698,"context_line":"            except Exception:"},{"line_number":699,"context_line":"                LOG.exception(\"Delete Subnet Pool exception.\")"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"    def dry_run(self):"},{"line_number":702,"context_line":"        pools \u003d self.list()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_5603b442","line":699,"range":{"start_line":699,"start_character":31,"end_line":699,"end_character":60},"in_reply_to":"9fdfeff1_cd94f7ea","updated":"2019-02-27 08:13:56.000000000","message":"I agree, however, this log exception is the same in all other classes .. I\u0027d suggest making this change in a different review, let me write it.","commit_id":"e681998023d9714da346d117e18ab673b5657f71"}]}