)]}'
{"smaug/services/protection/clients/heat.py":[{"author":{"_account_id":20883,"name":"Yuval Brik","email":"yuval@brik.org.il","username":"yuvalbr"},"change_message_id":"14b878690e447da7a2fe8636a49a5a65f8339ed1","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"SERVICE \u003d \u0027heat\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"KEYSTONECLIENT_VERSION \u003d (2, 0)"},{"line_number":24,"context_line":"HEATCLIENT_VERSION \u003d \u00271\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_15288699","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":31},"updated":"2016-05-10 10:55:14.000000000","message":"move to keystone v3?","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"},{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"9b15a3e25dba2fe75c883a4c2f3925ce3e50adbe","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"SERVICE \u003d \u0027heat\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"KEYSTONECLIENT_VERSION \u003d (2, 0)"},{"line_number":24,"context_line":"HEATCLIENT_VERSION \u003d \u00271\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_b12e8140","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":31},"in_reply_to":"dab17558_15288699","updated":"2016-05-12 06:20:39.000000000","message":"Done","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"},{"author":{"_account_id":20883,"name":"Yuval Brik","email":"yuval@brik.org.il","username":"yuvalbr"},"change_message_id":"14b878690e447da7a2fe8636a49a5a65f8339ed1","unresolved":false,"context_lines":[{"line_number":24,"context_line":"HEATCLIENT_VERSION \u003d \u00271\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"def create(context, username, password, auth_url):"},{"line_number":28,"context_line":"    LOG.info(_LI(\u0027Creating heat client with url %s.\u0027), auth_url)"},{"line_number":29,"context_line":"    tenant_name \u003d context.project_name"},{"line_number":30,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_3554aa1f","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":50},"updated":"2016-05-10 10:55:14.000000000","message":"create recieves (context, conf) as parameters","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"},{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"9b15a3e25dba2fe75c883a4c2f3925ce3e50adbe","unresolved":false,"context_lines":[{"line_number":24,"context_line":"HEATCLIENT_VERSION \u003d \u00271\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"def create(context, username, password, auth_url):"},{"line_number":28,"context_line":"    LOG.info(_LI(\u0027Creating heat client with url %s.\u0027), auth_url)"},{"line_number":29,"context_line":"    tenant_name \u003d context.project_name"},{"line_number":30,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_9131bda4","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":50},"in_reply_to":"dab17558_3554aa1f","updated":"2016-05-12 06:20:39.000000000","message":"Done","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"},{"author":{"_account_id":20883,"name":"Yuval Brik","email":"yuval@brik.org.il","username":"yuvalbr"},"change_message_id":"14b878690e447da7a2fe8636a49a5a65f8339ed1","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        heat \u003d hc.Client(HEATCLIENT_VERSION, endpoint\u003dheat_endpoint,"},{"line_number":44,"context_line":"                         token\u003dauth_token)"},{"line_number":45,"context_line":"        return heat"},{"line_number":46,"context_line":"    except Exception:"},{"line_number":47,"context_line":"        LOG.error(_LE(\u0027Creating heat client with url %s.\u0027), auth_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_3502ea0b","line":47,"range":{"start_line":46,"start_character":0,"end_line":47,"end_character":69},"updated":"2016-05-10 10:55:14.000000000","message":"consider breaking down the try block\nmust reraise after exception","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"}],"smaug/services/protection/flows/create_restoration.py":[{"author":{"_account_id":20883,"name":"Yuval Brik","email":"yuval@brik.org.il","username":"yuvalbr"},"change_message_id":"14b878690e447da7a2fe8636a49a5a65f8339ed1","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    heat_client \u003d heat.create(context\u003dcontext, username\u003dusername,"},{"line_number":95,"context_line":"                              password\u003dpassword, auth_url\u003dtarget)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    # TODO(luobin): create a heat_template"},{"line_number":98,"context_line":"    heat_template \u003d None"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    ctx \u003d {\u0027context\u0027: context,"},{"line_number":101,"context_line":"           \u0027checkpoint\u0027: checkpoint,"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_f8017739","line":98,"range":{"start_line":97,"start_character":0,"end_line":98,"end_character":24},"updated":"2016-05-10 10:55:14.000000000","message":"isn\u0027t this blocking this patch?","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"},{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"9b15a3e25dba2fe75c883a4c2f3925ce3e50adbe","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    heat_client \u003d heat.create(context\u003dcontext, username\u003dusername,"},{"line_number":95,"context_line":"                              password\u003dpassword, auth_url\u003dtarget)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    # TODO(luobin): create a heat_template"},{"line_number":98,"context_line":"    heat_template \u003d None"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    ctx \u003d {\u0027context\u0027: context,"},{"line_number":101,"context_line":"           \u0027checkpoint\u0027: checkpoint,"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_710259b5","line":98,"range":{"start_line":97,"start_character":0,"end_line":98,"end_character":24},"in_reply_to":"dab17558_f8017739","updated":"2016-05-12 06:20:39.000000000","message":"Done","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"}],"smaug/services/protection/manager.py":[{"author":{"_account_id":20883,"name":"Yuval Brik","email":"yuval@brik.org.il","username":"yuvalbr"},"change_message_id":"14b878690e447da7a2fe8636a49a5a65f8339ed1","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        try:"},{"line_number":115,"context_line":"            checkpoint_collection \u003d provider.get_checkpoint_collection()"},{"line_number":116,"context_line":"            checkpoint \u003d checkpoint_collection.get(checkpoint_id)"},{"line_number":117,"context_line":"            checkpoint.status \u003d constants.CHECKPOINT_STATUS_PROTECTING"},{"line_number":118,"context_line":"            checkpoint.commit()"},{"line_number":119,"context_line":"        except Exception:"},{"line_number":120,"context_line":"            LOG.error(_LE(\"get checkpoint failed, checkpoint_id:%s\"),"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_e9cab3a0","line":117,"range":{"start_line":117,"start_character":0,"end_line":117,"end_character":70},"updated":"2016-05-10 10:55:14.000000000","message":"PROTECTING -\u003e RESTORING","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"},{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"9b15a3e25dba2fe75c883a4c2f3925ce3e50adbe","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        try:"},{"line_number":115,"context_line":"            checkpoint_collection \u003d provider.get_checkpoint_collection()"},{"line_number":116,"context_line":"            checkpoint \u003d checkpoint_collection.get(checkpoint_id)"},{"line_number":117,"context_line":"            checkpoint.status \u003d constants.CHECKPOINT_STATUS_PROTECTING"},{"line_number":118,"context_line":"            checkpoint.commit()"},{"line_number":119,"context_line":"        except Exception:"},{"line_number":120,"context_line":"            LOG.error(_LE(\"get checkpoint failed, checkpoint_id:%s\"),"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_31fc51b7","line":117,"range":{"start_line":117,"start_character":0,"end_line":117,"end_character":70},"in_reply_to":"1a122d0e_e9cab3a0","updated":"2016-05-12 06:20:39.000000000","message":"Done","commit_id":"ce877ade74bcfaa99aac9f9b336a31b137064c35"}]}