)]}'
{"rally/cli/commands/deployment.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"3c5e6b7d54bf290c7b556f1adc060b43240d2929","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                    \"password\": os.environ[\"OS_PASSWORD\"],"},{"line_number":106,"context_line":"                    \"tenant_name\": os.environ[\"OS_TENANT_NAME\"]"},{"line_number":107,"context_line":"                },"},{"line_number":108,"context_line":"                \"https_cacert\": os.environ.get(\"OS_CACERT\", \"\")"},{"line_number":109,"context_line":"            }"},{"line_number":110,"context_line":"            region_name \u003d os.environ.get(\"OS_REGION_NAME\")"},{"line_number":111,"context_line":"            if region_name and region_name !\u003d \"None\":"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_1b9b07cb","line":108,"updated":"2015-12-10 12:52:17.000000000","message":"you should add \"insecure\" var too.","commit_id":"10a99e70820c04f599d09553b5b757128569eb15"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"99a8d8c157602c875393e6354d805a5510b2533a","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"            config \u003d {"},{"line_number":101,"context_line":"                \"type\": \"ExistingCloud\","},{"line_number":102,"context_line":"                \"auth_url\": os.environ[\"OS_AUTH_URL\"],"},{"line_number":103,"context_line":"                \"endpoint\": os.environ.get(\"OS_ENDPOINT\"),"},{"line_number":104,"context_line":"                \"admin\": {"},{"line_number":105,"context_line":"                    \"username\": os.environ[\"OS_USERNAME\"],"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_eac64e15","line":102,"updated":"2015-12-10 18:26:28.000000000","message":"can you move all this os.environment stuff to separate method? so it can be used in both \"deployment\" and \"osclients\".","commit_id":"043b2ae214474b2284fb743107956d2121cae9e6"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"748ae643ecae1436fc722877b268ce692ef61e4e","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"            config \u003d {"},{"line_number":101,"context_line":"                \"type\": \"ExistingCloud\","},{"line_number":102,"context_line":"                \"auth_url\": os.environ[\"OS_AUTH_URL\"],"},{"line_number":103,"context_line":"                \"endpoint\": os.environ.get(\"OS_ENDPOINT\"),"},{"line_number":104,"context_line":"                \"admin\": {"},{"line_number":105,"context_line":"                    \"username\": os.environ[\"OS_USERNAME\"],"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_20e79541","line":102,"in_reply_to":"7a740942_85c53b3e","updated":"2015-12-10 18:51:46.000000000","message":"\u003eYes, but where should this method be located?\n\nrally.commands.cliutils looks like a good place:)\n\n\u003eProbably we should create a separate patch depending on this one. What do you think about it?\n\nYes, it can be done in the separate patch, but I\u0027ll nit review this one, until I saw that separate patch exists:)","commit_id":"043b2ae214474b2284fb743107956d2121cae9e6"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"d610cf56d4cac94368f1b26b2e4c953aead50674","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"            config \u003d {"},{"line_number":101,"context_line":"                \"type\": \"ExistingCloud\","},{"line_number":102,"context_line":"                \"auth_url\": os.environ[\"OS_AUTH_URL\"],"},{"line_number":103,"context_line":"                \"endpoint\": os.environ.get(\"OS_ENDPOINT\"),"},{"line_number":104,"context_line":"                \"admin\": {"},{"line_number":105,"context_line":"                    \"username\": os.environ[\"OS_USERNAME\"],"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_2577a703","line":102,"in_reply_to":"7a740942_85c53b3e","updated":"2015-12-10 18:40:46.000000000","message":"Probably we should create a separate patch depending on this one. What do you think about it?","commit_id":"043b2ae214474b2284fb743107956d2121cae9e6"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"872f48adc48974678e9c37cbf262b21fdcb142c8","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"            config \u003d {"},{"line_number":101,"context_line":"                \"type\": \"ExistingCloud\","},{"line_number":102,"context_line":"                \"auth_url\": os.environ[\"OS_AUTH_URL\"],"},{"line_number":103,"context_line":"                \"endpoint\": os.environ.get(\"OS_ENDPOINT\"),"},{"line_number":104,"context_line":"                \"admin\": {"},{"line_number":105,"context_line":"                    \"username\": os.environ[\"OS_USERNAME\"],"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_85c53b3e","line":102,"in_reply_to":"7a740942_eac64e15","updated":"2015-12-10 18:38:39.000000000","message":"Yes, but where should this method be located?","commit_id":"043b2ae214474b2284fb743107956d2121cae9e6"}]}