)]}'
{"sahara/service/ops.py":[{"author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"change_message_id":"d59bee9ba545beb6b9f931cf77734cbbcb044268","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            cluster \u003d conductor.cluster_get(ctx, cluster_id)"},{"line_number":146,"context_line":"            # check if cluster still exists (it might have been removed)"},{"line_number":147,"context_line":"            if cluster is None or cluster.status \u003d\u003d \u0027Deleting\u0027:"},{"line_number":148,"context_line":"                LOG.info(_LI(\"Cluster with %s was deleted. Canceling current \""},{"line_number":149,"context_line":"                             \"operation.\"),  cluster_id)"},{"line_number":150,"context_line":"                return"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_d726cbb9","line":148,"updated":"2014-11-21 07:26:37.000000000","message":"I think, that it\u0027s strange to see the message \"Cluster with %s was deleted\" in logs for \"Deleting\" cluster.","commit_id":"32a0d05468dbd9c257637f801f5c9d66b68bf39a"},{"author":{"_account_id":8411,"name":"Andrew Lazarev","email":"alazarev@mirantis.com","username":"alazarev"},"change_message_id":"66bdcfe4f9f2d890bcbf53a0feaffb7e7c9ee0ef","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            cluster \u003d conductor.cluster_get(ctx, cluster_id)"},{"line_number":146,"context_line":"            # check if cluster still exists (it might have been removed)"},{"line_number":147,"context_line":"            if cluster is None or cluster.status \u003d\u003d \u0027Deleting\u0027:"},{"line_number":148,"context_line":"                LOG.info(_LI(\"Cluster with %s was deleted. Canceling current \""},{"line_number":149,"context_line":"                             \"operation.\"),  cluster_id)"},{"line_number":150,"context_line":"                return"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_037f2267","line":148,"in_reply_to":"5a890539_9d2c5304","updated":"2014-11-21 17:00:16.000000000","message":"Done","commit_id":"32a0d05468dbd9c257637f801f5c9d66b68bf39a"},{"author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"change_message_id":"e5bf62d1b79f10e8110b3d2144defc982907977d","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            cluster \u003d conductor.cluster_get(ctx, cluster_id)"},{"line_number":146,"context_line":"            # check if cluster still exists (it might have been removed)"},{"line_number":147,"context_line":"            if cluster is None or cluster.status \u003d\u003d \u0027Deleting\u0027:"},{"line_number":148,"context_line":"                LOG.info(_LI(\"Cluster with %s was deleted. Canceling current \""},{"line_number":149,"context_line":"                             \"operation.\"),  cluster_id)"},{"line_number":150,"context_line":"                return"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_9d2c5304","line":148,"in_reply_to":"5a890539_aae0e4cc","updated":"2014-11-21 16:15:33.000000000","message":"Yes, I suggest \"was marked for deletion or deleted\". But it is just a nit","commit_id":"32a0d05468dbd9c257637f801f5c9d66b68bf39a"},{"author":{"_account_id":8411,"name":"Andrew Lazarev","email":"alazarev@mirantis.com","username":"alazarev"},"change_message_id":"95ecbee68bcc65f25fd9cc3eb342a2dc4edfd050","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            cluster \u003d conductor.cluster_get(ctx, cluster_id)"},{"line_number":146,"context_line":"            # check if cluster still exists (it might have been removed)"},{"line_number":147,"context_line":"            if cluster is None or cluster.status \u003d\u003d \u0027Deleting\u0027:"},{"line_number":148,"context_line":"                LOG.info(_LI(\"Cluster with %s was deleted. Canceling current \""},{"line_number":149,"context_line":"                             \"operation.\"),  cluster_id)"},{"line_number":150,"context_line":"                return"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_aae0e4cc","line":148,"in_reply_to":"5a890539_d726cbb9","updated":"2014-11-21 15:28:41.000000000","message":"Why not? I doing some operation that failed. The reason is that some part of cluster is missing already. What wrong with with \"was deleted\"? Do you suggest \"was marked for deletion or deleted\"?","commit_id":"32a0d05468dbd9c257637f801f5c9d66b68bf39a"}]}