)]}'
{"cinderclient/client.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f7490031ed091b2dee9cf57fc2ce65de994fd697","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        # Note(tpatil): The standard call raises errors from"},{"line_number":100,"context_line":"        # keystoneauth, here we need to raise the cinderclient errors."},{"line_number":101,"context_line":"        raise_exc \u003d kwargs.pop(\u0027raise_exc\u0027, True)"},{"line_number":102,"context_line":"        resp \u003d super(SessionClient, self).request(*args,"},{"line_number":103,"context_line":"                                                  raise_exc\u003dFalse,"},{"line_number":104,"context_line":"                                                  **kwargs)"},{"line_number":105,"context_line":"        if raise_exc and resp.status_code \u003e\u003d 400:"},{"line_number":106,"context_line":"            raise exceptions.from_response(resp, resp.text)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        return resp, resp.text"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def _cs_request(self, url, method, **kwargs):"},{"line_number":111,"context_line":"        # this function is mostly redundant but makes compatibility easier"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_8729153a","line":108,"range":{"start_line":102,"start_character":8,"end_line":108,"end_character":30},"updated":"2016-06-10 22:46:34.000000000","message":"this change is causing the functional tests to fail, since resp was a tuple","commit_id":"d6afade4281d0fd62ae863158b38d20edd17cf49"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"567dc954681028979eda920993534c671818b8cd","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        # Note(tpatil): The standard call raises errors from"},{"line_number":100,"context_line":"        # keystoneauth, here we need to raise the cinderclient errors."},{"line_number":101,"context_line":"        raise_exc \u003d kwargs.pop(\u0027raise_exc\u0027, True)"},{"line_number":102,"context_line":"        resp \u003d super(SessionClient, self).request(*args,"},{"line_number":103,"context_line":"                                                  raise_exc\u003dFalse,"},{"line_number":104,"context_line":"                                                  **kwargs)"},{"line_number":105,"context_line":"        if raise_exc and resp.status_code \u003e\u003d 400:"},{"line_number":106,"context_line":"            raise exceptions.from_response(resp, resp.text)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        return resp, resp.text"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def _cs_request(self, url, method, **kwargs):"},{"line_number":111,"context_line":"        # this function is mostly redundant but makes compatibility easier"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_53943d36","line":108,"range":{"start_line":102,"start_character":8,"end_line":108,"end_character":30},"in_reply_to":"7aa08908_8729153a","updated":"2016-06-22 14:12:33.000000000","message":"yeah, I\u0027m not sure why this was changed. I\u0027ll restore the tuple version","commit_id":"d6afade4281d0fd62ae863158b38d20edd17cf49"}],"cinderclient/tests/unit/test_client.py":[{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"df7f2d7b17ed6a91c9cedaa40990dd2d97f0bc16","unresolved":false,"context_lines":[{"line_number":79,"context_line":"                          cinderclient.client.get_volume_api_from_url,"},{"line_number":80,"context_line":"                          unknown_url)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    @mock.patch.object(adapter.LegacyJsonAdapter, \u0027request\u0027)"},{"line_number":83,"context_line":"    @mock.patch.object(exceptions, \u0027from_response\u0027)"},{"line_number":84,"context_line":"    def test_sessionclient_request_method("},{"line_number":85,"context_line":"            self, mock_from_resp, mock_request):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_0af6058f","line":82,"range":{"start_line":82,"start_character":4,"end_line":82,"end_character":60},"updated":"2016-07-04 14:19:37.000000000","message":"keeping this line unaltered would cause body.json() to work improperly. refer to: https://github.com/openstack/keystoneauth/blob/80b58cc2f2aa73e2d3fc88ef2e27fa3db6ed0025/keystoneauth1/adapter.py#L334","commit_id":"fa07872078ca85b4db61ad2e2612764f38246cc9"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"df7f2d7b17ed6a91c9cedaa40990dd2d97f0bc16","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        # \u0027request\u0027 method of Adaptor will return 202 response"},{"line_number":114,"context_line":"        mock_request.return_value \u003d mock_response"},{"line_number":115,"context_line":"        session_client \u003d cinderclient.client.SessionClient(session\u003dmock.Mock())"},{"line_number":116,"context_line":"        response, body \u003d session_client.request(mock.sentinel.url,"},{"line_number":117,"context_line":"                                                \u0027POST\u0027, **kwargs)"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"        # In this case, from_response method will not get called"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_e4331012","line":116,"range":{"start_line":116,"start_character":25,"end_line":116,"end_character":47},"updated":"2016-07-04 14:19:37.000000000","message":"it seems that, for some reason, the above mocked adapter is not being called here, causing the internal session_client version of the request function to be called","commit_id":"fa07872078ca85b4db61ad2e2612764f38246cc9"}],"cinderclient/tests/unit/test_shell.py":[{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"8eacc18b20a1a765a58d64b37b1c0812c06624e9","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        os_auth_url \u003d \"https://WrongDiscoveryResponse.discovery.com:35357/v2.0\""},{"line_number":108,"context_line":"        self.register_keystone_auth_fixture(mocker, os_auth_url)"},{"line_number":109,"context_line":"        self.assertRaises(DiscoveryFailure, _shell._discover_auth_versions,"},{"line_number":110,"context_line":"                          None, auth_url\u003dos_auth_url)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        os_auth_url \u003d \"https://DiscoveryNotSupported.discovery.com:35357/v2.0\""},{"line_number":113,"context_line":"        self.register_keystone_auth_fixture(mocker, os_auth_url)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_d3adcd32","line":110,"range":{"start_line":110,"start_character":26,"end_line":110,"end_character":31},"updated":"2016-06-22 14:18:16.000000000","message":"having problems with this one, I\u0027m sure how this was working before \u003d\\ \npassing no session here should raise a KeyError: https://github.com/openstack/keystoneauth/blob/80b58cc2f2aa73e2d3fc88ef2e27fa3db6ed0025/keystoneauth1/discover.py#L40","commit_id":"fa07872078ca85b4db61ad2e2612764f38246cc9"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"59b68b83444973a33f0cc01d46933bae3698904d","unresolved":false,"context_lines":[{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    @requests_mock.Mocker()"},{"line_number":104,"context_line":"    @mock.patch(\u0027keystoneauth1.session.Session\u0027)"},{"line_number":105,"context_line":"    def test_version_discovery(self, mock_session, mocker):"},{"line_number":106,"context_line":"        _shell \u003d shell.OpenStackCinderShell()"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        os_auth_url \u003d \"https://WrongDiscoveryResponse.discovery.com:35357/v2.0\""}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_7a07f317","line":105,"range":{"start_line":105,"start_character":57,"end_line":105,"end_character":59},"updated":"2016-07-13 04:08:07.000000000","message":"this is failing because when you mock the session you prevent it from ever actually calling requests and hitting the requests_mock.","commit_id":"bbba84d347d1c2f3e33fa549dd00c6fe8f0fd311"}],"cinderclient/tests/unit/utils.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f7490031ed091b2dee9cf57fc2ce65de994fd697","unresolved":false,"context_lines":[{"line_number":93,"context_line":"    \"\"\""},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def __init__(self, data):"},{"line_number":96,"context_line":"        super(TestResponse, self).__init__()"},{"line_number":97,"context_line":"        self._text \u003d None"},{"line_number":98,"context_line":"        if isinstance(data, dict):"},{"line_number":99,"context_line":"            self.status_code \u003d data.get(\u0027status_code\u0027, None)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_c71b1d1b","line":96,"range":{"start_line":96,"start_character":8,"end_line":96,"end_character":44},"updated":"2016-06-10 22:46:34.000000000","message":"++ this matches novaclient","commit_id":"d6afade4281d0fd62ae863158b38d20edd17cf49"}],"requirements.txt":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f7490031ed091b2dee9cf57fc2ce65de994fd697","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr\u003e\u003d1.6 # Apache-2.0"},{"line_number":5,"context_line":"PrettyTable\u003c0.8,\u003e\u003d0.7 # BSD"},{"line_number":6,"context_line":"python-keystoneclient!\u003d1.8.0,!\u003d2.1.0,\u003e\u003d1.7.0 # Apache-2.0"},{"line_number":7,"context_line":"keystoneauth1\u003e\u003d2.1.0 # Apache-2.0"},{"line_number":8,"context_line":"requests\u003e\u003d2.10.0 # Apache-2.0"},{"line_number":9,"context_line":"simplejson\u003e\u003d2.2.0 # MIT"}],"source_content_type":"text/plain","patch_set":3,"id":"7aa08908_e720d94b","line":6,"range":{"start_line":6,"start_character":0,"end_line":6,"end_character":57},"updated":"2016-06-10 22:46:34.000000000","message":"remove this requirement","commit_id":"d6afade4281d0fd62ae863158b38d20edd17cf49"},{"author":{"_account_id":15742,"name":"Paulo Ewerton Gomes Fragoso","email":"pauloewerton@lsd.ufcg.edu.br","username":"pauloewerton"},"change_message_id":"567dc954681028979eda920993534c671818b8cd","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr\u003e\u003d1.6 # Apache-2.0"},{"line_number":5,"context_line":"PrettyTable\u003c0.8,\u003e\u003d0.7 # BSD"},{"line_number":6,"context_line":"python-keystoneclient!\u003d1.8.0,!\u003d2.1.0,\u003e\u003d1.7.0 # Apache-2.0"},{"line_number":7,"context_line":"keystoneauth1\u003e\u003d2.1.0 # Apache-2.0"},{"line_number":8,"context_line":"requests\u003e\u003d2.10.0 # Apache-2.0"},{"line_number":9,"context_line":"simplejson\u003e\u003d2.2.0 # MIT"}],"source_content_type":"text/plain","patch_set":3,"id":"3aaa91ec_53bb9dc4","line":6,"range":{"start_line":6,"start_character":0,"end_line":6,"end_character":57},"in_reply_to":"7aa08908_e720d94b","updated":"2016-06-22 14:12:33.000000000","message":"Done","commit_id":"d6afade4281d0fd62ae863158b38d20edd17cf49"}]}
