)]}'
{"tools/cleanup-containers":[{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"496a6abbf95d97c9e7840cf0706a1e3a4ec412e2","unresolved":false,"context_lines":[{"line_number":69,"context_line":"echo \"Removing volumes...\""},{"line_number":70,"context_line":"(docker volume rm ${volumes_to_remove[@]} 2\u003e\u00261) \u003e /dev/null"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"echo \"Creating a fstab backup...\""},{"line_number":73,"context_line":"cp /etc/fstab /etc/fstab_backup"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":76,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab #Clean all lines containing ceph"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"echo \"All cleaned up!\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba0121b8_e462f8ac","line":76,"range":{"start_line":72,"start_character":0,"end_line":76,"end_character":65},"updated":"2016-03-29 00:56:30.000000000","message":"Since this is host-level cleanup, could you move this to https://github.com/openstack/kolla/blob/master/tools/cleanup-host.","commit_id":"be1a08d946f3a21509496567050dad4c497cfea8"}],"tools/cleanup-host":[{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"f0655088f9107889175f662d4a413d248625f4b0","unresolved":false,"context_lines":[{"line_number":28,"context_line":"done"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"echo \"Creating a fstab backup...\""},{"line_number":31,"context_line":"cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab #Clean all lines containing ceph"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"ba0121b8_82992222","line":31,"updated":"2016-03-30 09:12:58.000000000","message":"Don\u0027t you need sudo to do this?","commit_id":"dae8e027677f53363df1db8fa1a805b95efed18e"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"26851ad2239e45ce65599515cdffa74db19daab0","unresolved":false,"context_lines":[{"line_number":28,"context_line":"done"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"echo \"Creating a fstab backup...\""},{"line_number":31,"context_line":"cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab #Clean all lines containing ceph"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"ba0121b8_124d553e","line":31,"in_reply_to":"ba0121b8_82992222","updated":"2016-03-30 11:59:36.000000000","message":"Done","commit_id":"dae8e027677f53363df1db8fa1a805b95efed18e"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"f0655088f9107889175f662d4a413d248625f4b0","unresolved":false,"context_lines":[{"line_number":31,"context_line":"cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab #Clean all lines containing ceph"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"ba0121b8_82c002e2","line":34,"updated":"2016-03-30 09:12:58.000000000","message":"I think the comment is unnecessary but if you want to leave it in please add a space after the #","commit_id":"dae8e027677f53363df1db8fa1a805b95efed18e"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"26851ad2239e45ce65599515cdffa74db19daab0","unresolved":false,"context_lines":[{"line_number":31,"context_line":"cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab #Clean all lines containing ceph"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"ba0121b8_325219de","line":34,"in_reply_to":"ba0121b8_82c002e2","updated":"2016-03-30 11:59:36.000000000","message":"Done","commit_id":"dae8e027677f53363df1db8fa1a805b95efed18e"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"0c33579058888b19560c49d15334323214250cfe","unresolved":false,"context_lines":[{"line_number":31,"context_line":"sudo cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"ba0121b8_a9f17b03","line":34,"updated":"2016-03-30 22:18:30.000000000","message":"I\u0027d like to see lines that are removed (perhaps with grep), just to make it more obvious if something got removed that shouldn\u0027t have.","commit_id":"6be3d82d1cbaea72abb079b3392efc659768889b"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"d55dcbd090c4d29a78317b0dcc3641bf5644caad","unresolved":false,"context_lines":[{"line_number":31,"context_line":"sudo cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"ba0121b8_21de1fdd","line":34,"updated":"2016-03-31 00:08:56.000000000","message":"could we use a more accurate pattern here? Simple ceph may cause some incorrect delete.\n\nuse like\n    sed -i \u0027/\\/var\\/lib\\/ceph\\/osd\\/d\u0027 /etc/fstab","commit_id":"6be3d82d1cbaea72abb079b3392efc659768889b"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"ab90342cd44b19e021ccfdd45f32ff2b855edf30","unresolved":false,"context_lines":[{"line_number":31,"context_line":"sudo cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"ba0121b8_e115dc49","line":34,"in_reply_to":"ba0121b8_21de1fdd","updated":"2016-04-01 20:00:41.000000000","message":"Done","commit_id":"6be3d82d1cbaea72abb079b3392efc659768889b"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"69c2ce2266ba75d1d74e2a876d533cdbb10a337d","unresolved":false,"context_lines":[{"line_number":31,"context_line":"sudo cp /etc/fstab /etc/fstab_backup"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo \"Removing ceph references from fstab...\""},{"line_number":34,"context_line":"sudo sed -i \u0027/ceph/d\u0027 /etc/fstab"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"ba0121b8_01895bc9","line":34,"in_reply_to":"ba0121b8_21de1fdd","updated":"2016-03-31 00:10:51.000000000","message":"sorry but I can not see how it can go wrong.","commit_id":"6be3d82d1cbaea72abb079b3392efc659768889b"}]}