)]}'
{"bin/swift-dispersion-populate":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"43700f5daa1962dba7a597dd594a1d6e07301a90","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        policy \u003d POLICIES.get_by_name(options.policy_name)"},{"line_number":126,"context_line":"        if policy is None:"},{"line_number":127,"context_line":"            exit(\u0027Unable to find policy: %s\u0027 % options.policy_name)"},{"line_number":128,"context_line":"    print \u0027Using storage policy: %s \u0027 % policy.name"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    swift_dir \u003d conf.get(\u0027swift_dir\u0027, \u0027/etc/swift\u0027)"},{"line_number":131,"context_line":"    dispersion_coverage \u003d float(conf.get(\u0027dispersion_coverage\u0027, 1))"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7a016987_26028b0c","line":128,"updated":"2015-05-26 06:20:24.000000000","message":"I wonder if we should use print () for python 3 support though I know current swift allows to leave a ton of print statements in some binaries. I\u0027m not sure.","commit_id":"faff9e74e6ab7c9392b2ba6646c5e3ad676b5f3f"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"995ac07f1c9ef736eaf6ff40ee3ae8ee24e089eb","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        policy \u003d POLICIES.get_by_name(options.policy_name)"},{"line_number":126,"context_line":"        if policy is None:"},{"line_number":127,"context_line":"            exit(\u0027Unable to find policy: %s\u0027 % options.policy_name)"},{"line_number":128,"context_line":"    print \u0027Using storage policy: %s \u0027 % policy.name"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    swift_dir \u003d conf.get(\u0027swift_dir\u0027, \u0027/etc/swift\u0027)"},{"line_number":131,"context_line":"    dispersion_coverage \u003d float(conf.get(\u0027dispersion_coverage\u0027, 1))"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7a016987_8f92164a","line":128,"in_reply_to":"7a016987_07ba900a","updated":"2015-05-28 06:40:35.000000000","message":"Ok, maybe we would have the py3 compatible discussion on irc meeting according to agent[1], that\u0027s fine :)\n\n1: https://wiki.openstack.org/wiki/Meetings/Swift","commit_id":"faff9e74e6ab7c9392b2ba6646c5e3ad676b5f3f"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"d4a140026c32c46c16ee69f8ebd529724534eea4","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        policy \u003d POLICIES.get_by_name(options.policy_name)"},{"line_number":126,"context_line":"        if policy is None:"},{"line_number":127,"context_line":"            exit(\u0027Unable to find policy: %s\u0027 % options.policy_name)"},{"line_number":128,"context_line":"    print \u0027Using storage policy: %s \u0027 % policy.name"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    swift_dir \u003d conf.get(\u0027swift_dir\u0027, \u0027/etc/swift\u0027)"},{"line_number":131,"context_line":"    dispersion_coverage \u003d float(conf.get(\u0027dispersion_coverage\u0027, 1))"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7a016987_07ba900a","line":128,"in_reply_to":"7a016987_26028b0c","updated":"2015-05-28 05:29:14.000000000","message":"There are other print statements in this file that will need brackets too, so maybe it\u0027s ok as it is for now, but a future patch should be raised to make it py3 compatible.","commit_id":"faff9e74e6ab7c9392b2ba6646c5e3ad676b5f3f"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"43557be1d225016182929a1ce7ab91e563083a50","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    conf \u003d dict(c.items(\u0027dispersion\u0027))"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    if options.policy_name is None:"},{"line_number":123,"context_line":"        policy \u003d POLICIES.legacy"},{"line_number":124,"context_line":"    else:"},{"line_number":125,"context_line":"        policy \u003d POLICIES.get_by_name(options.policy_name)"},{"line_number":126,"context_line":"        if policy is None:"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5afe65bd_e52dfbfe","line":123,"updated":"2015-06-04 18:22:41.000000000","message":"i would find the default policy to be a more intuitive choice","commit_id":"248e2aa237129dff3d8d7f78a221fd4a2b512646"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"43557be1d225016182929a1ce7ab91e563083a50","unresolved":false,"context_lines":[{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    if object_populate:"},{"line_number":214,"context_line":"        container \u003d \u0027dispersion_objects_%d\u0027 % policy.idx"},{"line_number":215,"context_line":"        put_container(connpool, container, None, headers\u003dheaders)"},{"line_number":216,"context_line":"        object_ring \u003d Ring(swift_dir, ring_name\u003dpolicy.ring_name)"},{"line_number":217,"context_line":"        parts_left \u003d dict((x, x) for x in xrange(object_ring.partition_count))"},{"line_number":218,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5afe65bd_0fa9567b","line":215,"updated":"2015-06-04 18:22:41.000000000","message":"nit: headers is a positional arg so:\n\n put_container(connpool, container, None, headers)","commit_id":"248e2aa237129dff3d8d7f78a221fd4a2b512646"}],"bin/swift-dispersion-report":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"43557be1d225016182929a1ce7ab91e563083a50","unresolved":false,"context_lines":[{"line_number":344,"context_line":"    conf \u003d dict(c.items(\u0027dispersion\u0027))"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"    if options.policy_name is None:"},{"line_number":347,"context_line":"        policy \u003d POLICIES.legacy"},{"line_number":348,"context_line":"    else:"},{"line_number":349,"context_line":"        policy \u003d POLICIES.get_by_name(options.policy_name)"},{"line_number":350,"context_line":"        if policy is None:"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5afe65bd_05d7f7ed","line":347,"updated":"2015-06-04 18:22:41.000000000","message":"this caught me out - i was expecting it to be the default, i think that would be more intuitive","commit_id":"248e2aa237129dff3d8d7f78a221fd4a2b512646"}]}