)]}'
{"tools/kolla-ansible":[{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"300de3459f0926792baef823b8ae04c1fad91782","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    --keyfile, -k \u003ckey_file\u003e         Specify path to ansible vault keyfile"},{"line_number":35,"context_line":"    --help, -h                       Show this usage information"},{"line_number":36,"context_line":"    --tags, -t \u003ctags\u003e                Only run plays and tasks tagged with these values"},{"line_number":37,"context_line":"    --extra, -e \u003cansible variables\u003e  Add extra arguments to run ansible-playbook"},{"line_number":38,"context_line":"    --verbose, -v                    Increase verbosity of ansible-playbook"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Commands:"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba0121b8_e9fe376a","line":37,"range":{"start_line":37,"start_character":37,"end_line":37,"end_character":80},"updated":"2016-03-29 13:18:20.000000000","message":"This patch set will be very useful.\nSince these variables are passed to the -e argument of ansible-playbook in line 164 below, could you change this to match the description for -e in the output of  \"ansible-playbook --help\"  so that the user knows that the variables need to be set as key\u003dvalue or YAML/JSON.\n\n  $ ansible-playbook --help\n    -e EXTRA_VARS, --extra-vars\u003dEXTRA_VARS\n           set additional variables as key\u003dvalue or YAML/JSON\n\nI\u0027d change:\n\n  Add extra arguments to run ansible-playbook\n\nto:\n\n  Set additional variables as key\u003dvalue or YAML/JSON passed to ansible-playbook","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"ef6208ab483addeec58f1a9dbf1442ca0fc74649","unresolved":false,"context_lines":[{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"SHORT_OPTS\u003d\"hi:p:t:k:e:v\""},{"line_number":53,"context_line":"LONG_OPTS\u003d\"help,inventory:,playbook:,tags:,keyfile:,verbose,configdir:,extra:\""},{"line_number":54,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":55,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da0c15f0_87d8651f","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":14},"updated":"2016-03-24 22:38:19.000000000","message":"should it not be \u0027:\u0027 between h and i?? these two are different parameters.","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"24ddc5e49a9a9e769263b3c071516e3fee383570","unresolved":false,"context_lines":[{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"SHORT_OPTS\u003d\"hi:p:t:k:e:v\""},{"line_number":53,"context_line":"LONG_OPTS\u003d\"help,inventory:,playbook:,tags:,keyfile:,verbose,configdir:,extra:\""},{"line_number":54,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":55,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da0c15f0_27f878e5","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":14},"in_reply_to":"da0c15f0_273b59ed","updated":"2016-03-24 22:51:10.000000000","message":"No, there should not be any \":\" after h. \"-h\" displays the help message; it does not require any argument.  -Hui","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"cf8c48848484f7e3097689ad3eb72d0f8909451f","unresolved":false,"context_lines":[{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"SHORT_OPTS\u003d\"hi:p:t:k:e:v\""},{"line_number":53,"context_line":"LONG_OPTS\u003d\"help,inventory:,playbook:,tags:,keyfile:,verbose,configdir:,extra:\""},{"line_number":54,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":55,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da0c15f0_e76200ad","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":14},"in_reply_to":"da0c15f0_27f878e5","updated":"2016-03-24 22:52:59.000000000","message":"Got it thank you for explanation..","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"738cddca1e4aa162ad91237497c752894543a5cc","unresolved":false,"context_lines":[{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"SHORT_OPTS\u003d\"hi:p:t:k:e:v\""},{"line_number":53,"context_line":"LONG_OPTS\u003d\"help,inventory:,playbook:,tags:,keyfile:,verbose,configdir:,extra:\""},{"line_number":54,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":55,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da0c15f0_273b59ed","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":14},"in_reply_to":"da0c15f0_47d80df0","updated":"2016-03-24 22:47:12.000000000","message":"so you are say that since -h does not have any arguments it does not require \u0027:\u0027 following it right? it should not be like \"h:i:p:t:k:e:v\"?","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"8f3533c540e22e84aba11d09708e6366811ac7e3","unresolved":false,"context_lines":[{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"SHORT_OPTS\u003d\"hi:p:t:k:e:v\""},{"line_number":53,"context_line":"LONG_OPTS\u003d\"help,inventory:,playbook:,tags:,keyfile:,verbose,configdir:,extra:\""},{"line_number":54,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":55,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da0c15f0_47d80df0","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":14},"in_reply_to":"da0c15f0_87d8651f","updated":"2016-03-24 22:42:09.000000000","message":"Hi, Serguer, thanks for your comment. If I understand your correctly, there is not \":\" between h and i because h does not need any argument.","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"cc472563f65b63c0cc9a4155e6c1a35f2c61575a","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"SHORT_OPTS\u003d\"hi:p:t:k:e:v\""},{"line_number":53,"context_line":"LONG_OPTS\u003d\"help,inventory:,playbook:,tags:,keyfile:,verbose,configdir:,extra:\""},{"line_number":54,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"eval set -- \"$ARGS\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba0121b8_87f5cdcc","line":53,"range":{"start_line":53,"start_character":70,"end_line":53,"end_character":77},"updated":"2016-03-29 19:29:11.000000000","message":"follow the order, it must be so \"help,inventory:,playbook:,tags:,keyfile:,extra:,verbose,configdir:\"","commit_id":"11f41e335d1c053787350e88e12503087a3c0e15"}]}
