)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"fac3ae571011711189944493a537985584051467","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     zwei \u003cleidong@unitedstack.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-25 10:09:04 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"volumes number greater than osapi_max_limit"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    It is possible that the length of the list we request is longer"},{"line_number":10,"context_line":"than osapi_max_limit, so we have to retrieve multiple times to"},{"line_number":11,"context_line":"get the complete list."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"But next request url is  will chanage `self.management_url + url`"},{"line_number":14,"context_line":"eg: self.management \u003d u\u0027http://10.0.5.123:8776/v2/1ac89d567ec24341b632cb8aaedfc41e\u0027"},{"line_number":15,"context_line":"eg: url \u003d u\u0027http://10.0.5.123:8776/v2/1ac89d567ec24341b632cb8aaedfc41e/volumes/detail?marker\u003d42da912f-43fa-49b3-9fb8-85f28b0e94a1\u0027"},{"line_number":16,"context_line":"So `self.management_url + url`  is error"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-bug: #1560862"},{"line_number":19,"context_line":"Change-Id: I416b63758373dc8ad29818d6952d1e9dd58d05f2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"da0c15f0_32a6ddc1","line":19,"range":{"start_line":7,"start_character":0,"end_line":19,"end_character":52},"updated":"2016-03-25 14:43:49.000000000","message":"Fix wrong request url when retrieving multiple request\n\nWhen the length of resource list is larger than osapi_max_limit.\nCinder will return a \"maker url\" which marks index of resources\nalready returned to client.\n\n_cs_request() function fo HTTPClient will link the endpoint url of cinder\nand this \"marker url\" together then use this wrong linked url to request\nthe server side, which cause a NotFound exception.","commit_id":"1b55aee0a10b9c4dbcbb62c0e3eab7d84b00015c"}],"cinderclient/client.py":[{"author":{"_account_id":842,"name":"Fergal Mc Carthy","email":"fergal.mc.carthy@suse.com","username":"fergal-mccarthy"},"change_message_id":"d5f80f6c5ca207f38e97b15eb0458f8d2d125093","unresolved":false,"context_lines":[{"line_number":291,"context_line":"            if self.projectid:"},{"line_number":292,"context_line":"                kwargs[\u0027headers\u0027][\u0027X-Auth-Project-Id\u0027] \u003d self.projectid"},{"line_number":293,"context_line":"            try:"},{"line_number":294,"context_line":"                if self.management_url not in url:"},{"line_number":295,"context_line":"                    url \u003d self.management_url + url"},{"line_number":296,"context_line":"                resp, body \u003d self.request(url, method, **kwargs)"},{"line_number":297,"context_line":"                return resp, body"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa0719c6_540bc87c","line":294,"range":{"start_line":294,"start_character":24,"end_line":294,"end_character":38},"updated":"2016-03-24 14:58:47.000000000","message":"Technically using \"in\" here is not correct... It will match the occurrence of the test string anywhere in the string we are testing against, e.g. \"la\" in \"blah\" is true.\n\nSo really this test should be:\n\n  if not url.startswith(self.management_url):\n\nPractically speaking it is unlikely that you would have an occurrence of self.management_url that is not at the start of url, but probably better to be specific/exact here.","commit_id":"315529a762458bc6b302b7d92672f41751de3e7a"}]}