)]}'
{"bin/swift-get-nodes":[{"author":{"_account_id":2696,"name":"Darrell Bishop","email":"dbishop@nvidia.com","username":"darrellb"},"change_message_id":"a6c607e7dfec6e7a71ea0fe174d2600233ff5c5d","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    usage \u003d \u0027\u0027\u0027"},{"line_number":28,"context_line":"    Shows the nodes responsible for the item specified."},{"line_number":29,"context_line":"    Usage: %prog [-a] \u003cring.gz\u003e \u003caccount\u003e [\u003ccontainer\u003e[\u003cobject\u003e]]"},{"line_number":30,"context_line":"    Or: %prog [-a] \u003cring.gz\u003e -p partition"},{"line_number":31,"context_line":"    Or: %prog [-a] -P policy_name \u003caccount\u003e \u003ccontainer\u003e \u003cobject\u003e"},{"line_number":32,"context_line":"    Note: account, container, object can also be a single arg separated by /"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5afe65bd_91e92883","line":29,"updated":"2015-06-03 14:28:06.000000000","message":"Can you put a space between the \"\u003e\" of \"\u003ccontainer\u003e\" and the \"[\"?  Without the space and because of whatever font I\u0027m using, I guess, I read that as a vertical pipe for \"or\", which isn\u0027t right.\n\nThe manpage edit is correct in this respect.","commit_id":"6d6dbda33dbebddfe528c467259c6be83996e064"}],"swift/cli/info.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2f0721b471c810a559f055422f4005f52cfe651f","unresolved":false,"context_lines":[{"line_number":382,"context_line":"            if (policy_index is not None and"},{"line_number":383,"context_line":"               policy_index_for_name is not None and"},{"line_number":384,"context_line":"               policy_index !\u003d policy_index_for_name):"},{"line_number":385,"context_line":"                print \u0027Attention: Ring does not match policy!\u0027"},{"line_number":386,"context_line":"                print \u0027Double check your policy name!\u0027"},{"line_number":387,"context_line":"            if not ring and policy_index_for_name:"},{"line_number":388,"context_line":"                ring \u003d POLICIES.get_object_ring(policy_index_for_name,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5afe65bd_d42d4e47","line":385,"updated":"2015-06-03 14:31:31.000000000","message":"this should be \u0027Warning\u0027 too","commit_id":"6d6dbda33dbebddfe528c467259c6be83996e064"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2f0721b471c810a559f055422f4005f52cfe651f","unresolved":false,"context_lines":[{"line_number":474,"context_line":"            if ring_name !\u003d policy.ring_name:"},{"line_number":475,"context_line":"                print \u0027Warning: mismatch between ring and policy names!\u0027"},{"line_number":476,"context_line":"        else:"},{"line_number":477,"context_line":"            print \u0027Error: Policy %s is not valid\u0027 % policy_name"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":"    policy_index \u003d None"},{"line_number":480,"context_line":"    if ring is None and (obj or part):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5afe65bd_141c0611","line":477,"updated":"2015-06-03 14:31:31.000000000","message":"IMHO \u0027Error\u0027 implies immediate exit. I think this is a warning.\n\nLines 471-477 are just for information - if a ring file is specified then the policy name never influences the outcome, so this is just reporting than an ignored argument is also inconistent.\n\nmaybe add \"(ignoring)\" at end of line","commit_id":"6d6dbda33dbebddfe528c467259c6be83996e064"},{"author":{"_account_id":2696,"name":"Darrell Bishop","email":"dbishop@nvidia.com","username":"darrellb"},"change_message_id":"3547aa30d0b8ebbcd190152d8665ac074050080e","unresolved":false,"context_lines":[{"line_number":472,"context_line":"        policy \u003d POLICIES.get_by_name(policy_name)"},{"line_number":473,"context_line":"        if policy:"},{"line_number":474,"context_line":"            if ring_name !\u003d policy.ring_name:"},{"line_number":475,"context_line":"                print \u0027Warning: mismatch between ring and policy names!\u0027"},{"line_number":476,"context_line":"        else:"},{"line_number":477,"context_line":"            print \u0027Warning: Policy %s is not valid\u0027 % policy_name"},{"line_number":478,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5afe65bd_3a792930","line":475,"updated":"2015-06-03 15:10:55.000000000","message":"\"names\" should be \"name\"","commit_id":"ed0559f0ef9c3b7edfb6f4b1444fbb2ecdcaa0f4"}]}