)]}'
{"magnumclient/v1/certificates_shell.py":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f2c84bd51c55806408403f3ec68070907110263c","unresolved":false,"context_lines":[{"line_number":44,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"},{"line_number":45,"context_line":"           help\u003d_(\u0027ID or name of the bay.\u0027))"},{"line_number":46,"context_line":"@utils.arg(\u0027--cluster\u0027,"},{"line_number":47,"context_line":"           required\u003dFalse,"},{"line_number":48,"context_line":"           metavar\u003d\u0027\u003ccluster\u003e\u0027,"},{"line_number":49,"context_line":"           help\u003d_(\u0027ID or name of the cluster.\u0027))"},{"line_number":50,"context_line":"def do_ca_show(cs, args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa31d9ce_6cf236d4","side":"PARENT","line":47,"updated":"2017-02-13 09:24:41.000000000","message":"For mandatory args, can\u0027t we just change to required\u003dTrue","commit_id":"7cb91456538a1a72033a5c6df55aae99ec51c8fe"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f2c84bd51c55806408403f3ec68070907110263c","unresolved":false,"context_lines":[{"line_number":43,"context_line":"           required\u003dFalse,"},{"line_number":44,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"},{"line_number":45,"context_line":"           help\u003d_(\u0027ID or name of the bay.\u0027))"},{"line_number":46,"context_line":"@utils.arg(\u0027cluster\u0027,"},{"line_number":47,"context_line":"           metavar\u003d\u0027\u003ccluster\u003e\u0027,"},{"line_number":48,"context_line":"           help\u003d_(\u0027ID or name of the cluster.\u0027))"},{"line_number":49,"context_line":"def do_ca_show(cs, args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa31d9ce_cc6162aa","line":46,"updated":"2017-02-13 09:24:41.000000000","message":"I think we need to deprecate the earlier option before we can change the parameter type to positional.\n\nWhat i can understand from here is: out of --bay or --cluster only one should be used. Please confirm?\nIf yes, we should update the code and UT to work with the behaviour(out of two one is manadtory). What happens if both are provided","commit_id":"8e8a92e46f2d37c65594bc92607331e0fd74a3ac"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"41fa0c9212793a4a2bd3642c846211b8b376517e","unresolved":false,"context_lines":[{"line_number":43,"context_line":"           required\u003dFalse,"},{"line_number":44,"context_line":"           metavar\u003d\u0027\u003cbay\u003e\u0027,"},{"line_number":45,"context_line":"           help\u003d_(\u0027ID or name of the bay.\u0027))"},{"line_number":46,"context_line":"@utils.arg(\u0027cluster\u0027,"},{"line_number":47,"context_line":"           metavar\u003d\u0027\u003ccluster\u003e\u0027,"},{"line_number":48,"context_line":"           help\u003d_(\u0027ID or name of the cluster.\u0027))"},{"line_number":49,"context_line":"def do_ca_show(cs, args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfeb2761_97e74a13","line":46,"in_reply_to":"fa31d9ce_cc6162aa","updated":"2017-04-04 06:27:28.000000000","message":"Yes either bay or cluster is needed. In case both the bay and cluster is provided, the cluster will be chosen.","commit_id":"8e8a92e46f2d37c65594bc92607331e0fd74a3ac"}]}