)]}'
{"keystoneclient/tests/unit/v3/test_roles.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"54afafc877c8a1830f5d29b23795aa65f45c89f2","unresolved":false,"context_lines":[{"line_number":608,"context_line":"        self.model \u003d roles.Role"},{"line_number":609,"context_line":"        self.manager \u003d self.client.roles"},{"line_number":610,"context_line":""},{"line_number":611,"context_line":"    def test_implied_create(self):"},{"line_number":612,"context_line":"        prior_id \u003d uuid.uuid4().hex"},{"line_number":613,"context_line":"        prior_name \u003d uuid.uuid4().hex"},{"line_number":614,"context_line":"        implied_id \u003d uuid.uuid4().hex"}],"source_content_type":"text/x-python","patch_set":8,"id":"1a6eadb0_21808546","line":611,"range":{"start_line":611,"start_character":8,"end_line":611,"end_character":27},"updated":"2016-12-19 03:12:16.000000000","message":"I only tested the deprecation behaviour on create, since it\u0027s a wrapped and should be the same message for all","commit_id":"604c78ff1a9c65eda4722560f1ebcf32b9f2c6e8"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"19a6ce332938af6fa70b7137b0447e55a778fed6","unresolved":false,"context_lines":[{"line_number":784,"context_line":"                    \"id\": implied_id,"},{"line_number":785,"context_line":"                    \"links\": {\"self\": \"http://host/v3/roles/%s\" % implied_id},"},{"line_number":786,"context_line":"                    \"name\": implied_name"},{"line_number":787,"context_line":"                }],"},{"line_number":788,"context_line":"                \"prior_role\": {"},{"line_number":789,"context_line":"                    \"id\": prior_id,"},{"line_number":790,"context_line":"                    \"links\": {\"self\": \"http://host/v3/roles/%s\" % prior_id},"}],"source_content_type":"text/x-python","patch_set":9,"id":"1a6eadb0_9e1e8a6e","line":787,"updated":"2016-12-21 13:30:44.000000000","message":"it\u0027s odd that we don\u0027t return a set of role_inference objects, but modify the object instead to return implies as a list.","commit_id":"0b00c0e7b17ad1a156fc2dbaccde97e3d47ba115"}],"keystoneclient/v3/roles.py":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"c64b5e364615f2a8353a47681ab9dbd1838b7569","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                     \u0027implied_role_id\u0027: base.getid(implied_role)})"},{"line_number":125,"context_line":"        return base_url"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"    def create_implied(self, prior_role, implied_role, **kwargs):"},{"line_number":128,"context_line":"        \"\"\"Create an inference rule."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"        :param prior_role: the role which implies ``implied_role``."}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_6b85fcab","side":"PARENT","line":127,"range":{"start_line":127,"start_character":63,"end_line":127,"end_character":65},"updated":"2016-12-19 02:15:32.000000000","message":"so unfortunatley i think you should probably figure out which of these work and deprecate them. You can do this fairly easily by creating and proxying to your own InferenceRulesManager.","commit_id":"08dba1fa8b24c9916e57f5dd9b1d06a2af7771e2"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"5fe334603ea559195157a123bd11cd9452a1ea0e","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                     \u0027implied_role_id\u0027: base.getid(implied_role)})"},{"line_number":125,"context_line":"        return base_url"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"    def create_implied(self, prior_role, implied_role, **kwargs):"},{"line_number":128,"context_line":"        \"\"\"Create an inference rule."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"        :param prior_role: the role which implies ``implied_role``."}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_21a965e0","side":"PARENT","line":127,"range":{"start_line":127,"start_character":63,"end_line":127,"end_character":65},"in_reply_to":"1a6eadb0_6b85fcab","updated":"2016-12-19 03:10:31.000000000","message":"thanks for the idea jamie","commit_id":"08dba1fa8b24c9916e57f5dd9b1d06a2af7771e2"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"c64b5e364615f2a8353a47681ab9dbd1838b7569","unresolved":false,"context_lines":[{"line_number":363,"context_line":"    collection_key \u003d \u0027role_inferences\u0027"},{"line_number":364,"context_line":"    key \u003d \u0027role_inference\u0027"},{"line_number":365,"context_line":""},{"line_number":366,"context_line":"    def _implied_role_url_tail(self, prior_role, implied_role):"},{"line_number":367,"context_line":"        base_url \u003d (\u0027/%(prior_role_id)s/implies/%(implied_role_id)s\u0027 %"},{"line_number":368,"context_line":"                    {\u0027prior_role_id\u0027: base.getid(prior_role),"},{"line_number":369,"context_line":"                     \u0027implied_role_id\u0027: base.getid(implied_role)})"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_abf8f44a","line":366,"range":{"start_line":366,"start_character":8,"end_line":366,"end_character":30},"updated":"2016-12-19 02:15:32.000000000","message":"so you could override build_url with this, I don\u0027t know if that\u0027s any nicer because the API doesn\u0027t actually line up with the client. ie the collection_key is not the url and other things the manager expects.\n\nedit: maybe not because then list_role_inferences fails","commit_id":"ddb9f0a03f87ee5ee54235d1af54daeeb0f2ddd5"}]}