)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"7d115bf3907af5ac114aad65f72c125a4629d0df","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Julien Danjou \u003cjulien@danjou.info\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-07-10 16:03:47 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a check so configuration file is uptodate"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We added this check recently to Ceilometer, and it helps a lot to keep this"},{"line_number":10,"context_line":"file always up to date and not falling behind."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAP3%2F%2Fgg0%3D","line":7,"updated":"2013-07-17 07:20:08.000000000","message":"How about \"Check that the configuration file sample is up to date\"?","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"3fb5421b2a6908176abbd62a6d994332b8f47d9e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Julien Danjou \u003cjulien@danjou.info\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-07-10 16:03:47 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a check so configuration file is uptodate"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We added this check recently to Ceilometer, and it helps a lot to keep this"},{"line_number":10,"context_line":"file always up to date and not falling behind."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAP3%2F%2FgPo%3D","line":7,"in_reply_to":"AAAAP3%2F%2Fgg0%3D","updated":"2013-07-17 09:18:07.000000000","message":"Done","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"}],"etc/nova/nova.conf.sample":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d1816f4e0b35f18cb5de562055cf8c8f500471c8","unresolved":false,"context_lines":[{"line_number":99,"context_line":"#pybasedir\u003d/usr/lib/python/site-packages"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"# Directory where nova binaries are installed (string value)"},{"line_number":102,"context_line":"#bindir\u003d/usr/local/bin"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"# Top-level directory for maintaining nova\u0027s state (string"},{"line_number":105,"context_line":"# value)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAP3%2F%2FiZg%3D","line":102,"updated":"2013-07-16 20:52:39.000000000","message":"Any reason for losing the expando?","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"e83865808a9cab349a688790adb0416c8ab49e0c","unresolved":false,"context_lines":[{"line_number":99,"context_line":"#pybasedir\u003d/usr/lib/python/site-packages"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"# Directory where nova binaries are installed (string value)"},{"line_number":102,"context_line":"#bindir\u003d/usr/local/bin"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"# Top-level directory for maintaining nova\u0027s state (string"},{"line_number":105,"context_line":"# value)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAP3%2F%2FgJw%3D","line":102,"in_reply_to":"AAAAP3%2F%2Fggw%3D","updated":"2013-07-17 09:35:43.000000000","message":"If you have read nova/paths.py, this is exactly what\u0027s written::\nos.path.join(sys.prefix, \u0027local\u0027, \u0027bin\u0027)\n\nSo this shouldn\u0027t be a blocker for this patch.","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"7d115bf3907af5ac114aad65f72c125a4629d0df","unresolved":false,"context_lines":[{"line_number":99,"context_line":"#pybasedir\u003d/usr/lib/python/site-packages"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"# Directory where nova binaries are installed (string value)"},{"line_number":102,"context_line":"#bindir\u003d/usr/local/bin"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"# Top-level directory for maintaining nova\u0027s state (string"},{"line_number":105,"context_line":"# value)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAP3%2F%2Fggw%3D","line":102,"in_reply_to":"AAAAP3%2F%2FiZg%3D","updated":"2013-07-17 07:20:08.000000000","message":"I think this is blocking... We don\u0027t always get installed in /usr/local/bin...","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"}],"nova/api/openstack/compute/contrib/fping.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"18d393b0b1941cc0725b39f5e521e1899afcecfc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"from nova.api.openstack.compute.plugins.v3.fping import *"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAP3%2F%2F5n4%3D","line":1,"updated":"2013-07-09 15:08:11.000000000","message":"No, this isn\u0027t right\n\nSee I5e7c3069913eea11633f35273183c3c3b46f4969 and I1db2a91f7e579356a77a196c46307492dd36e3d1","commit_id":"101c397cbd0f84d9c1da40348703d5d5ca79a632"}],"test-requirements.txt":[{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"bb3918547cf0dea5056b29f11c3cda200ae4061a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"MySQL-python"},{"line_number":13,"context_line":"psycopg2"},{"line_number":14,"context_line":"pylint\u003d\u003d0.25.2"},{"line_number":15,"context_line":"python-ldap\u003d\u003d2.3.13"},{"line_number":16,"context_line":"python-subunit"},{"line_number":17,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":18,"context_line":"sphinx\u003e\u003d1.1.2"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAPn%2F%2FVK8%3D","line":15,"updated":"2013-07-03 21:28:23.000000000","message":"Why are you adding this in this commit?","commit_id":"439302437980d0dec1b16665f863023fa82e6827"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"8388f6f95fea88f1933072363f6510dbc6ed9071","unresolved":false,"context_lines":[{"line_number":12,"context_line":"MySQL-python"},{"line_number":13,"context_line":"psycopg2"},{"line_number":14,"context_line":"pylint\u003d\u003d0.25.2"},{"line_number":15,"context_line":"python-ldap\u003d\u003d2.3.13"},{"line_number":16,"context_line":"python-subunit"},{"line_number":17,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":18,"context_line":"sphinx\u003e\u003d1.1.2"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAPn%2F%2FUFY%3D","line":15,"in_reply_to":"AAAAPn%2F%2FVK8%3D","updated":"2013-07-04 08:31:32.000000000","message":"It is needed to generate the configuration file.\nThe generator loads every Python file. Soi f not installed, loading the ldap fails wrt the ldapdns code.","commit_id":"439302437980d0dec1b16665f863023fa82e6827"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d1816f4e0b35f18cb5de562055cf8c8f500471c8","unresolved":false,"context_lines":[{"line_number":12,"context_line":"MySQL-python"},{"line_number":13,"context_line":"psycopg2"},{"line_number":14,"context_line":"pylint\u003d\u003d0.25.2"},{"line_number":15,"context_line":"python-ldap\u003d\u003d2.3.13"},{"line_number":16,"context_line":"python-subunit"},{"line_number":17,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":18,"context_line":"sphinx\u003e\u003d1.1.2"}],"source_content_type":"text/plain","patch_set":3,"id":"AAAAP3%2F%2FiZE%3D","line":15,"updated":"2013-07-16 20:52:39.000000000","message":"OK, I see why this is here; I would like a comment explaining that, though, as it\u0027s likely to be yanked by someone at some point if you don\u0027t.","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"3fb5421b2a6908176abbd62a6d994332b8f47d9e","unresolved":false,"context_lines":[{"line_number":12,"context_line":"MySQL-python"},{"line_number":13,"context_line":"psycopg2"},{"line_number":14,"context_line":"pylint\u003d\u003d0.25.2"},{"line_number":15,"context_line":"python-ldap\u003d\u003d2.3.13"},{"line_number":16,"context_line":"python-subunit"},{"line_number":17,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":18,"context_line":"sphinx\u003e\u003d1.1.2"}],"source_content_type":"text/plain","patch_set":3,"id":"AAAAP3%2F%2FgNU%3D","line":15,"in_reply_to":"AAAAP3%2F%2FiZE%3D","updated":"2013-07-17 09:18:07.000000000","message":"There\u0027s no chance it\u0027ll be removed, otherwise the config file generation will fail and the tox pep8 check will faill too anyway.","commit_id":"346e1b68e1ee1d8f4454398a6968599e8f253157"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"282be69264fef0dc280bb987f842f86d63ad7138","unresolved":false,"context_lines":[{"line_number":12,"context_line":"MySQL-python"},{"line_number":13,"context_line":"psycopg2"},{"line_number":14,"context_line":"pylint\u003d\u003d0.25.2"},{"line_number":15,"context_line":"python-ldap\u003d\u003d2.3.13"},{"line_number":16,"context_line":"python-subunit"},{"line_number":17,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":18,"context_line":"sphinx\u003e\u003d1.1.2"}],"source_content_type":"text/plain","patch_set":4,"id":"AAAAP3%2F%2FenQ%3D","line":15,"updated":"2013-07-17 16:49:23.000000000","message":"I still want the comment string, because the test failure will likely be obscure enough that the person who removed it may not understand where it really came from.","commit_id":"0b13193f08a4921f3dce49691b2c8df681e142bc"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"67c4032bdd976d9cebddc305faa2269e07dc4d17","unresolved":false,"context_lines":[{"line_number":12,"context_line":"MySQL-python"},{"line_number":13,"context_line":"psycopg2"},{"line_number":14,"context_line":"pylint\u003d\u003d0.25.2"},{"line_number":15,"context_line":"python-ldap\u003d\u003d2.3.13"},{"line_number":16,"context_line":"python-subunit"},{"line_number":17,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":18,"context_line":"sphinx\u003e\u003d1.1.2"}],"source_content_type":"text/plain","patch_set":4,"id":"AAAAQn%2F%2F94E%3D","line":15,"in_reply_to":"AAAAP3%2F%2FenQ%3D","updated":"2013-07-18 10:28:41.000000000","message":"Done","commit_id":"0b13193f08a4921f3dce49691b2c8df681e142bc"}]}