)]}'
{"swift/common/middleware/keystoneauth.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"e84bbe5c4c57de20a472c0021063c0437f31a24f","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        operator_roles"},{"line_number":107,"context_line":"        service_roles"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    For backward compatibility, no prefix implies the operator_roles parameter"},{"line_number":110,"context_line":"    applies to all reseller_prefixes. Here is an example, using two prefixes::"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        reseller_prefix \u003d AUTH, SERVICE"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_0d9fb953","line":109,"updated":"2015-03-23 14:25:43.000000000","message":"The problem I have with this change is that it implies that *only* the operator_roles parameter behave this way. But in without a prefix service _roles will also apply to all prefixes, and there would no longer be anything stating that.\n\nHow about just dropping \u0027For backwards compatibility\u0027:\n\n  If either of these parameters is specified without a prefix then it applies to all reseller_prefixes.","commit_id":"2f7027cc28b4ba733a595c2ecf4f638b3ecaa953"},{"author":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"change_message_id":"ac2184ae99d758e8470a4f2f54e07e8e79b757e9","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        operator_roles"},{"line_number":107,"context_line":"        service_roles"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    For backward compatibility, no prefix implies the operator_roles parameter"},{"line_number":110,"context_line":"    applies to all reseller_prefixes. Here is an example, using two prefixes::"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        reseller_prefix \u003d AUTH, SERVICE"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6ced46_56485e2d","line":109,"in_reply_to":"1a6ced46_0d9fb953","updated":"2015-03-23 15:13:30.000000000","message":"I agree with Alistair here. Just drop \"For backwards compatibility\" -- it adds nothing to the description","commit_id":"2f7027cc28b4ba733a595c2ecf4f638b3ecaa953"}]}