)]}'
{"patrole_tempest_plugin/rbac_policy_parser.py":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"a74bdd86304347a07cda2822cfc05002da2fa5bb","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        # Cache the list of available services in memory to avoid needlessly"},{"line_number":92,"context_line":"        # doing an API call every time."},{"line_number":93,"context_line":"        if hasattr(cls, \u0027available_services\u0027):"},{"line_number":94,"context_line":"            available_services \u003d cls.available_services"},{"line_number":95,"context_line":"        else:"},{"line_number":96,"context_line":"            admin_mgr \u003d credentials.AdminManager()"},{"line_number":97,"context_line":"            services \u003d admin_mgr.identity_services_v3_client.\\"}],"source_content_type":"text/x-python","patch_set":5,"id":"7f231b9d_eba08e44","line":94,"range":{"start_line":94,"start_character":12,"end_line":94,"end_character":30},"updated":"2017-06-16 14:09:01.000000000","message":"nit: This seems like an extraneous variable. It looks like all you\u0027re doing is assigning it to cls.available_services or being assigned to from it","commit_id":"0a07df95b85cc1c248c7bedb58794dd815ce6d45"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"756fdcef77368a8998ba1a408eb2647704db0264","unresolved":false,"context_lines":[{"line_number":86,"context_line":"    @classmethod"},{"line_number":87,"context_line":"    def validate_service(cls, service):"},{"line_number":88,"context_line":"        \"\"\"Validate whether the service passed to ``init`` exists.\"\"\""},{"line_number":89,"context_line":"        service \u003d service.lower().strip() if service else None"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"        # Cache the list of available services in memory to avoid needlessly"},{"line_number":92,"context_line":"        # doing an API call every time."}],"source_content_type":"text/x-python","patch_set":6,"id":"7f231b9d_9685b609","line":89,"range":{"start_line":89,"start_character":45,"end_line":89,"end_character":52},"updated":"2017-06-16 21:33:25.000000000","message":"Better to use isinstance(service, six.string_types)","commit_id":"d9607c407c02bd23c28a308ae0d0c9ae6934a97c"}]}