)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e5c48d3aa68200743478538275476b454cf0acba","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Liang Fang \u003cliang.a.fang@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-14 17:06:31 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix uploading cinder volumes to images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When uploading a volume to glance as an image, the glance server don\u0027t"},{"line_number":10,"context_line":"know the image size, so the backend storage server(such as ceph) need to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_7b192c20","line":7,"updated":"2018-09-14 16:39:46.000000000","message":"This description is very misleading. This should state what is actually being done.","commit_id":"4867b59c472aac494a3ace2776659192f88fe91c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e5c48d3aa68200743478538275476b454cf0acba","unresolved":false,"context_lines":[{"line_number":16,"context_line":"image again and again"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Story: 2003774"},{"line_number":19,"context_line":"Task: 26466"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: I372ad7c29b952ebf7e020154b1fd566d2748581a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_3bd17430","line":19,"updated":"2018-09-14 16:39:46.000000000","message":"Glance does not use storyboard.","commit_id":"4867b59c472aac494a3ace2776659192f88fe91c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4b929aa310d0eaeed1f2321a7ac33eb284df9b28","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Liang Fang \u003cliang.a.fang@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-17 11:29:30 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Specify image size to avoid glance backend resize when uploading volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When uploading a volume to glance as an image, the glance server don\u0027t"},{"line_number":10,"context_line":"know the image size, so the backend storage server(such as ceph) need to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_70db657c","line":7,"updated":"2018-09-17 14:11:59.000000000","message":"Please read https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"1c837ef46045e4cca2bac9c451b34d75a1b7e89a"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4b929aa310d0eaeed1f2321a7ac33eb284df9b28","unresolved":false,"context_lines":[{"line_number":15,"context_line":"ceph can allocate an image with exact size, and don\u0027t need to resize the"},{"line_number":16,"context_line":"image again and again"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Partial-Bug: #1792710"},{"line_number":19,"context_line":"Story: 2003774"},{"line_number":20,"context_line":"Task: 26466"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_f099b5cf","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":7},"updated":"2018-09-17 14:11:59.000000000","message":"Should be Closes-bug.","commit_id":"1c837ef46045e4cca2bac9c451b34d75a1b7e89a"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4b929aa310d0eaeed1f2321a7ac33eb284df9b28","unresolved":false,"context_lines":[{"line_number":16,"context_line":"image again and again"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Partial-Bug: #1792710"},{"line_number":19,"context_line":"Story: 2003774"},{"line_number":20,"context_line":"Task: 26466"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I372ad7c29b952ebf7e020154b1fd566d2748581a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_d0a4b909","line":20,"range":{"start_line":19,"start_character":0,"end_line":20,"end_character":11},"updated":"2018-09-17 14:11:59.000000000","message":"These don\u0027t belong in this patch.","commit_id":"1c837ef46045e4cca2bac9c451b34d75a1b7e89a"}],"glance/api/v2/image_data.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e5c48d3aa68200743478538275476b454cf0acba","unresolved":false,"context_lines":[{"line_number":419,"context_line":"            request.is_body_readable \u003d True"},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"        image_size \u003d request.content_length or None"},{"line_number":422,"context_line":"        if image_size is None or image_size \u003d\u003d 0:"},{"line_number":423,"context_line":"            image_size \u003d request.headers.get(\u0027x-image-meta-size\u0027, 0)"},{"line_number":424,"context_line":"            if image_size is None or image_size \u003d\u003d \u0027None\u0027:"},{"line_number":425,"context_line":"                image_size \u003d 0"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_bb926457","line":422,"updated":"2018-09-14 16:39:46.000000000","message":"The same thing is being done in two different locations. This would be better to factor out into a function that can just be called from both locations.","commit_id":"4867b59c472aac494a3ace2776659192f88fe91c"}]}