)]}'
{"magnumclient/v1/baymodels_shell.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"a7f653c9e46193b8748b91f429a14def130c2ffd","unresolved":false,"context_lines":[{"line_number":77,"context_line":"@utils.arg(\u0027--docker-storage-driver\u0027,"},{"line_number":78,"context_line":"           metavar\u003d\u0027\u003cdocker-storage-driver\u003e\u0027,"},{"line_number":79,"context_line":"           default\u003d\u0027devicemapper\u0027,"},{"line_number":80,"context_line":"           help\u003d\u0027Select a docker storage driver\u0027)"},{"line_number":81,"context_line":"@utils.arg(\u0027--http-proxy\u0027,"},{"line_number":82,"context_line":"           metavar\u003d\u0027\u003chttp-proxy\u003e\u0027,"},{"line_number":83,"context_line":"           help\u003d\u0027The http_proxy address to use for nodes in bay.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9abb7d3a_66662fee","line":80,"range":{"start_line":80,"start_character":11,"end_line":80,"end_character":15},"updated":"2016-06-01 22:39:30.000000000","message":"Better to list the allowed values in the help text. E.g. \u0027devicemapper\u0027, \u0027overlayfs\u0027. Also, better to mention the default as well.","commit_id":"b907f2c549fb641ff42d8f1f062bb2f616cca716"},{"author":{"_account_id":9095,"name":"Murali Allada","email":"amuralis@hotmail.com","username":"murali.allada"},"change_message_id":"40a76b296c86e08a8c4e173a759b395f5801d7fb","unresolved":false,"context_lines":[{"line_number":77,"context_line":"@utils.arg(\u0027--docker-storage-driver\u0027,"},{"line_number":78,"context_line":"           metavar\u003d\u0027\u003cdocker-storage-driver\u003e\u0027,"},{"line_number":79,"context_line":"           default\u003d\u0027devicemapper\u0027,"},{"line_number":80,"context_line":"           help\u003d\u0027Select a docker storage driver\u0027)"},{"line_number":81,"context_line":"@utils.arg(\u0027--http-proxy\u0027,"},{"line_number":82,"context_line":"           metavar\u003d\u0027\u003chttp-proxy\u003e\u0027,"},{"line_number":83,"context_line":"           help\u003d\u0027The http_proxy address to use for nodes in bay.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9abb7d3a_b9e2f1d2","line":80,"range":{"start_line":80,"start_character":11,"end_line":80,"end_character":15},"in_reply_to":"9abb7d3a_66662fee","updated":"2016-06-02 01:10:13.000000000","message":"+1","commit_id":"b907f2c549fb641ff42d8f1f062bb2f616cca716"}]}