)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"0669f3c9c9de92865294f1f26bee6e64fd42366c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tony Breeds \u003ctony@bakeyournoodle.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-12-08 10:18:27 +1100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch from openstack.common olso_concurrency for lockutils"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The aim is to remove the last user of posix-ipc so it can be removed from"},{"line_number":10,"context_line":"global-requirements.txt."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3a71b18c_94d40b48","line":7,"updated":"2016-12-08 12:44:27.000000000","message":"Missing \"to\" here: \"from openstack.common to oslo_concurrency\"\n\nThe first commit line is also longer than 50 characters, you could drop \"for lockutils\".","commit_id":"280b462f4f2bf420f3a77edb749d0f8e303c9e05"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"0669f3c9c9de92865294f1f26bee6e64fd42366c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch from openstack.common olso_concurrency for lockutils"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The aim is to remove the last user of posix-ipc so it can be removed from"},{"line_number":10,"context_line":"global-requirements.txt."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit also re-generates the sample config to reflect the config"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3a71b18c_d4868312","line":9,"updated":"2016-12-08 12:44:27.000000000","message":"This line is too long, please wrap to 72 characters.","commit_id":"280b462f4f2bf420f3a77edb749d0f8e303c9e05"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"0669f3c9c9de92865294f1f26bee6e64fd42366c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"global-requirements.txt."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit also re-generates the sample config to reflect the config"},{"line_number":13,"context_line":"group change for \u0027disable_process_locking\u0027 and \u0027lock_path\u0027"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I777424370dc02c3164803a63331a1befb8531de5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3a71b18c_74876f12","line":13,"updated":"2016-12-08 12:44:27.000000000","message":"Missing dot at the end of the sentence.","commit_id":"280b462f4f2bf420f3a77edb749d0f8e303c9e05"}],"climate/config.py":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"e02cb15d9b77bb88a842207818ee1f507c27553b","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# Import oslo_concurrency.lockutils to get the oslo_concurrency group"},{"line_number":17,"context_line":"import oslo_concurrency.lockutils  # noqa: F401"},{"line_number":18,"context_line":"import socket"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a71b18c_6d5afd2b","line":17,"updated":"2016-12-05 02:11:12.000000000","message":"I\u0027m not 100% certain that this is needed but it certainly means that those options are visible when loading climate/config.py","commit_id":"a21449ebdef4066e7727c6f2f403b9d5313f4d3a"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"fe0a4a73994abfeabf6bbb0b31bc0eee55fd066f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# Import oslo_concurrency.lockutils to get the oslo_concurrency group"},{"line_number":17,"context_line":"import oslo_concurrency.lockutils  # noqa: F401"},{"line_number":18,"context_line":"import socket"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a71b18c_947ad5ef","line":17,"in_reply_to":"3a71b18c_1fa99d24","updated":"2016-12-05 22:04:23.000000000","message":"I couldn\u0027t convince myself that *all* the config options would be generated correctly without this.  If you can provide an command line that works to generate the sample config I can happily dig into this but without that I erred on the side of caution.","commit_id":"a21449ebdef4066e7727c6f2f403b9d5313f4d3a"},{"author":{"_account_id":8878,"name":"Masahito Muroi","email":"masahito.muroi@g.softbank.co.jp","username":"masa"},"change_message_id":"9284b5cdd2552b948adf1e79ec41a36578dfbe5f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# Import oslo_concurrency.lockutils to get the oslo_concurrency group"},{"line_number":17,"context_line":"import oslo_concurrency.lockutils  # noqa: F401"},{"line_number":18,"context_line":"import socket"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a71b18c_1fa99d24","line":17,"in_reply_to":"3a71b18c_6d5afd2b","updated":"2016-12-05 11:54:48.000000000","message":"When it\u0027s removed from here, does it cause any error?  If not so, we can remove this line.","commit_id":"a21449ebdef4066e7727c6f2f403b9d5313f4d3a"}]}