)]}'
{"cinder/volume/drivers/nimble.py":[{"author":{"_account_id":8247,"name":"Thang Pham","email":"thang.g.pham@gmail.com","username":"thang.pham"},"change_message_id":"7fe89977bfff1752e2d4ddefd019e7965fbd83e9","unresolved":false,"context_lines":[{"line_number":286,"context_line":"            # Just use a single pool for now, FIXME to support multiple"},{"line_number":287,"context_line":"            # pools"},{"line_number":288,"context_line":"            single_pool \u003d {}"},{"line_number":289,"context_line":"            single_pool.update(dict("},{"line_number":290,"context_line":"                pool_name\u003dbackend_name,"},{"line_number":291,"context_line":"                total_capacity_gb\u003dtotal_capacity,"},{"line_number":292,"context_line":"                free_capacity_gb\u003dfree_space,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_04bffefb","line":289,"updated":"2015-08-25 00:57:40.000000000","message":"Not sure why you have to use update here, instead of just doing:\n    single_pool \u003d dict(pool_name\u003dbackend_name, ...)","commit_id":"0648992430a98a6c987979b64ae02c36a79572d8"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"737cdf013e37b05d4e1f28c8ec2a369a85bff5f2","unresolved":false,"context_lines":[{"line_number":286,"context_line":"            # Just use a single pool for now, FIXME to support multiple"},{"line_number":287,"context_line":"            # pools"},{"line_number":288,"context_line":"            single_pool \u003d {}"},{"line_number":289,"context_line":"            single_pool.update(dict("},{"line_number":290,"context_line":"                pool_name\u003dbackend_name,"},{"line_number":291,"context_line":"                total_capacity_gb\u003dtotal_capacity,"},{"line_number":292,"context_line":"                free_capacity_gb\u003dfree_space,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_926ef0c7","line":289,"in_reply_to":"fa1b9901_04bffefb","updated":"2015-08-25 01:56:00.000000000","message":"This is carried over from the lvm driver which uses the same style. I could change it, if it matters.","commit_id":"0648992430a98a6c987979b64ae02c36a79572d8"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"6e6cd976cb3e4f60b9b86845ed7ed8e73dc187f4","unresolved":false,"context_lines":[{"line_number":286,"context_line":"            # Just use a single pool for now, FIXME to support multiple"},{"line_number":287,"context_line":"            # pools"},{"line_number":288,"context_line":"            single_pool \u003d {}"},{"line_number":289,"context_line":"            single_pool.update(dict("},{"line_number":290,"context_line":"                pool_name\u003dbackend_name,"},{"line_number":291,"context_line":"                total_capacity_gb\u003dtotal_capacity,"},{"line_number":292,"context_line":"                free_capacity_gb\u003dfree_space,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_ff111353","line":289,"in_reply_to":"fa1b9901_926ef0c7","updated":"2015-08-25 15:27:37.000000000","message":"Please update it just to set single_pool. The LVM driver should probably be updated in a future patch as well. No need to duplicate bad (or just less than optimal) code.","commit_id":"0648992430a98a6c987979b64ae02c36a79572d8"}]}
