)]}'
{"scripts/tripleo-ansible-inventory":[{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"32bcce62c36a7345b074d3e07cbd610e764f6b15","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        ansible_ssh_user\u003dconfigs.ansible_ssh_user,"},{"line_number":142,"context_line":"        plan_name\u003dconfigs.stack or configs.plan,"},{"line_number":143,"context_line":"        ansible_python_interpreter\u003dconfigs.ansible_python_interpreter,"},{"line_number":144,"context_line":"        undercloud_connection\u003dconfigs.undercloud_connection,"},{"line_number":145,"context_line":"        undercloud_key_file\u003dconfigs.undercloud_key_file,"},{"line_number":146,"context_line":"        serial\u003dconfigs.serial)"},{"line_number":147,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"7faddb67_5138353a","line":144,"updated":"2019-08-23 11:47:17.000000000","message":"I think we should add that modification from https://review.opendev.org/#/c/653937 into\nthat review and call that one \"add missing parameter in tripleo-ansilble-inventory\"\n\nThe goal here is to be able to trigger the same call as in deployment https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/ansible.py#L675-L687 which is triggered from that mistral flow: https://github.com/openstack/tripleo-common/blob/master/workbooks/deployment.yaml#L492-L500","commit_id":"141d6e3a559ba4a544c0c30e5fc29de62bbe87cd"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"c5d8c7f8d75165da845e06e91c97134902cfb04c","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        ansible_python_interpreter\u003dconfigs.ansible_python_interpreter,"},{"line_number":144,"context_line":"        undercloud_connection\u003dconfigs.undercloud_connection,"},{"line_number":145,"context_line":"        undercloud_key_file\u003dconfigs.undercloud_key_file,"},{"line_number":146,"context_line":"        serial\u003dconfigs.serial)"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"    if configs.list:"},{"line_number":149,"context_line":"        try:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"7faddb67_7110d15a","line":146,"updated":"2019-08-23 12:06:05.000000000","message":"Note that this call also miss the host_network parameter as seen there https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/ansible.py#L687\n\nWe should add it as well to make sure we are on par with deployment.","commit_id":"141d6e3a559ba4a544c0c30e5fc29de62bbe87cd"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"c8e998f3c0956e33e6afaef2186de1b93a3e101b","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        ansible_ssh_user\u003dconfigs.ansible_ssh_user,"},{"line_number":141,"context_line":"        plan_name\u003dconfigs.stack or configs.plan,"},{"line_number":142,"context_line":"        ansible_python_interpreter\u003dconfigs.ansible_python_interpreter,"},{"line_number":143,"context_line":"        undercloud_connection\u003dconfigs.undercloud_connection,"},{"line_number":144,"context_line":"        undercloud_key_file\u003dconfigs.undercloud_key_file,"},{"line_number":145,"context_line":"        serial\u003dconfigs.serial)"},{"line_number":146,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"7faddb67_6718f61d","line":143,"updated":"2019-08-26 10:51:41.000000000","message":"I still would like to have the host_network parameter added here to match what is done during deployment call https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/ansible.py#L687","commit_id":"0edf47317871c065df7a6380ca5ab4b5ad7f15db"}]}