)]}'
{"magnumclient/v1/shell.py":[{"author":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"change_message_id":"1b0261ce64077f32ad2745cac8ad3910244b0c25","unresolved":false,"context_lines":[{"line_number":532,"context_line":"           help\u003d\u0027name of the container\u0027)"},{"line_number":533,"context_line":"@utils.arg(\u0027--image\u0027,"},{"line_number":534,"context_line":"           metavar\u003d\u0027\u003cimage\u003e\u0027,"},{"line_number":535,"context_line":"           help\u003d\u0027name or ID of the image\u0027)"},{"line_number":536,"context_line":"@utils.arg(\u0027--bay\u0027,"},{"line_number":537,"context_line":"           required\u003dTrue,"},{"line_number":538,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_0e5fcc0f","line":535,"updated":"2015-06-24 08:24:45.000000000","message":"Change to this\n\nID or name of the image.\n\n\nBTW:\nFor ID here, it means image id in docker server side, I think.\nFrom magnum client side, it can not directly get it. I am not sure if we expose such ID to user is meaningful, maybe it is OK.","commit_id":"252586a7b762e3c3947f08fffd3055b4c48137ea"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"299d9a218475100d167c36d6ba7d6927e72e1ea4","unresolved":false,"context_lines":[{"line_number":532,"context_line":"           help\u003d\u0027name of the container\u0027)"},{"line_number":533,"context_line":"@utils.arg(\u0027--image\u0027,"},{"line_number":534,"context_line":"           metavar\u003d\u0027\u003cimage\u003e\u0027,"},{"line_number":535,"context_line":"           help\u003d\u0027name or ID of the image\u0027)"},{"line_number":536,"context_line":"@utils.arg(\u0027--bay\u0027,"},{"line_number":537,"context_line":"           required\u003dTrue,"},{"line_number":538,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_6d2bd382","line":535,"in_reply_to":"fa32b979_0e5fcc0f","updated":"2015-06-25 00:31:50.000000000","message":"+1 for Kennan\u0027s comment. We can\u0027t get docker images id from magnum client.","commit_id":"252586a7b762e3c3947f08fffd3055b4c48137ea"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"69b28d27fbb090b430eb13301a6efe4f5c07d76d","unresolved":false,"context_lines":[{"line_number":532,"context_line":"           help\u003d\u0027name of the container\u0027)"},{"line_number":533,"context_line":"@utils.arg(\u0027--image\u0027,"},{"line_number":534,"context_line":"           metavar\u003d\u0027\u003cimage\u003e\u0027,"},{"line_number":535,"context_line":"           help\u003d\u0027name or ID of the image\u0027)"},{"line_number":536,"context_line":"@utils.arg(\u0027--bay\u0027,"},{"line_number":537,"context_line":"           required\u003dTrue,"},{"line_number":538,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_2e75a5f8","line":535,"in_reply_to":"fa32b979_0e5fcc0f","updated":"2015-06-25 15:06:48.000000000","message":"What is the difference of \"name or ID of the image\" and \"ID or name of the image\"?\n\nIMHO, as far as end user can get or know the docker image id, they can use it as they want.","commit_id":"252586a7b762e3c3947f08fffd3055b4c48137ea"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"85b502e0131232cffcbb5223737756abeed5bf56","unresolved":false,"context_lines":[{"line_number":532,"context_line":"           help\u003d\u0027name of the container\u0027)"},{"line_number":533,"context_line":"@utils.arg(\u0027--image\u0027,"},{"line_number":534,"context_line":"           metavar\u003d\u0027\u003cimage\u003e\u0027,"},{"line_number":535,"context_line":"           help\u003d\u0027name or ID of the image\u0027)"},{"line_number":536,"context_line":"@utils.arg(\u0027--bay\u0027,"},{"line_number":537,"context_line":"           required\u003dTrue,"},{"line_number":538,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_64013b04","line":535,"in_reply_to":"fa32b979_2e75a5f8","updated":"2015-06-27 15:33:31.000000000","message":"Suggest to set the debate aside and merge this patch first. Currently, the container API is broken and needs to be fixed by this patch. The help text issue is minor and we can adjust them later if we want.","commit_id":"252586a7b762e3c3947f08fffd3055b4c48137ea"}]}