)]}'
{"tempest/common/negative_rest_client.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"581fd3e4e220b0d07bd4cca986f28e1825a32769","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        region \u003d self._get_region(service)"},{"line_number":33,"context_line":"        super(NegativeRestClient, self).__init__(auth_provider,"},{"line_number":34,"context_line":"                                                 service, region)"},{"line_number":35,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":36,"context_line":"        ca_certs \u003d CONF.identity.ca_certificates_file"},{"line_number":37,"context_line":"        self.http_obj \u003d http.ClosingHttp("},{"line_number":38,"context_line":"            disable_ssl_certificate_validation\u003ddscv, ca_certs\u003dca_certs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_4913695c","line":35,"updated":"2015-01-21 18:02:55.000000000","message":"Considering the effort in progress to remove any dependency from CONF in the service clients, in preparation for migration to tempest-lib, this may not the best place to fix the issue.\n\nWhat you need is to add dscv and ca_certs to the __init__ in L31. In any case you should pass both values down to RestClient in L34, rather than redefining http_obj.\n\nAnother way to solve this may be to inherit from ServiceClient instead of RestClient, but I you\u0027ll have to check if there isn\u0027t any unwanted side effect in doing so.","commit_id":"81d5ce69ffd89d59531941ba6e39c1093f4bbeff"},{"author":{"_account_id":13372,"name":"Eugene Bagdasaryan","email":"ebagdasaryan@me.com","username":"ebagdasa"},"change_message_id":"8f892441b0c37046af4aae0cfd669a43d85565d8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"                 trace_requests\u003d\u0027\u0027):"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"        region \u003d self._get_region(service)"},{"line_number":35,"context_line":"        dscv \u003d disable_ssl_certificate_validation"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        super(NegativeRestClient, self).__init__("},{"line_number":38,"context_line":"            auth_provider, service, region,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a930d6b_1a0eb88d","line":35,"range":{"start_line":35,"start_character":8,"end_line":35,"end_character":12},"updated":"2015-01-23 12:03:19.000000000","message":"Do you need this variable, looks like you just pass it as a param? Or it is just to follow line length policy?","commit_id":"8edf53148be15b881addb89866eba14d9f69e67e"}]}