)]}'
{"etc/nova/nova.conf.sample":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"189d411ed7f92b884aa73e39c87b2df44558037d","unresolved":false,"context_lines":[{"line_number":62,"context_line":"# not necessarily a hostname, FQDN, or IP address. However,"},{"line_number":63,"context_line":"# the node name must be valid within an AMQP key, and if using"},{"line_number":64,"context_line":"# ZeroMQ, a valid hostname, FQDN, or IP address (string value)"},{"line_number":65,"context_line":"#host\u003ddex"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"# use ipv6 (boolean value)"},{"line_number":68,"context_line":"#use_ipv6\u003dfalse"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAPX%2F%2F%2Ba8%3D","line":65,"updated":"2013-06-04 19:47:16.000000000","message":"?","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b6fab59373aff0a9d37c0fadbab4599fd94a3188","unresolved":false,"context_lines":[{"line_number":62,"context_line":"# not necessarily a hostname, FQDN, or IP address. However,"},{"line_number":63,"context_line":"# the node name must be valid within an AMQP key, and if using"},{"line_number":64,"context_line":"# ZeroMQ, a valid hostname, FQDN, or IP address (string value)"},{"line_number":65,"context_line":"#host\u003ddex"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"# use ipv6 (boolean value)"},{"line_number":68,"context_line":"#use_ipv6\u003dfalse"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAPX%2F%2F2ns%3D","line":65,"in_reply_to":"AAAAPX%2F%2F%2Ba8%3D","updated":"2013-06-07 15:03:31.000000000","message":"yeah, the fqdn call should be mocked I guess. Fixed.","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"326f48885db24b1470b450b30633e7dd8e9e2406","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":4,"id":"AAAAPX%2F%2F2X4%3D","line":102,"updated":"2013-06-07 16:12:09.000000000","message":"*eyebrow raise*\n\nNot worth holding up this patch, but just wanted to draw attention to this change in case it wasn\u0027t intentional…","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"622ea736aedd039c8785979b78718dc6022226d3","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":4,"id":"AAAAPX%2F%2F18o%3D","line":102,"in_reply_to":"AAAAPX%2F%2F2FE%3D","updated":"2013-06-07 18:50:58.000000000","message":"just pointing it out :)","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"f892cb49e768095ed2e2ea17684fb7362525173b","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":4,"id":"AAAAPX%2F%2F2FE%3D","line":102,"in_reply_to":"AAAAPX%2F%2F2X4%3D","updated":"2013-06-07 18:10:32.000000000","message":"It\u0027s autogenerated :) but I can manually reset it if you prefer, or someone should fix generator.py","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"326f48885db24b1470b450b30633e7dd8e9e2406","unresolved":false,"context_lines":[{"line_number":1303,"context_line":""},{"line_number":1304,"context_line":"# The SQLAlchemy connection string used to connect to the"},{"line_number":1305,"context_line":"# database (string value)"},{"line_number":1306,"context_line":"#sql_connection\u003dsqlite:////common/db/$sqlite_db"},{"line_number":1307,"context_line":""},{"line_number":1308,"context_line":"# the filename to use with sqlite (string value)"},{"line_number":1309,"context_line":"#sqlite_db\u003dnova.sqlite"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAAPX%2F%2F2X0%3D","line":1306,"updated":"2013-06-07 16:12:09.000000000","message":"Another *eyebrow raise*","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"f892cb49e768095ed2e2ea17684fb7362525173b","unresolved":false,"context_lines":[{"line_number":1303,"context_line":""},{"line_number":1304,"context_line":"# The SQLAlchemy connection string used to connect to the"},{"line_number":1305,"context_line":"# database (string value)"},{"line_number":1306,"context_line":"#sql_connection\u003dsqlite:////common/db/$sqlite_db"},{"line_number":1307,"context_line":""},{"line_number":1308,"context_line":"# the filename to use with sqlite (string value)"},{"line_number":1309,"context_line":"#sqlite_db\u003dnova.sqlite"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAAPX%2F%2F2E0%3D","line":1306,"in_reply_to":"AAAAPX%2F%2F2X0%3D","updated":"2013-06-07 18:10:32.000000000","message":"This is fixed in https://review.openstack.org/#/c/32159/","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"change_message_id":"4e416e7ce25c7e8c83ba5034e3f2109927958cee","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":5,"id":"AAAAPX%2F%2F1i0%3D","line":102,"updated":"2013-06-07 22:13:03.000000000","message":"I know this was commented on as intentional in a previous comment, but this seems less correct than just using $pybasedir. I\u0027d be happier keeping the more abstract default","commit_id":"6e58d9fa406a885f32930be7217348545ab017f4"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b200db5cb1a1ff8c517efc5cb6accb91848f7340","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":5,"id":"AAAAPX%2F%2FzjA%3D","line":102,"in_reply_to":"AAAAPX%2F%2F1i0%3D","updated":"2013-06-10 08:19:12.000000000","message":"Done","commit_id":"6e58d9fa406a885f32930be7217348545ab017f4"}],"nova/cmd/novnc.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"189d411ed7f92b884aa73e39c87b2df44558037d","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    ]"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"CONF \u003d cfg.CONF"},{"line_number":45,"context_line":"CONF.register_cli_opts(opts)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F%2BZI%3D","line":45,"updated":"2013-06-04 19:47:16.000000000","message":"I don\u0027t really like the idea that importing this results in a side-effect of declaring these options.  I would much rather you leave out these last two lines and do an explicit register_cli_opts() in the files that use them…","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b6fab59373aff0a9d37c0fadbab4599fd94a3188","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    ]"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"CONF \u003d cfg.CONF"},{"line_number":45,"context_line":"CONF.register_cli_opts(opts)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F2nU%3D","line":45,"in_reply_to":"AAAAPX%2F%2F%2BZI%3D","updated":"2013-06-07 15:03:31.000000000","message":"Except that the config generator scripts works by importing the whole code base, and calling twice register_cli_opts will break it because it won\u0027t find the option.\nSince the whole code base declare configuration by importing side effect, I\u0027m not really eager to try to change this.\n\nHowever, to be more explicit I changed to import_opt in the 2 files using these options.","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d6feae78c20dded643e70426b3fd5ca4b389aec4","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    ]"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"CONF \u003d cfg.CONF"},{"line_number":45,"context_line":"CONF.register_cli_opts(opts)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F2YY%3D","line":45,"in_reply_to":"AAAAPX%2F%2F2nU%3D","updated":"2013-06-07 16:12:14.000000000","message":"OK, fair enough; I guess my problem then is with oslo.config :)","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"}],"nova/cmd/spicehtml5proxy.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"189d411ed7f92b884aa73e39c87b2df44558037d","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    ]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"CONF \u003d cfg.CONF"},{"line_number":42,"context_line":"CONF.register_cli_opts(opts)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def main():"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F%2BY0%3D","line":42,"updated":"2013-06-04 19:47:16.000000000","message":"I seem to remember that these two lines were actually moved into the main() function by a recent patch.  Do you know why that was?","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b6fab59373aff0a9d37c0fadbab4599fd94a3188","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    ]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"CONF \u003d cfg.CONF"},{"line_number":42,"context_line":"CONF.register_cli_opts(opts)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def main():"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F2nE%3D","line":42,"in_reply_to":"AAAAPX%2F%2F%2BY0%3D","updated":"2013-06-07 15:03:31.000000000","message":"Checked why:\n\n    \"Fix flake8 errors in anticipation of flake8.\"\n\nBut flake seems happy anyway.","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"}],"nova/openstack/common/config/generator.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"326f48885db24b1470b450b30633e7dd8e9e2406","unresolved":false,"context_lines":[{"line_number":250,"context_line":"    generate(sys.argv[1:])"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":253,"context_line":"    main()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2F2Xg%3D","line":253,"updated":"2013-06-07 16:12:09.000000000","message":"No problem with it, but I find it a little odd that this is in oslo-incubator instead of oslo.config…","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"622ea736aedd039c8785979b78718dc6022226d3","unresolved":false,"context_lines":[{"line_number":250,"context_line":"    generate(sys.argv[1:])"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":253,"context_line":"    main()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2F18A%3D","line":253,"in_reply_to":"AAAAPX%2F%2F2Eo%3D","updated":"2013-06-07 18:50:58.000000000","message":"*shrug* :)","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"f892cb49e768095ed2e2ea17684fb7362525173b","unresolved":false,"context_lines":[{"line_number":250,"context_line":"    generate(sys.argv[1:])"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":253,"context_line":"    main()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2F2Eo%3D","line":253,"in_reply_to":"AAAAPX%2F%2F2Xg%3D","updated":"2013-06-07 18:10:32.000000000","message":"Ask the Oslo team, not me. :)","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"}],"nova/openstack/common/rpc/matchmaker_ring.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"326f48885db24b1470b450b30633e7dd8e9e2406","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":"The MatchMaker classes should except a Topic or Fanout exchange key and"},{"line_number":18,"context_line":"return keys for direct exchanges, per (approximate) AMQP parlance."},{"line_number":19,"context_line":"\"\"\""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"import itertools"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2F2Xc%3D","line":18,"updated":"2013-06-07 16:12:09.000000000","message":"I don\u0027t think this is related to your patch…","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"f892cb49e768095ed2e2ea17684fb7362525173b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":"The MatchMaker classes should except a Topic or Fanout exchange key and"},{"line_number":18,"context_line":"return keys for direct exchanges, per (approximate) AMQP parlance."},{"line_number":19,"context_line":"\"\"\""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"import itertools"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2F2Eg%3D","line":18,"in_reply_to":"AAAAPX%2F%2F2Xc%3D","updated":"2013-06-07 18:10:32.000000000","message":"You\u0027re right, pulling oslo pulled this too (and a lot of other things) but I missed this one, removing in the next patchset!","commit_id":"e883012bfbd05a699038a9057df380d9093956b9"}],"nova/utils.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"189d411ed7f92b884aa73e39c87b2df44558037d","unresolved":false,"context_lines":[{"line_number":69,"context_line":"               help\u003d\u0027Length of generated instance admin passwords\u0027),"},{"line_number":70,"context_line":"    cfg.BoolOpt(\u0027disable_process_locking\u0027,"},{"line_number":71,"context_line":"                default\u003dFalse,"},{"line_number":72,"context_line":"                help\u003d\u0027Whether to disable inter-process locks\u0027),"},{"line_number":73,"context_line":"    cfg.StrOpt(\u0027instance_usage_audit_period\u0027,"},{"line_number":74,"context_line":"               default\u003d\u0027month\u0027,"},{"line_number":75,"context_line":"               help\u003d\u0027time period to generate instance usages for.  \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F%2BaA%3D","side":"PARENT","line":72,"updated":"2013-06-04 19:47:16.000000000","message":"Why have you removed this option?","commit_id":"73077b4c5dd137ed5c0571c963319c0ec92b4e80"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b6fab59373aff0a9d37c0fadbab4599fd94a3188","unresolved":false,"context_lines":[{"line_number":69,"context_line":"               help\u003d\u0027Length of generated instance admin passwords\u0027),"},{"line_number":70,"context_line":"    cfg.BoolOpt(\u0027disable_process_locking\u0027,"},{"line_number":71,"context_line":"                default\u003dFalse,"},{"line_number":72,"context_line":"                help\u003d\u0027Whether to disable inter-process locks\u0027),"},{"line_number":73,"context_line":"    cfg.StrOpt(\u0027instance_usage_audit_period\u0027,"},{"line_number":74,"context_line":"               default\u003d\u0027month\u0027,"},{"line_number":75,"context_line":"               help\u003d\u0027time period to generate instance usages for.  \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F2m8%3D","side":"PARENT","line":72,"in_reply_to":"AAAAPX%2F%2F%2BaA%3D","updated":"2013-06-07 15:03:31.000000000","message":"Because the option is already declared in oslo","commit_id":"73077b4c5dd137ed5c0571c963319c0ec92b4e80"}],"tools/conf/extract_opts.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"189d411ed7f92b884aa73e39c87b2df44558037d","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    \"Check if opt is in group.\""},{"line_number":121,"context_line":"    for key, value in group._opts.items():"},{"line_number":122,"context_line":"        if value[\u0027opt\u0027] \u003d\u003d opt:"},{"line_number":123,"context_line":"            return True"},{"line_number":124,"context_line":"    return False"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F%2BZs%3D","line":123,"updated":"2013-06-04 19:47:16.000000000","message":"You\u0027re not using key, so why generate it?  Just use values() instead of items().  For that matter, you could do something like:\n\n    return opt in [v[\u0027opt\u0027] for v in group._opts.values()]\n\nThough that might be a bit more obscure.","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b6fab59373aff0a9d37c0fadbab4599fd94a3188","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    \"Check if opt is in group.\""},{"line_number":121,"context_line":"    for key, value in group._opts.items():"},{"line_number":122,"context_line":"        if value[\u0027opt\u0027] \u003d\u003d opt:"},{"line_number":123,"context_line":"            return True"},{"line_number":124,"context_line":"    return False"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F2ms%3D","line":123,"in_reply_to":"AAAAPX%2F%2F%2BZs%3D","updated":"2013-06-07 15:03:31.000000000","message":"The script is now in oslo-incubator, i\u0027ll fix it there.","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d6feae78c20dded643e70426b3fd5ca4b389aec4","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    \"Check if opt is in group.\""},{"line_number":121,"context_line":"    for key, value in group._opts.items():"},{"line_number":122,"context_line":"        if value[\u0027opt\u0027] \u003d\u003d opt:"},{"line_number":123,"context_line":"            return True"},{"line_number":124,"context_line":"    return False"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2F2YI%3D","line":123,"in_reply_to":"AAAAPX%2F%2F2ms%3D","updated":"2013-06-07 16:12:14.000000000","message":"OK, fair enough…","commit_id":"bcebeb5edb36b9e4cc13608339611c3e3714c059"}]}