)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"c331f08b7f80edaf389507cc6b7814791eb9016f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"change of deployment when they should be created. Adding them"},{"line_number":12,"context_line":"in plan_management update_deployment_plan fixes these cases."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Resolves: rhbz#1455616"},{"line_number":15,"context_line":"Change-Id: Ifcedd54da5c74f12c863b8ec63679360673dd8f6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"df140735_5f017537","line":14,"updated":"2017-06-05 12:50:44.000000000","message":"This should really reference an upstream bug.","commit_id":"18a45d814b67eb6ec5cf35722a408ad90c8b5729"}],"workbooks/deployment.yaml":[{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"cd81daa90320a880c48639561e5a6003ea247cd8","unresolved":false,"context_lines":[{"line_number":140,"context_line":"          status: FAILED"},{"line_number":141,"context_line":"          message: \u003c% task(run_validations).result %\u003e"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"      call_create_swift_rings_backup_plan:"},{"line_number":144,"context_line":"        workflow: tripleo.swift_rings_backup.v1.create_swift_rings_backup_container_plan"},{"line_number":145,"context_line":"        on-success: get_heat_stack"},{"line_number":146,"context_line":"        on-error: call_create_swift_rings_backup_plan_set_status_failed"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_462bf881","line":143,"range":{"start_line":143,"start_character":6,"end_line":143,"end_character":41},"updated":"2017-06-05 05:17:03.000000000","message":"I dont see this task being invoked anywhere, I assume that you want it to be called instead of check_container.","commit_id":"f2a97b2af3d9e32b3ca7723540f00a69bbac6f09"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"cd81daa90320a880c48639561e5a6003ea247cd8","unresolved":false,"context_lines":[{"line_number":140,"context_line":"          status: FAILED"},{"line_number":141,"context_line":"          message: \u003c% task(run_validations).result %\u003e"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"      call_create_swift_rings_backup_plan:"},{"line_number":144,"context_line":"        workflow: tripleo.swift_rings_backup.v1.create_swift_rings_backup_container_plan"},{"line_number":145,"context_line":"        on-success: get_heat_stack"},{"line_number":146,"context_line":"        on-error: call_create_swift_rings_backup_plan_set_status_failed"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_a66774e9","line":143,"range":{"start_line":143,"start_character":6,"end_line":143,"end_character":11},"updated":"2017-06-05 05:17:03.000000000","message":"create_swift_rings_backup_plan would be a good name rather than prefixing it with \"call\"","commit_id":"f2a97b2af3d9e32b3ca7723540f00a69bbac6f09"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"cd81daa90320a880c48639561e5a6003ea247cd8","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        on-error: set_deployment_failed"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"      call_create_swift_rings_backup_plan_set_status_failed:"},{"line_number":175,"context_line":"        on-success: notify_zaqar"},{"line_number":176,"context_line":"        publish:"},{"line_number":177,"context_line":"          status: FAILED"},{"line_number":178,"context_line":"          message: \u003c% task(call_create_swift_rings_backup_plan).result %\u003e"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_06a4e006","line":175,"updated":"2017-06-05 05:17:03.000000000","message":"It should be send_message?","commit_id":"f2a97b2af3d9e32b3ca7723540f00a69bbac6f09"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"5f2f016e1e8f522087aa3c8b9d32035699999968","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        on-error: set_deployment_failed"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"      create_swift_rings_backup_plan_set_status_failed:"},{"line_number":175,"context_line":"        on-success: notify_zaqar"},{"line_number":176,"context_line":"        publish:"},{"line_number":177,"context_line":"          status: FAILED"},{"line_number":178,"context_line":"          message: \u003c% task(create_swift_rings_backup_plan).result %\u003e"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df140735_d3d22728","line":175,"range":{"start_line":175,"start_character":20,"end_line":175,"end_character":32},"updated":"2017-06-05 09:31:13.000000000","message":"notify_zaqar -\u003e send_message","commit_id":"43149c244c641669c483879c1a3bc932be9028dc"}],"workbooks/plan_management.yaml":[{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"cd81daa90320a880c48639561e5a6003ea247cd8","unresolved":false,"context_lines":[{"line_number":217,"context_line":""},{"line_number":218,"context_line":"      upload_templates_directory:"},{"line_number":219,"context_line":"        action: tripleo.templates.upload container\u003d\u003c% $.container %\u003e templates_path\u003d\u003c% task(clone_git_repo).result %\u003e"},{"line_number":220,"context_line":"        on-success: check_container"},{"line_number":221,"context_line":"        on-complete: cleanup_temporary_files"},{"line_number":222,"context_line":"        on-error: upload_templates_directory_set_status_failed"},{"line_number":223,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_86b7d03a","line":220,"range":{"start_line":220,"start_character":20,"end_line":220,"end_character":35},"updated":"2017-06-05 05:17:03.000000000","message":"Should be call_create_swift_rings_backup_plan IIUC?","commit_id":"f2a97b2af3d9e32b3ca7723540f00a69bbac6f09"}],"workbooks/swift_rings_backup.yaml":[{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"cd81daa90320a880c48639561e5a6003ea247cd8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"version: \u00272.0\u0027"},{"line_number":3,"context_line":"name: tripleo.swift_rings_backup.v1"},{"line_number":4,"context_line":"description: TripleO Swift Rings Deploymetn Workflow v1"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"workflows:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_c69a68b7","line":3,"range":{"start_line":3,"start_character":14,"end_line":3,"end_character":32},"updated":"2017-06-05 05:17:03.000000000","message":"Does the \"backup\" word has any significance or we can call it swift_rings?","commit_id":"f2a97b2af3d9e32b3ca7723540f00a69bbac6f09"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"cd81daa90320a880c48639561e5a6003ea247cd8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"version: \u00272.0\u0027"},{"line_number":3,"context_line":"name: tripleo.swift_rings_backup.v1"},{"line_number":4,"context_line":"description: TripleO Swift Rings Deploymetn Workflow v1"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"workflows:"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_268a4469","line":4,"range":{"start_line":4,"start_character":33,"end_line":4,"end_character":43},"updated":"2017-06-05 05:17:03.000000000","message":"spelling","commit_id":"f2a97b2af3d9e32b3ca7723540f00a69bbac6f09"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6b811751219ca23a0585eabf01fc8c3efcc2df35","unresolved":false,"context_lines":[{"line_number":72,"context_line":"          status: FAILED"},{"line_number":73,"context_line":"          message: \u003c% task(create_container_failed).result %\u003e"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"      notify_zaqar:"},{"line_number":76,"context_line":"        action: zaqar.queue_post"},{"line_number":77,"context_line":"        input:"},{"line_number":78,"context_line":"          queue_name: \u003c% $.queue_name %\u003e"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df140735_6fb6ac8c","line":75,"range":{"start_line":75,"start_character":6,"end_line":75,"end_character":18},"updated":"2017-06-06 07:18:40.000000000","message":"beyond the scope of this patch, but it looks like this notify_zaqar block is identical in all three files here would be great if we could tidy that if it is possible","commit_id":"bfabb3f5755628f707be58b1ead41b2f799ca099"}]}
