)]}'
{"patrole_tempest_plugin/tests/api/volume/test_capabilities_rbac.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"91dc7dd34fd425fe3f043cb658c022091411f7e8","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    @decorators.idempotent_id(\u002740928b74-2141-11e7-93ae-92361f002671\u0027)"},{"line_number":41,"context_line":"    def test_show_back_end_capabilities(self):"},{"line_number":42,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"},{"line_number":43,"context_line":"        try:"},{"line_number":44,"context_line":"            self.client.show_backend_capabilities(\"dummyhost@hostname\")"},{"line_number":45,"context_line":"        except exc.NotFound:"},{"line_number":46,"context_line":"            # NotFound will be thrown for admin roles as the above"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_49d98481","line":43,"range":{"start_line":43,"start_character":8,"end_line":43,"end_character":12},"updated":"2017-04-14 20:56:30.000000000","message":"If you want to use a try/except block, then add expected_error_code\u003d404 to rbac_rule_validation.\n\nLooking at Tempest for guidance I see this:\n\ncls.hosts \u003d [\n            pool[\u0027name\u0027] for pool in\n            cls.admin_scheduler_stats_client.list_pools()[\u0027pools\u0027]\n        ]\n\nCan that not be done here to get the right host_name?","commit_id":"91af4416c7d2bc0758e226673b49ac7440a80bde"}],"patrole_tempest_plugin/tests/api/volume/test_scheduler_stats_rbac.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"91dc7dd34fd425fe3f043cb658c022091411f7e8","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        cls.client \u003d cls.os.volume_scheduler_stats_v2_client"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    @rbac_rule_validation.action(service\u003d\"cinder\","},{"line_number":38,"context_line":"                                 rule\u003d\"scheduler_extension:\""},{"line_number":39,"context_line":"                                      \"scheduler_stats:get_pools\")"},{"line_number":40,"context_line":"    @decorators.idempotent_id(\u00275f800441-4d30-48ec-9e5b-0d55bc86acbb\u0027)"},{"line_number":41,"context_line":"    def test_list_back_end_storage_pools(self):"},{"line_number":42,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_a9cb68a3","line":39,"range":{"start_line":38,"start_character":33,"end_line":39,"end_character":65},"updated":"2017-04-14 20:56:30.000000000","message":"nit: can we format this on same line?","commit_id":"91af4416c7d2bc0758e226673b49ac7440a80bde"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"9960ce76b2ad95e583cc8487e5d7dc8c688b87b1","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        cls.client \u003d cls.os.volume_scheduler_stats_v2_client"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    @rbac_rule_validation.action(service\u003d\"cinder\","},{"line_number":38,"context_line":"                                 rule\u003d\"scheduler_extension:\""},{"line_number":39,"context_line":"                                      \"scheduler_stats:get_pools\")"},{"line_number":40,"context_line":"    @decorators.idempotent_id(\u00275f800441-4d30-48ec-9e5b-0d55bc86acbb\u0027)"},{"line_number":41,"context_line":"    def test_list_back_end_storage_pools(self):"},{"line_number":42,"context_line":"        self.rbac_utils.switch_role(self, toggle_rbac_role\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_69cfa07f","line":39,"range":{"start_line":38,"start_character":33,"end_line":39,"end_character":65},"in_reply_to":"7ffa3b31_a9cb68a3","updated":"2017-04-14 21:09:07.000000000","message":"Unfortunately the rule name is too long to be on one line with \"rule\u003d\" and if I include a slash (rule\u003d\\) to get the rule name on its own line, pep8 complains.","commit_id":"91af4416c7d2bc0758e226673b49ac7440a80bde"}]}