)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"5638f3e45734673674d47ec28fc836d4f1624ee4","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9daec0eadc8dad3d37e03e7d69783af003af0916"},{"line_number":18,"context_line":"Closes-bug: #1598929"},{"line_number":19,"context_line":"Partially-implements: bp watcher"},{"line_number":20,"context_line":"Signed-off-by: Dave Walker (Daviey) \u003cemail@daviey.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_b177d79e","line":19,"range":{"start_line":19,"start_character":22,"end_line":19,"end_character":32},"updated":"2016-08-01 14:55:34.000000000","message":"Is there a bp for this? This link is not valid.","commit_id":"760da80d2913155b49e0e0a8e09d02b29ea9bce4"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"89dd1cfe2647a8a0d322770f66157a5ea5ba90ea","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9daec0eadc8dad3d37e03e7d69783af003af0916"},{"line_number":18,"context_line":"Closes-bug: #1598929"},{"line_number":19,"context_line":"Partially-implements: bp watcher"},{"line_number":20,"context_line":"Signed-off-by: Dave Walker (Daviey) \u003cemail@daviey.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_f8a76fa4","line":19,"range":{"start_line":19,"start_character":22,"end_line":19,"end_character":32},"in_reply_to":"bacf61ea_b177d79e","updated":"2016-08-01 15:54:11.000000000","message":"It isn\u0027t invalid, just returns multiple BP\u0027s.  It also has a bug attached to it.\n\nhttps://blueprints.launchpad.net/kolla/+spec/watcher\n\nPS. Does this really warrant a -1?","commit_id":"760da80d2913155b49e0e0a8e09d02b29ea9bce4"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"82ba0462754703637c6e9e48282855ab250f53be","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I9daec0eadc8dad3d37e03e7d69783af003af0916"},{"line_number":18,"context_line":"Closes-bug: #1598929"},{"line_number":19,"context_line":"Partially-implements: bp watcher"},{"line_number":20,"context_line":"Signed-off-by: Dave Walker (Daviey) \u003cemail@daviey.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_08443572","line":19,"range":{"start_line":19,"start_character":22,"end_line":19,"end_character":32},"in_reply_to":"bacf61ea_f8a76fa4","updated":"2016-08-03 17:24:02.000000000","message":"It is invalid, it should be \"blueprint watcher\" and not \"bp watcher\". The blueprint is available at https://blueprints.launchpad.net/kolla/+spec/watcher.","commit_id":"760da80d2913155b49e0e0a8e09d02b29ea9bce4"}],"ansible/inventory/multinode":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"51aa3f4b0522d8294a7e503d83fda8c2d9918977","unresolved":false,"context_lines":[{"line_number":99,"context_line":"[ceilometer:children]"},{"line_number":100,"context_line":"control"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"# Tempest"},{"line_number":103,"context_line":"[tempest:children]"},{"line_number":104,"context_line":"control"},{"line_number":105,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"bacf61ea_94d8e554","side":"PARENT","line":102,"updated":"2016-08-05 07:11:54.000000000","message":"please do not touch this in this PS. if you want to remove it,\ndo it in separate PS.","commit_id":"0ff90a6f10494f9f33d665a3367dfaf0486b94df"}],"etc/kolla/globals.yml":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"98120d5cc3f417f841f45c24a77c4f49a95e9313","unresolved":false,"context_lines":[{"line_number":122,"context_line":"#enable_neutron_lbaas: \"no\""},{"line_number":123,"context_line":"#enable_neutron_qos: \"no\""},{"line_number":124,"context_line":"#enable_swift: \"no\""},{"line_number":125,"context_line":"#enable_watcher: \"no\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"###################"},{"line_number":128,"context_line":"# Ceph options"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_c3e95f1c","line":125,"updated":"2016-07-05 12:06:54.000000000","message":"Please add a default value to ansible/group_vars/all.yml","commit_id":"41278c543567d7f5652b61685073932fdb43d3f6"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"61baf2cc246d449bbd8e0bf9f30bbf86452c246e","unresolved":false,"context_lines":[{"line_number":122,"context_line":"#enable_neutron_lbaas: \"no\""},{"line_number":123,"context_line":"#enable_neutron_qos: \"no\""},{"line_number":124,"context_line":"#enable_swift: \"no\""},{"line_number":125,"context_line":"#enable_watcher: \"no\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"###################"},{"line_number":128,"context_line":"# Ceph options"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_277e6eae","line":125,"in_reply_to":"1aa78d24_836d979a","updated":"2016-07-05 12:26:24.000000000","message":"I\u0027ve intentionally broken them out to make the reviews less heavy.","commit_id":"41278c543567d7f5652b61685073932fdb43d3f6"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"b724c3d7245c887ce51609994603da544e9e1303","unresolved":false,"context_lines":[{"line_number":122,"context_line":"#enable_neutron_lbaas: \"no\""},{"line_number":123,"context_line":"#enable_neutron_qos: \"no\""},{"line_number":124,"context_line":"#enable_swift: \"no\""},{"line_number":125,"context_line":"#enable_watcher: \"no\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"###################"},{"line_number":128,"context_line":"# Ceph options"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_836d979a","line":125,"in_reply_to":"1aa78d24_c3e95f1c","updated":"2016-07-05 12:09:19.000000000","message":"Change in all.yml is part of https://review.openstack.org/#/c/337556/. I think it makes sense to merge this patch with https://review.openstack.org/#/c/337556/.","commit_id":"41278c543567d7f5652b61685073932fdb43d3f6"}]}