)]}'
{"cinder/tests/test_volume.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"e0249c77eeb51a4631fce81fc8738463cbf7e391","unresolved":false,"context_lines":[{"line_number":897,"context_line":"    def test_create_volume_with_mindisk_error(self):"},{"line_number":898,"context_line":"        \"\"\""},{"line_number":899,"context_line":"        Verify that a volume which is smaller than minDisk will"},{"line_number":900,"context_line":"        fail correctly."},{"line_number":901,"context_line":"        \"\"\""},{"line_number":902,"context_line":"        class _FakeImageService:"},{"line_number":903,"context_line":"            def __init__(self, db_driver\u003dNone, image_service\u003dNone):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2Fl0M%3D","line":900,"updated":"2013-05-02 16:57:27.000000000","message":"I know it\u0027s wrong in other places in this file, but could you please fix this to match the format in HACKING for multi line docstrings?","commit_id":"266bad1cc9525ea6a45185f1d9dc4194aecd2822"},{"author":{"_account_id":5215,"name":"Dermot Tynan","email":"tynan@hpe.com","username":"dtynan"},"change_message_id":"38a120ab93dda36b0692635d741dea5814c31bea","unresolved":false,"context_lines":[{"line_number":897,"context_line":"    def test_create_volume_with_mindisk_error(self):"},{"line_number":898,"context_line":"        \"\"\""},{"line_number":899,"context_line":"        Verify that a volume which is smaller than minDisk will"},{"line_number":900,"context_line":"        fail correctly."},{"line_number":901,"context_line":"        \"\"\""},{"line_number":902,"context_line":"        class _FakeImageService:"},{"line_number":903,"context_line":"            def __init__(self, db_driver\u003dNone, image_service\u003dNone):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2Flyc%3D","line":900,"in_reply_to":"AAAAOn%2F%2Fl0M%3D","updated":"2013-05-02 17:29:18.000000000","message":"Done","commit_id":"266bad1cc9525ea6a45185f1d9dc4194aecd2822"}],"cinder/volume/api.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"ef8d6ee8e44269295e6292d6a8e71e3fd135da23","unresolved":false,"context_lines":[{"line_number":148,"context_line":"            #check image size is not larger than volume size."},{"line_number":149,"context_line":"            if image_size_in_gb \u003e size:"},{"line_number":150,"context_line":"                msg \u003d _(\u0027Size of specified image is larger than volume size.\u0027)"},{"line_number":151,"context_line":"                raise exception.InvalidInput(reason\u003dmsg)"},{"line_number":152,"context_line":"            if image_meta.get(\u0027min_disk\u0027, 0) \u003e size:"},{"line_number":153,"context_line":"                msg \u003d _(\u0027Image minDisk size is larger than the volume size.\u0027)"},{"line_number":154,"context_line":"                raise exception.InvalidInput(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOn%2F%2FlIw%3D","line":151,"updated":"2013-05-03 02:18:55.000000000","message":"Just a suggestion.  I had to read the commit message several times to understand what the goal really was here.  Perhaps a comment explaining the check would help.","commit_id":"c7678157b1a9930b0d47be7e8ff0ec0d07accbce"},{"author":{"_account_id":5215,"name":"Dermot Tynan","email":"tynan@hpe.com","username":"dtynan"},"change_message_id":"017eb38388bda673e3f72d3ac85ed1d7ba6a59ea","unresolved":false,"context_lines":[{"line_number":148,"context_line":"            #check image size is not larger than volume size."},{"line_number":149,"context_line":"            if image_size_in_gb \u003e size:"},{"line_number":150,"context_line":"                msg \u003d _(\u0027Size of specified image is larger than volume size.\u0027)"},{"line_number":151,"context_line":"                raise exception.InvalidInput(reason\u003dmsg)"},{"line_number":152,"context_line":"            if image_meta.get(\u0027min_disk\u0027, 0) \u003e size:"},{"line_number":153,"context_line":"                msg \u003d _(\u0027Image minDisk size is larger than the volume size.\u0027)"},{"line_number":154,"context_line":"                raise exception.InvalidInput(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOn%2F%2FkwA%3D","line":151,"in_reply_to":"AAAAOn%2F%2FlIw%3D","updated":"2013-05-03 10:29:47.000000000","message":"Done","commit_id":"c7678157b1a9930b0d47be7e8ff0ec0d07accbce"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"ef8d6ee8e44269295e6292d6a8e71e3fd135da23","unresolved":false,"context_lines":[{"line_number":394,"context_line":"                    try:"},{"line_number":395,"context_line":"                        filter_func \u003d filter_mapping[opt]"},{"line_number":396,"context_line":"                    except KeyError:"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"                        def filter_func(volume, value):"},{"line_number":399,"context_line":"                            return volume.get(opt, not_found) \u003d\u003d value"},{"line_number":400,"context_line":"                    if not filter_func(volume, values):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOn%2F%2FlJk%3D","line":397,"updated":"2013-05-03 02:18:55.000000000","message":"Did you mean to add an extra line here?","commit_id":"c7678157b1a9930b0d47be7e8ff0ec0d07accbce"},{"author":{"_account_id":5215,"name":"Dermot Tynan","email":"tynan@hpe.com","username":"dtynan"},"change_message_id":"017eb38388bda673e3f72d3ac85ed1d7ba6a59ea","unresolved":false,"context_lines":[{"line_number":394,"context_line":"                    try:"},{"line_number":395,"context_line":"                        filter_func \u003d filter_mapping[opt]"},{"line_number":396,"context_line":"                    except KeyError:"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"                        def filter_func(volume, value):"},{"line_number":399,"context_line":"                            return volume.get(opt, not_found) \u003d\u003d value"},{"line_number":400,"context_line":"                    if not filter_func(volume, values):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOn%2F%2Fkv4%3D","line":397,"in_reply_to":"AAAAOn%2F%2FlJk%3D","updated":"2013-05-03 10:29:47.000000000","message":"pep8 seemed to have an issue with it. I\u0027ll remove it.","commit_id":"c7678157b1a9930b0d47be7e8ff0ec0d07accbce"}]}