)]}'
{"roles/baremetal-undercloud/tasks/machine-provisioning.yml":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"2a2a38bbee1695bc2c18f5d6f52efe2cfa0466e6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Provision script execution"},{"line_number":2,"context_line":"  shell: \u003e"},{"line_number":3,"context_line":"    set -o pipefail \u0026\u0026"},{"line_number":4,"context_line":"    {{ baremetal_provisioning_script }} 2\u003e\u00261 {{ timestamper_cmd }} |\u0026 tee"},{"line_number":5,"context_line":"    {{ baremetal_provisioning_log }}"},{"line_number":6,"context_line":"  when: step_provide_undercloud|bool"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_97140350","line":4,"range":{"start_line":4,"start_character":68,"end_line":4,"end_character":69},"updated":"2017-08-24 14:13:42.000000000","message":"why is this \"\u0026\"?","commit_id":"e1adb71acc40c9ef4ec106ba1f92329e8ca8c0b9"},{"author":{"_account_id":20936,"name":"Raoul Scarazzini","email":"rasca@mmul.it","username":"rasca"},"change_message_id":"3247ff51eb959128f3a3199cef44448572e10356","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Provision script execution"},{"line_number":2,"context_line":"  shell: \u003e"},{"line_number":3,"context_line":"    set -o pipefail \u0026\u0026"},{"line_number":4,"context_line":"    {{ baremetal_provisioning_script }} 2\u003e\u00261 {{ timestamper_cmd }} |\u0026 tee"},{"line_number":5,"context_line":"    {{ baremetal_provisioning_log }}"},{"line_number":6,"context_line":"  when: step_provide_undercloud|bool"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_546336b2","line":4,"range":{"start_line":4,"start_character":68,"end_line":4,"end_character":69},"in_reply_to":"7f287b81_97140350","updated":"2017-08-24 14:46:52.000000000","message":"It pipes to \"tee\" also the standard error, otherwise you\u0027ll get just stdout.\nBut in this case, since we\u0027re already using 2\u003e\u00261 we are fine and this can be omitted.\nIn addition since timestamper_cmd does not have |\u0026 in its definition, without 2\u003e\u00261 we would not see the standard error from baremetal_provision_script in any case, so I think that being coherent here is good, so I\u0027m going to remove the \u0026 from the pipe.","commit_id":"e1adb71acc40c9ef4ec106ba1f92329e8ca8c0b9"}]}