)]}'
{"saharaclient/shell.py":[{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"f391244b1c1123220a6756184ca657f7ec9406a3","unresolved":false,"context_lines":[{"line_number":276,"context_line":"                            default\u003dcliutils.env(\u0027OS_REGION_NAME\u0027,"},{"line_number":277,"context_line":"                                                 \u0027SAHARA_REGION_NAME\u0027),"},{"line_number":278,"context_line":"                            help\u003d\u0027Defaults to env[OS_REGION_NAME].\u0027)"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--os_region_name\u0027,"},{"line_number":280,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"        parser.add_argument(\u0027--service-type\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa53dc9_9f835d56","line":279,"updated":"2015-04-08 19:02:51.000000000","message":"should be \"region_name\", right? or above should be --os-region-name","commit_id":"5d31a1337351271c4ef8bc24c6715ee27f85fd8b"},{"author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"change_message_id":"a3f9a7ef6679a01af0c215de22f0e60842e8cbdf","unresolved":false,"context_lines":[{"line_number":276,"context_line":"                            default\u003dcliutils.env(\u0027OS_REGION_NAME\u0027,"},{"line_number":277,"context_line":"                                                 \u0027SAHARA_REGION_NAME\u0027),"},{"line_number":278,"context_line":"                            help\u003d\u0027Defaults to env[OS_REGION_NAME].\u0027)"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--os_region_name\u0027,"},{"line_number":280,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"        parser.add_argument(\u0027--service-type\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa53dc9_1a0e3b3b","line":279,"in_reply_to":"9aa53dc9_9f835d56","updated":"2015-04-08 19:49:43.000000000","message":"Done","commit_id":"5d31a1337351271c4ef8bc24c6715ee27f85fd8b"},{"author":{"_account_id":13953,"name":"Elise Gafford","email":"egafford@redhat.com","username":"egafford"},"change_message_id":"009ae70e60974ba221d141e74c2dcfc7b0c1d0aa","unresolved":false,"context_lines":[{"line_number":274,"context_line":"        parser.add_argument(\u0027--region-name\u0027,"},{"line_number":275,"context_line":"                            metavar\u003d\u0027\u003cregion-name\u003e\u0027,"},{"line_number":276,"context_line":"                            default\u003dcliutils.env(\u0027OS_REGION_NAME\u0027,"},{"line_number":277,"context_line":"                                                 \u0027SAHARA_REGION_NAME\u0027),"},{"line_number":278,"context_line":"                            help\u003d\u0027Defaults to env[OS_REGION_NAME].\u0027)"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--region_name\u0027,"},{"line_number":280,"context_line":"                            help\u003dargparse.SUPPRESS)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_3e02ee3d","line":277,"updated":"2015-04-09 16:02:08.000000000","message":"It looks like cliutils.env returns the first set var. As SAHARA_REGION_NAME is more specific that OS_REGION_NAME, should we place that arg first and document the fallthrough (or just remove that arg and only use OS_REGION_NAME?)\n\n(By the way, if this is a backward-compatibility issue with a prior set of configurations, this is utterly reasonable.)","commit_id":"a495addf90e6c5a568ca59f64fac0314756b21b2"},{"author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"change_message_id":"22bca82052b3c3ad76bed8ae5089276696db7c25","unresolved":false,"context_lines":[{"line_number":274,"context_line":"        parser.add_argument(\u0027--region-name\u0027,"},{"line_number":275,"context_line":"                            metavar\u003d\u0027\u003cregion-name\u003e\u0027,"},{"line_number":276,"context_line":"                            default\u003dcliutils.env(\u0027OS_REGION_NAME\u0027,"},{"line_number":277,"context_line":"                                                 \u0027SAHARA_REGION_NAME\u0027),"},{"line_number":278,"context_line":"                            help\u003d\u0027Defaults to env[OS_REGION_NAME].\u0027)"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--region_name\u0027,"},{"line_number":280,"context_line":"                            help\u003dargparse.SUPPRESS)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_c0ca576e","line":277,"in_reply_to":"9aa53dc9_3e02ee3d","updated":"2015-04-14 07:30:18.000000000","message":"Done","commit_id":"a495addf90e6c5a568ca59f64fac0314756b21b2"}]}