)]}'
{"octavia_tempest_plugin/config.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"b1760411dd30c57306840730cc8199cadd240026","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    cfg.DictOpt(\u0027expected_flavor_capability\u0027,"},{"line_number":115,"context_line":"                help\u003d(\u0027Defines a provider flavor capability that is expected \u0027"},{"line_number":116,"context_line":"                      \u0027to be present in the selected provider under test. \u0027"},{"line_number":117,"context_line":"                      \u0027It is specified in a \"name\": \"description\" dict. \u0027"},{"line_number":118,"context_line":"                      \u0027Example: {\"loadbalancer_topology\": \"The load balancer \u0027"},{"line_number":119,"context_line":"                      \u0027topology. One of: SINGLE - One amphora per load \u0027"},{"line_number":120,"context_line":"                      \u0027balancer. ACTIVE_STANDBY - Two amphora per load \u0027"}],"source_content_type":"text/x-python","patch_set":10,"id":"7faddb67_f27d5c6e","line":117,"updated":"2019-07-26 21:39:38.000000000","message":"We really have to expect a *description*? This seems a little awkward...","commit_id":"19d980e01cfb5cf94cfa2848199b2aba13dace09"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"669b8de671fa0ee7fb2d8fbedb4962683361a8bc","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    cfg.DictOpt(\u0027expected_flavor_capability\u0027,"},{"line_number":115,"context_line":"                help\u003d(\u0027Defines a provider flavor capability that is expected \u0027"},{"line_number":116,"context_line":"                      \u0027to be present in the selected provider under test. \u0027"},{"line_number":117,"context_line":"                      \u0027It is specified in a \"name\": \"description\" dict. \u0027"},{"line_number":118,"context_line":"                      \u0027Example: {\"loadbalancer_topology\": \"The load balancer \u0027"},{"line_number":119,"context_line":"                      \u0027topology. One of: SINGLE - One amphora per load \u0027"},{"line_number":120,"context_line":"                      \u0027balancer. ACTIVE_STANDBY - Two amphora per load \u0027"}],"source_content_type":"text/x-python","patch_set":10,"id":"7faddb67_990aef76","line":117,"in_reply_to":"7faddb67_f27d5c6e","updated":"2019-07-30 10:38:14.000000000","message":"Whatever, this is janky but fixing it seems complex apparently, so I guess we just do this for now. We can revisit if it is actually a problem.","commit_id":"19d980e01cfb5cf94cfa2848199b2aba13dace09"}],"octavia_tempest_plugin/tests/api/v2/test_flavor_capabilities.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"42513e26903603b15a07899946a51b85a320692d","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        # Test fields"},{"line_number":69,"context_line":"        capabilities \u003d admin_capabilities_client.list_flavor_capabilities("},{"line_number":70,"context_line":"            CONF.load_balancer.provider,"},{"line_number":71,"context_line":"            query_params\u003d\u0027{fields}\u003d{field}\u0027.format(fields\u003dconst.FIELDS,"},{"line_number":72,"context_line":"                                                   field\u003dconst.NAME))"},{"line_number":73,"context_line":"        self.assertEqual(1, len(capabilities[0]))"},{"line_number":74,"context_line":"        self.assertEqual(expected_name, capabilities[0][const.NAME])"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fce034c_d5757965","line":71,"updated":"2019-04-18 19:05:57.000000000","message":"These should have a query param with the expected name to have it return only the record we are looking for.","commit_id":"28d9186fe26e70ccbc710f84cfe8d0eee29bf29d"},{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"4ebb782d66f20ea0dd37f1b29fba91c5eb957977","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        # Test that a user without the load balancer admin role cannot"},{"line_number":45,"context_line":"        # list provider flavor capabilities."},{"line_number":46,"context_line":"        if CONF.load_balancer.RBAC_test_type \u003d\u003d const.ADVANCED:"},{"line_number":47,"context_line":"            os_primary_capabilities_client \u003d ("},{"line_number":48,"context_line":"                self.os_primary.flavor_capabilities_client)"},{"line_number":49,"context_line":"            self.assertRaises("}],"source_content_type":"text/x-python","patch_set":7,"id":"ffb9cba7_a82989e6","line":46,"range":{"start_line":46,"start_character":48,"end_line":46,"end_character":62},"updated":"2019-04-28 19:39:36.000000000","message":"can\u0027t we also test the same logic for the \u0027admin\u0027 role in a case of legacy RBAC?\n\nhttps://docs.openstack.org/octavia/stein/configuration/policy.html","commit_id":"ada89c94c2d5cdd041c977d2287b8dff73da69b9"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"af1619484911fe7909754f60196a3e804929dafc","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        # Test that a user without the load balancer admin role cannot"},{"line_number":45,"context_line":"        # list provider flavor capabilities."},{"line_number":46,"context_line":"        if CONF.load_balancer.RBAC_test_type \u003d\u003d const.ADVANCED:"},{"line_number":47,"context_line":"            os_primary_capabilities_client \u003d ("},{"line_number":48,"context_line":"                self.os_primary.flavor_capabilities_client)"},{"line_number":49,"context_line":"            self.assertRaises("}],"source_content_type":"text/x-python","patch_set":7,"id":"dfbec78f_dd5b10f3","line":46,"range":{"start_line":46,"start_character":48,"end_line":46,"end_character":62},"in_reply_to":"ffb9cba7_a82989e6","updated":"2019-05-16 15:03:46.000000000","message":"I don\u0027t understand your question. If this test is false, it runs with the legacy admin role.","commit_id":"ada89c94c2d5cdd041c977d2287b8dff73da69b9"}]}