)]}'
{"manilaclient/osc/v2/share_snapshots.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c55b0a23303e93c99c95a7cda5997d372a389539","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                   \"Options include : available, error, creating, \""},{"line_number":194,"context_line":"                   \"deleting, manage_starting, manage_error, \""},{"line_number":195,"context_line":"                   \"unmanage_starting, unmanage_error, error_deleting. \""},{"line_number":196,"context_line":"                   \"If no state is provided, available will be used.\""},{"line_number":197,"context_line":"                   )"},{"line_number":198,"context_line":"        )"},{"line_number":199,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":10,"id":"9f560f44_44f0d4fd","line":196,"range":{"start_line":196,"start_character":20,"end_line":196,"end_character":68},"updated":"2020-09-10 05:20:45.000000000","message":"Then this option needs to have a default value.. \n\nit\u0027s good not to have a default value.","commit_id":"44d3f806d39d2436803a2237247e74a695a5beba"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"2699afeeb226f089a8df70d8d6f48d9b5f33ef1c","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                   \"Options include : available, error, creating, \""},{"line_number":194,"context_line":"                   \"deleting, manage_starting, manage_error, \""},{"line_number":195,"context_line":"                   \"unmanage_starting, unmanage_error, error_deleting. \""},{"line_number":196,"context_line":"                   \"If no state is provided, available will be used.\""},{"line_number":197,"context_line":"                   )"},{"line_number":198,"context_line":"        )"},{"line_number":199,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":10,"id":"9f560f44_b52bcee8","line":196,"range":{"start_line":196,"start_character":20,"end_line":196,"end_character":68},"in_reply_to":"9f560f44_44f0d4fd","updated":"2020-09-10 16:22:58.000000000","message":"Yeah, this doesn\u0027t make sense, I\u0027ll go ahead and delete this sentence here :)","commit_id":"44d3f806d39d2436803a2237247e74a695a5beba"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c55b0a23303e93c99c95a7cda5997d372a389539","unresolved":false,"context_lines":[{"line_number":333,"context_line":"            metavar\u003d\u0027\u003cusage\u003e\u0027,"},{"line_number":334,"context_line":"            default\u003d\u0027any\u0027,"},{"line_number":335,"context_line":"            choices\u003d[\u0027any\u0027, \u0027unused\u0027, \u0027used\u0027],"},{"line_number":336,"context_line":"            help\u003d_(\"Option to filter snapshots by usage. Default\u003d\u0027any\u0027\")"},{"line_number":337,"context_line":"        )"},{"line_number":338,"context_line":"        parser.add_argument("},{"line_number":339,"context_line":"            \"--limit\","}],"source_content_type":"text/x-python","patch_set":10,"id":"9f560f44_04389c9d","line":336,"range":{"start_line":336,"start_character":57,"end_line":336,"end_character":70},"updated":"2020-09-10 05:20:45.000000000","message":"How do you provide a default? This doesn\u0027t seem to work:\n\n\n $ openstack share snapshot list --usage\nusage: openstack share snapshot list [-h] [-f {csv,json,table,value,yaml}]\n                                     [-c COLUMN]\n                                     [--quote {all,minimal,none,nonnumeric}]\n                                     [--noindent] [--max-width \u003cinteger\u003e]\n                                     [--fit-width] [--print-empty]\n                                     [--sort-column SORT_COLUMN]\n                                     [--all-projects] [--name \u003cname\u003e]\n                                     [--description \u003cdescription\u003e]\n                                     [--status \u003cstatus\u003e] [--share \u003cshare\u003e]\n                                     [--usage \u003cusage\u003e]\n                                     [--limit \u003cnum-snapshots\u003e]\n                                     [--marker \u003csnapshot\u003e]\n                                     [--sort \u003ckey\u003e[:\u003cdirection\u003e]]\n                                     [--name~ \u003cname~\u003e]\n                                     [--description~ \u003cdescription~\u003e]\n                                     [--detail]\nopenstack share snapshot list: error: argument --usage: expected one argument","commit_id":"44d3f806d39d2436803a2237247e74a695a5beba"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"2699afeeb226f089a8df70d8d6f48d9b5f33ef1c","unresolved":false,"context_lines":[{"line_number":333,"context_line":"            metavar\u003d\u0027\u003cusage\u003e\u0027,"},{"line_number":334,"context_line":"            default\u003d\u0027any\u0027,"},{"line_number":335,"context_line":"            choices\u003d[\u0027any\u0027, \u0027unused\u0027, \u0027used\u0027],"},{"line_number":336,"context_line":"            help\u003d_(\"Option to filter snapshots by usage. Default\u003d\u0027any\u0027\")"},{"line_number":337,"context_line":"        )"},{"line_number":338,"context_line":"        parser.add_argument("},{"line_number":339,"context_line":"            \"--limit\","}],"source_content_type":"text/x-python","patch_set":10,"id":"9f560f44_55d232fe","line":336,"range":{"start_line":336,"start_character":57,"end_line":336,"end_character":70},"in_reply_to":"9f560f44_04389c9d","updated":"2020-09-10 16:22:58.000000000","message":"Hmm, good note. So presumably `openstack share snapshot list` and `openstack share snapshot list --usage any` would give the exact same result. Do we even need the \u0027any\u0027 option? Might just be no default but also no \u0027any\u0027 option?","commit_id":"44d3f806d39d2436803a2237247e74a695a5beba"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"241ff26f3938cd29177971e7b2e3b6689d7a2b74","unresolved":false,"context_lines":[{"line_number":333,"context_line":"            metavar\u003d\u0027\u003cusage\u003e\u0027,"},{"line_number":334,"context_line":"            default\u003d\u0027any\u0027,"},{"line_number":335,"context_line":"            choices\u003d[\u0027any\u0027, \u0027unused\u0027, \u0027used\u0027],"},{"line_number":336,"context_line":"            help\u003d_(\"Option to filter snapshots by usage. Default\u003d\u0027any\u0027\")"},{"line_number":337,"context_line":"        )"},{"line_number":338,"context_line":"        parser.add_argument("},{"line_number":339,"context_line":"            \"--limit\","}],"source_content_type":"text/x-python","patch_set":10,"id":"9f560f44_706024d0","line":336,"range":{"start_line":336,"start_character":57,"end_line":336,"end_character":70},"in_reply_to":"9f560f44_55d232fe","updated":"2020-09-10 16:42:09.000000000","message":"yes, that makes sense, we can drop the \"any\" and the default,\n\nthere\u0027s no need, as you observed, that\u0027s what the server will return when \"usage\" isn\u0027t specified.","commit_id":"44d3f806d39d2436803a2237247e74a695a5beba"}]}
