)]}'
{"tempest/lib/services/clients.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"41add1fda3e26d8826ba0aa85e78eea88dd35ffe","unresolved":false,"context_lines":[{"line_number":246,"context_line":""},{"line_number":247,"context_line":"        \u003e\u003e\u003e from tempest.lib.services import clients"},{"line_number":248,"context_line":"        \u003e\u003e\u003e johndoe \u003d cred_provider.get_creds_by_role([\u0027johndoe\u0027])"},{"line_number":249,"context_line":"        \u003e\u003e\u003e johndoe_clients \u003d clients.ServiceClients(johndoe, identity_uri)"},{"line_number":250,"context_line":"        \u003e\u003e\u003e"},{"line_number":251,"context_line":"        \u003e\u003e\u003e # List servers in default region"},{"line_number":252,"context_line":"        \u003e\u003e\u003e johndoe_servers_client \u003d johndoe_clients.compute.ServersClient()"},{"line_number":253,"context_line":"        \u003e\u003e\u003e johndoe_servers \u003d johndoe_servers_client.list_servers()"},{"line_number":254,"context_line":"        \u003e\u003e\u003e"},{"line_number":255,"context_line":"        \u003e\u003e\u003e # List servers in Region B"},{"line_number":256,"context_line":"        \u003e\u003e\u003e johndoe_servers_client_B \u003d johndoe_clients.compute.ServersClient("},{"line_number":257,"context_line":"        \u003e\u003e\u003e     region\u003d\u0027B\u0027)"},{"line_number":258,"context_line":"        \u003e\u003e\u003e johndoe_servers \u003d johndoe_servers_client_B.list_servers()"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"    \"\"\""},{"line_number":261,"context_line":"    # NOTE(andreaf) This class does not depend on tempest configuration"}],"source_content_type":"text/x-python","patch_set":2,"id":"bff0334d_10633b44","line":258,"range":{"start_line":249,"start_character":0,"end_line":258,"end_character":69},"updated":"2017-04-07 23:56:55.000000000","message":"This change seems unrelated to the purpose of this patch..","commit_id":"a05943be36b4bf4458d761b80ea96519f10832c4"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"41add1fda3e26d8826ba0aa85e78eea88dd35ffe","unresolved":false,"context_lines":[{"line_number":264,"context_line":"    # initialises this class using values from tempest CONF object. The wrapper"},{"line_number":265,"context_line":"    # class should only be used by tests hosted in Tempest."},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def __init__(self, credentials, identity_uri, region\u003dNone, scope\u003d\u0027project\u0027,"},{"line_number":268,"context_line":"                 disable_ssl_certificate_validation\u003dTrue, ca_certs\u003dNone,"},{"line_number":269,"context_line":"                 trace_requests\u003d\u0027\u0027):"},{"line_number":270,"context_line":"        \"\"\"Service Clients provider"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"        Instantiate a `ServiceClients` object, from a set of credentials and an"}],"source_content_type":"text/x-python","patch_set":2,"id":"bff0334d_d06dd304","line":269,"range":{"start_line":267,"start_character":0,"end_line":269,"end_character":36},"updated":"2017-04-07 23:56:55.000000000","message":"Yeah, I agree with Matthew. It would be nice to take some deprecation period before removing.\n\ndebtcollector\u0027s removed_kwarg() is useful for this case.\n\neg: https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/transport.py#L149","commit_id":"a05943be36b4bf4458d761b80ea96519f10832c4"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"768b57ebdd9b104af67cd9703be11350630e036f","unresolved":false,"context_lines":[{"line_number":276,"context_line":"        provider as well as any service clients provided by this manager."},{"line_number":277,"context_line":""},{"line_number":278,"context_line":"        Any other client parameter should be set via ClientsRegistry."},{"line_number":279,"context_line":"        "},{"line_number":280,"context_line":"        Client parameter used to be set via client_parameters, but this is"},{"line_number":281,"context_line":"        deprecated, and it is actually already not honoured"},{"line_number":282,"context_line":"        anymore: https://launchpad.net/bugs/1680915."}],"source_content_type":"text/x-python","patch_set":4,"id":"bff0334d_9eb3a70a","line":279,"updated":"2017-04-11 02:43:08.000000000","message":"heh, you need to remove these blanks.\nhttp://logs.openstack.org/86/454886/4/check/gate-tempest-pep8-ubuntu-xenial/9a5a824/console.html#_2017-04-09_17_10_29_301787","commit_id":"b1b025648383b3510c855e4e2df9f8d2d73a7dd6"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"768b57ebdd9b104af67cd9703be11350630e036f","unresolved":false,"context_lines":[{"line_number":280,"context_line":"        Client parameter used to be set via client_parameters, but this is"},{"line_number":281,"context_line":"        deprecated, and it is actually already not honoured"},{"line_number":282,"context_line":"        anymore: https://launchpad.net/bugs/1680915."},{"line_number":283,"context_line":"        "},{"line_number":284,"context_line":"        The list of available parameters is defined in the service clients"},{"line_number":285,"context_line":"        interfaces. For reference, most clients will accept \u0027region\u0027,"},{"line_number":286,"context_line":"        \u0027service\u0027, \u0027endpoint_type\u0027, \u0027build_timeout\u0027 and \u0027build_interval\u0027, which"}],"source_content_type":"text/x-python","patch_set":4,"id":"bff0334d_7eb8e3e3","line":283,"updated":"2017-04-11 02:43:08.000000000","message":"ditto","commit_id":"b1b025648383b3510c855e4e2df9f8d2d73a7dd6"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"768b57ebdd9b104af67cd9703be11350630e036f","unresolved":false,"context_lines":[{"line_number":294,"context_line":"        - Token clients for \u0027identity\u0027 must be given an \u0027auth_url\u0027 parameter"},{"line_number":295,"context_line":"        - Volume client for \u0027volume\u0027 accepts \u0027default_volume_size\u0027"},{"line_number":296,"context_line":"        - Servers client from \u0027compute\u0027 accepts \u0027enable_instance_password\u0027"},{"line_number":297,"context_line":"        "},{"line_number":298,"context_line":"        If Tempest configuration is used, parameters will loaded in the"},{"line_number":299,"context_line":"        Registry automatically for all service client (Tempest stable ones"},{"line_number":300,"context_line":"        and plugins)."}],"source_content_type":"text/x-python","patch_set":4,"id":"bff0334d_5ed19f22","line":297,"updated":"2017-04-11 02:43:08.000000000","message":"ditto","commit_id":"b1b025648383b3510c855e4e2df9f8d2d73a7dd6"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a1d7729780edf94177bc7d091955f09de796b4fb","unresolved":false,"context_lines":[{"line_number":295,"context_line":"        - Volume client for \u0027volume\u0027 accepts \u0027default_volume_size\u0027"},{"line_number":296,"context_line":"        - Servers client from \u0027compute\u0027 accepts \u0027enable_instance_password\u0027"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"        If Tempest configuration is used, parameters will loaded in the"},{"line_number":299,"context_line":"        Registry automatically for all service client (Tempest stable ones"},{"line_number":300,"context_line":"        and plugins)."},{"line_number":301,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bff0334d_144992b5","line":298,"updated":"2017-04-12 01:43:24.000000000","message":"\"will be loaded\"?","commit_id":"22ab5353ae07b9d6796baf811ed39c9b2b7fdf5a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b8c0e810ce333f67c412889ca13dffe4d2af9653","unresolved":false,"context_lines":[{"line_number":295,"context_line":"        - Volume client for \u0027volume\u0027 accepts \u0027default_volume_size\u0027"},{"line_number":296,"context_line":"        - Servers client from \u0027compute\u0027 accepts \u0027enable_instance_password\u0027"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"        If Tempest configuration is used, parameters will loaded in the"},{"line_number":299,"context_line":"        Registry automatically for all service client (Tempest stable ones"},{"line_number":300,"context_line":"        and plugins)."},{"line_number":301,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bff0334d_a1dd07e1","line":298,"in_reply_to":"bff0334d_144992b5","updated":"2017-04-12 03:42:16.000000000","message":"as this is doc string, we should fix this.","commit_id":"22ab5353ae07b9d6796baf811ed39c9b2b7fdf5a"}]}