)]}'
{"tripleoclient/v1/tripleo_deploy.py":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"cc85c635bce746c7440f6688229f7760938e90f5","unresolved":false,"context_lines":[{"line_number":1323,"context_line":"                                \u0027inventory.yaml\u0027"},{"line_number":1324,"context_line":"                            ),"},{"line_number":1325,"context_line":"                            workdir\u003dself.ansible_dir,"},{"line_number":1326,"context_line":"                            extra_env_variables\u003dextra_env_var,"},{"line_number":1327,"context_line":"                            **operation"},{"line_number":1328,"context_line":"                        )[0]"},{"line_number":1329,"context_line":"        except Exception as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_2b7ae1ba","line":1326,"range":{"start_line":1326,"start_character":27,"end_line":1326,"end_character":62},"updated":"2020-01-14 15:51:19.000000000","message":"Did we try using the set_logfile()[1] interface with ansible_runner? No tested it, just commenting looking at code.\n\nSomething like:\n\nfrom ansible_runner import output\n\noutput.configure()\noutput.set_logfile(\u003cwhatever file\u003e)\n\n[1] https://github.com/ansible/ansible-runner/blob/master/ansible_runner/output.py#L43","commit_id":"585e41d6b3191accd704f3c750ddf0dca81641e3"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"a5df9de172a9364b0b9afe6425e574259b9f5f54","unresolved":false,"context_lines":[{"line_number":1323,"context_line":"                                \u0027inventory.yaml\u0027"},{"line_number":1324,"context_line":"                            ),"},{"line_number":1325,"context_line":"                            workdir\u003dself.ansible_dir,"},{"line_number":1326,"context_line":"                            extra_env_variables\u003dextra_env_var,"},{"line_number":1327,"context_line":"                            **operation"},{"line_number":1328,"context_line":"                        )[0]"},{"line_number":1329,"context_line":"        except Exception as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_0059de27","line":1326,"range":{"start_line":1326,"start_character":27,"end_line":1326,"end_character":62},"in_reply_to":"3fa7e38b_0f27db65","updated":"2020-01-16 04:33:22.000000000","message":"OK, it\u0027s better to use available api with \u0027python libraries\u0027, rather than configuring things with environment variables generally.","commit_id":"585e41d6b3191accd704f3c750ddf0dca81641e3"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"c6489e3747b5d63ab543fe0728da890a32125904","unresolved":false,"context_lines":[{"line_number":1323,"context_line":"                                \u0027inventory.yaml\u0027"},{"line_number":1324,"context_line":"                            ),"},{"line_number":1325,"context_line":"                            workdir\u003dself.ansible_dir,"},{"line_number":1326,"context_line":"                            extra_env_variables\u003dextra_env_var,"},{"line_number":1327,"context_line":"                            **operation"},{"line_number":1328,"context_line":"                        )[0]"},{"line_number":1329,"context_line":"        except Exception as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_5dd11156","line":1326,"range":{"start_line":1326,"start_character":27,"end_line":1326,"end_character":62},"in_reply_to":"3fa7e38b_2b7ae1ba","updated":"2020-01-15 06:57:48.000000000","message":"ah, didn\u0027t see that. Not sure if it will help, since the ANSIBLE_LOG_PATH is exported within the wrapper. Maybe Kevin can comment as to why it was done like that when he\u0027s back from his PTO :/","commit_id":"585e41d6b3191accd704f3c750ddf0dca81641e3"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"2ff75befdb6df7cd41734272342313ff9921910f","unresolved":false,"context_lines":[{"line_number":1323,"context_line":"                                \u0027inventory.yaml\u0027"},{"line_number":1324,"context_line":"                            ),"},{"line_number":1325,"context_line":"                            workdir\u003dself.ansible_dir,"},{"line_number":1326,"context_line":"                            extra_env_variables\u003dextra_env_var,"},{"line_number":1327,"context_line":"                            **operation"},{"line_number":1328,"context_line":"                        )[0]"},{"line_number":1329,"context_line":"        except Exception as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_0f27db65","line":1326,"range":{"start_line":1326,"start_character":27,"end_line":1326,"end_character":62},"in_reply_to":"3fa7e38b_5dd11156","updated":"2020-01-15 13:53:05.000000000","message":"Cédric ++ \n\nWhile ansible runner has a couple interfaces for configuring ansible itself, to make things consistent, I elected to run all of the configuration through environment variables, which will ensure that we\u0027re not running into any nuance when utilizing the ansible-runner lib.","commit_id":"585e41d6b3191accd704f3c750ddf0dca81641e3"}]}