)]}'
{"playbooks/roles/start_fresh_logging/defaults/main.yaml":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"b86086ae22031bb7c3ca5303e019bb88f9ed5e44","unresolved":false,"context_lines":[{"line_number":1,"context_line":"BASE: \"{{ lookup(\u0027env\u0027, \u0027BASE\u0027)|default(\u0027BASE\u0027) }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5a74a57a_718938ca","line":1,"updated":"2016-11-30 18:28:33.000000000","message":"This looks like a copy-pasta issue?","commit_id":"0ced3f903ef9cd4da0fddc2fb627182039b70ab8"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"85cf2ae45dc2cac7f8646af1845c3c3c38ae871d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"BASE: \"{{ lookup(\u0027env\u0027, \u0027BASE\u0027)|default(\u0027BASE\u0027) }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5a74a57a_b24e6ebf","line":1,"in_reply_to":"5a74a57a_718938ca","updated":"2016-11-30 19:43:11.000000000","message":"Done","commit_id":"0ced3f903ef9cd4da0fddc2fb627182039b70ab8"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"04976bfcfeac28ea8f3f05fab641f83fd1563f62","unresolved":false,"context_lines":[{"line_number":1,"context_line":"BASE: \"{{ lookup(\u0027env\u0027, \u0027BASE\u0027)|default(\u0027/opt/stack\u0027) }}\""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"5a74a57a_bce7203f","line":1,"updated":"2016-11-30 21:00:41.000000000","message":"To prevent ending up with multiple BASE values across roles (the previous role uses BASE too), can we set this more globally as a playbook input?\n\nNot sure how to reconcile this against the idea that these roles should be independently useful though. Open to thoughts on that. Maybe we have a hard dep on a thing that sets the necessary vars.","commit_id":"4155294bd3f50e9e8ac82198e14c38eefa81d285"}],"playbooks/roles/start_fresh_logging/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"04976bfcfeac28ea8f3f05fab641f83fd1563f62","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for /bin/journalctl file"},{"line_number":3,"context_line":"  stat: path\u003d/bin/journalctl"},{"line_number":4,"context_line":"  register: st"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- block:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"5a74a57a_7c4bc82b","line":3,"updated":"2016-11-30 21:00:41.000000000","message":"journalctl doesn\u0027t have to live in /bin (it doesn\u0027t on my local machine for example). This should ideally use which which will check against $PATH.","commit_id":"4155294bd3f50e9e8ac82198e14c38eefa81d285"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"04976bfcfeac28ea8f3f05fab641f83fd1563f62","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      command: date +\"%Y-%m-%d %H:%M:%S\""},{"line_number":9,"context_line":"      register: date_out"},{"line_number":10,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":11,"context_line":"      lineinfile: dest\u003d{{ BASE }}/log-start-timestamp.txt insertafter\u003dEOF line\u003d{{ date_out.stdout }} create\u003dyes"},{"line_number":12,"context_line":"  when: st.stat.exists"},{"line_number":13,"context_line":"  become: yes"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"5a74a57a_5c128c0a","line":11,"updated":"2016-11-30 21:00:41.000000000","message":"This will append if the file already exists, which is not what the old behavior is. The old behavior will overwrite the file so that it only ever has the one timestamp in it. We should continue to have this behavior.","commit_id":"4155294bd3f50e9e8ac82198e14c38eefa81d285"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"211e8eec1c3c7aca1e8b162673281285ec25409d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for /bin/journalctl file"},{"line_number":3,"context_line":"  stat: path\u003d/bin/journalctl"},{"line_number":4,"context_line":"  register: st"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- block:"},{"line_number":7,"context_line":"    - name: Get current date"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"1a6eadb0_46c4b791","line":4,"updated":"2016-12-12 20:04:56.000000000","message":"This still doesn\u0027t address my original concern that journalctl may not be located in /bin/journalctl.\n\n  clark@nibbler:~$ which journalctl\n  /usr/bin/journalctl\n\nThis is a regression compared to the old code which used `which`.","commit_id":"942a4591f836f159f41d1261d02d3c82a3cf6711"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"211e8eec1c3c7aca1e8b162673281285ec25409d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  become: yes"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- block:"},{"line_number":16,"context_line":"    - name: Stop rsyslog"},{"line_number":17,"context_line":"      service: name\u003drsyslog state\u003dstopped"},{"line_number":18,"context_line":"    - name: Save syslog file prior to devstack run"},{"line_number":19,"context_line":"      command: mv /var/log/syslog /var/log/syslog-pre-devstack"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"1a6eadb0_c60447c8","line":16,"updated":"2016-12-12 20:04:56.000000000","message":"We might want to consider forcing ansible to run this playbook to completion for each host to reduce the total amount of time rsyslog is not running on each host. This may be premature optimization though the old setup would\u0027ve run this way on multinode do to how we invoked ansible.\n\nThis is not why I -1\u0027d, mostly something to think about.","commit_id":"942a4591f836f159f41d1261d02d3c82a3cf6711"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"115c896a1c50e047eaff48d41d49c00d925dd86d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  become: yes"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- block:"},{"line_number":16,"context_line":"    - name: Stop rsyslog"},{"line_number":17,"context_line":"      service: name\u003drsyslog state\u003dstopped"},{"line_number":18,"context_line":"    - name: Save syslog file prior to devstack run"},{"line_number":19,"context_line":"      command: mv /var/log/syslog /var/log/syslog-pre-devstack"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"1a6eadb0_08e2c9d5","line":16,"in_reply_to":"1a6eadb0_c60447c8","updated":"2016-12-12 21:40:50.000000000","message":"Yeah, I rather replace all bash for ansible to have feature parity and leave optimizations when we migrate all the code.","commit_id":"942a4591f836f159f41d1261d02d3c82a3cf6711"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"cce5306fc246477f5117a1b73d9fafc099b126b2","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- block:"},{"line_number":9,"context_line":"    - name: Get current date"},{"line_number":10,"context_line":"      command: date +\"%Y-%m-%d %H:%M:%S\""},{"line_number":11,"context_line":"      register: date_out"},{"line_number":12,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":13,"context_line":"      lineinfile: dest\u003d{{ BASE }}/log-start-timestamp.txt insertafter\u003dEOF line\u003d{{ date_out.stdout }} create\u003dyes"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_f4bac4bf","line":10,"updated":"2016-12-12 23:46:40.000000000","message":"Does this particular format matter? There is a built in fact we could use,\n\nset_fact:\n  date_out: \"{{ ansible_date_time.date }} {{ ansible_date_time.time }}\"\n\nOr if the format isn\u0027t critical, {{ ansible_date_time.iso8601 }}","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"7cc1d5387de31ceae0249251557e0150a8646087","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- block:"},{"line_number":9,"context_line":"    - name: Get current date"},{"line_number":10,"context_line":"      command: date +\"%Y-%m-%d %H:%M:%S\""},{"line_number":11,"context_line":"      register: date_out"},{"line_number":12,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":13,"context_line":"      lineinfile: dest\u003d{{ BASE }}/log-start-timestamp.txt insertafter\u003dEOF line\u003d{{ date_out.stdout }} create\u003dyes"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_52044cbf","line":10,"in_reply_to":"1a6eadb0_f4bac4bf","updated":"2016-12-13 13:31:09.000000000","message":"We are currently getting feature parity to what we have in BASH.\nEventual optimization or change on format could be looked at when we land this change.","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"cce5306fc246477f5117a1b73d9fafc099b126b2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"      command: date +\"%Y-%m-%d %H:%M:%S\""},{"line_number":11,"context_line":"      register: date_out"},{"line_number":12,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":13,"context_line":"      lineinfile: dest\u003d{{ BASE }}/log-start-timestamp.txt insertafter\u003dEOF line\u003d{{ date_out.stdout }} create\u003dyes"},{"line_number":14,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":15,"context_line":"  become: yes"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_b4c12c31","line":13,"updated":"2016-12-12 23:46:40.000000000","message":"Easier to read if less than 80 chars long.","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"7cc1d5387de31ceae0249251557e0150a8646087","unresolved":false,"context_lines":[{"line_number":10,"context_line":"      command: date +\"%Y-%m-%d %H:%M:%S\""},{"line_number":11,"context_line":"      register: date_out"},{"line_number":12,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":13,"context_line":"      lineinfile: dest\u003d{{ BASE }}/log-start-timestamp.txt insertafter\u003dEOF line\u003d{{ date_out.stdout }} create\u003dyes"},{"line_number":14,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":15,"context_line":"  become: yes"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_b23120a5","line":13,"in_reply_to":"1a6eadb0_b4c12c31","updated":"2016-12-13 13:31:09.000000000","message":"Done","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"cce5306fc246477f5117a1b73d9fafc099b126b2","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    - name: Save kern.log file prior to devstack run"},{"line_number":23,"context_line":"      command: mv /var/log/kern.log /var/log/kern_log-pre-devstack"},{"line_number":24,"context_line":"    - name: Recreate syslog file"},{"line_number":25,"context_line":"      command: touch /var/log/syslog"},{"line_number":26,"context_line":"    - name: Recreate syslog file owner and group"},{"line_number":27,"context_line":"      command: chown /var/log/syslog --ref /var/log/syslog-pre-devstack"},{"line_number":28,"context_line":"    - name: Recreate syslog file permissions"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_f4080494","line":25,"updated":"2016-12-12 23:46:40.000000000","message":"file: path\u003d/var/log/syslog state\u003dtouch","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"7cc1d5387de31ceae0249251557e0150a8646087","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    - name: Save kern.log file prior to devstack run"},{"line_number":23,"context_line":"      command: mv /var/log/kern.log /var/log/kern_log-pre-devstack"},{"line_number":24,"context_line":"    - name: Recreate syslog file"},{"line_number":25,"context_line":"      command: touch /var/log/syslog"},{"line_number":26,"context_line":"    - name: Recreate syslog file owner and group"},{"line_number":27,"context_line":"      command: chown /var/log/syslog --ref /var/log/syslog-pre-devstack"},{"line_number":28,"context_line":"    - name: Recreate syslog file permissions"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_d2269c67","line":25,"in_reply_to":"1a6eadb0_f4080494","updated":"2016-12-13 13:31:09.000000000","message":"Done","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"cce5306fc246477f5117a1b73d9fafc099b126b2","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    - name: Recreate syslog file permissions"},{"line_number":29,"context_line":"      command: chmod /var/log/syslog --ref /var/log/syslog-pre-devstack"},{"line_number":30,"context_line":"    - name: Add read permissions to all on syslog file"},{"line_number":31,"context_line":"      command: chmod a+r /var/log/syslog"},{"line_number":32,"context_line":"    - name: Recreate kern.log file"},{"line_number":33,"context_line":"      command: touch /var/log/kern.log"},{"line_number":34,"context_line":"    - name: Recreate kern.log file owner and group"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_74231406","line":31,"updated":"2016-12-12 23:46:40.000000000","message":"This could be a file module call to set permissions with a recursive: true argument.","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"7cc1d5387de31ceae0249251557e0150a8646087","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    - name: Recreate syslog file permissions"},{"line_number":29,"context_line":"      command: chmod /var/log/syslog --ref /var/log/syslog-pre-devstack"},{"line_number":30,"context_line":"    - name: Add read permissions to all on syslog file"},{"line_number":31,"context_line":"      command: chmod a+r /var/log/syslog"},{"line_number":32,"context_line":"    - name: Recreate kern.log file"},{"line_number":33,"context_line":"      command: touch /var/log/kern.log"},{"line_number":34,"context_line":"    - name: Recreate kern.log file owner and group"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_1221f44d","line":31,"in_reply_to":"1a6eadb0_74231406","updated":"2016-12-13 13:31:09.000000000","message":"Done","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"cce5306fc246477f5117a1b73d9fafc099b126b2","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    - name: Add read permissions to all on syslog file"},{"line_number":31,"context_line":"      command: chmod a+r /var/log/syslog"},{"line_number":32,"context_line":"    - name: Recreate kern.log file"},{"line_number":33,"context_line":"      command: touch /var/log/kern.log"},{"line_number":34,"context_line":"    - name: Recreate kern.log file owner and group"},{"line_number":35,"context_line":"      command: chown /var/log/kern.log --ref /var/log/kern_log-pre-devstack"},{"line_number":36,"context_line":"    - name: Recreate kern.log file permissions"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_9420b0f9","line":33,"updated":"2016-12-12 23:46:40.000000000","message":"file: state\u003dtouch","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"7cc1d5387de31ceae0249251557e0150a8646087","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    - name: Add read permissions to all on syslog file"},{"line_number":31,"context_line":"      command: chmod a+r /var/log/syslog"},{"line_number":32,"context_line":"    - name: Recreate kern.log file"},{"line_number":33,"context_line":"      command: touch /var/log/kern.log"},{"line_number":34,"context_line":"    - name: Recreate kern.log file owner and group"},{"line_number":35,"context_line":"      command: chown /var/log/kern.log --ref /var/log/kern_log-pre-devstack"},{"line_number":36,"context_line":"    - name: Recreate kern.log file permissions"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_321ef00c","line":33,"in_reply_to":"1a6eadb0_9420b0f9","updated":"2016-12-13 13:31:09.000000000","message":"Done","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"cce5306fc246477f5117a1b73d9fafc099b126b2","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    - name: Recreate kern.log file permissions"},{"line_number":37,"context_line":"      command: chmod /var/log/kern.log --ref /var/log/kern_log-pre-devstack"},{"line_number":38,"context_line":"    - name: Add read permissions to all on kern.log file"},{"line_number":39,"context_line":"      command: chmod a+r /var/log/kern.log"},{"line_number":40,"context_line":"    - name: Start rsyslog"},{"line_number":41,"context_line":"      service: name\u003drsyslog state\u003dstarted"},{"line_number":42,"context_line":"  when: which_out.rc \u003d\u003d 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_f47d64e8","line":39,"updated":"2016-12-12 23:46:40.000000000","message":"This could be file module as well.","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"7cc1d5387de31ceae0249251557e0150a8646087","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    - name: Recreate kern.log file permissions"},{"line_number":37,"context_line":"      command: chmod /var/log/kern.log --ref /var/log/kern_log-pre-devstack"},{"line_number":38,"context_line":"    - name: Add read permissions to all on kern.log file"},{"line_number":39,"context_line":"      command: chmod a+r /var/log/kern.log"},{"line_number":40,"context_line":"    - name: Start rsyslog"},{"line_number":41,"context_line":"      service: name\u003drsyslog state\u003dstarted"},{"line_number":42,"context_line":"  when: which_out.rc \u003d\u003d 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"1a6eadb0_92d34438","line":39,"in_reply_to":"1a6eadb0_f47d64e8","updated":"2016-12-13 13:31:09.000000000","message":"Done","commit_id":"6eec619e814e7174b292134a4887f073f2916592"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"ad7f6274322195e83b2953dc53f4d7913f4a61fc","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":14,"context_line":"      lineinfile:"},{"line_number":15,"context_line":"        dest\u003d{{ BASE }}/log-start-timestamp.txt"},{"line_number":16,"context_line":"        insertafter\u003dEOF"},{"line_number":17,"context_line":"        line\u003d{{ date_out.stdout }}"},{"line_number":18,"context_line":"        create\u003dyes"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"1a6eadb0_ef9ba1c7","line":15,"updated":"2016-12-13 18:42:07.000000000","message":"This looks really close to the full YAML syntax, but it isn\u0027t quite, which is confusing.\n\n  dest: \"{{ BASE }}/log-start-timestamp.txt\"\n  insertafter: EOF\n\nand so on.","commit_id":"58ef229f34a0cfe6e4139587154c6f9853695f97"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"0781c5f7eb69dc06756cd174d40c220c6a8a6600","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"    - name: Append current date to log-start-timestamp.txt"},{"line_number":14,"context_line":"      lineinfile:"},{"line_number":15,"context_line":"        dest\u003d{{ BASE }}/log-start-timestamp.txt"},{"line_number":16,"context_line":"        insertafter\u003dEOF"},{"line_number":17,"context_line":"        line\u003d{{ date_out.stdout }}"},{"line_number":18,"context_line":"        create\u003dyes"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"1a6eadb0_4a864314","line":15,"in_reply_to":"1a6eadb0_ef9ba1c7","updated":"2016-12-13 19:07:09.000000000","message":"Done","commit_id":"58ef229f34a0cfe6e4139587154c6f9853695f97"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6404d0d34934371f280a26bd2eddd896c41d76a1","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        dest: \"{{ BASE }}/log-start-timestamp.txt\""},{"line_number":16,"context_line":"        insertafter: EOF"},{"line_number":17,"context_line":"        line: \"{{ date_out.stdout }}\""},{"line_number":18,"context_line":"        create: yes"},{"line_number":19,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":20,"context_line":"  become: yes"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"1a6eadb0_4945686e","line":18,"updated":"2016-12-14 00:04:57.000000000","message":"Sorry, I caught this back in patchset 10 but it got lost in all the noise since then. This will append to a file if run multiple times. The old code truncated and then added to the file. My reading of the code is that only a single timestamp in the file will be understood. I think we need to preserve that behavior as at least some people run devstack/devstack-gate on the same host more than once.\n\nDoesn\u0027t look like there is a way to tell lineinfile to truncate so I think we want to use the copy module with content set to {{ date_out.stdout }}.","commit_id":"dc65ad7955c8738c8e82e66d19774f929914936d"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6404d0d34934371f280a26bd2eddd896c41d76a1","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        dest: \"{{ BASE }}/log-start-timestamp.txt\""},{"line_number":16,"context_line":"        insertafter: EOF"},{"line_number":17,"context_line":"        line: \"{{ date_out.stdout }}\""},{"line_number":18,"context_line":"        create: yes"},{"line_number":19,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":20,"context_line":"  become: yes"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"1a6eadb0_49a00844","line":18,"updated":"2016-12-14 00:04:57.000000000","message":"Sorry, I caught this back in patchset 10 but it got lost in all the noise since then. This will append to a file if run multiple times. The old code truncated and then added to the file. My reading of the code is that only a single timestamp in the file will be understood. I think we need to preserve that behavior as at least some people run devstack/devstack-gate on the same host more than once.\n\nDoesn\u0027t look like there is a way to tell lineinfile to truncate so I think we want to use the copy module with content set to {{ date_out.stdout }}.","commit_id":"dc65ad7955c8738c8e82e66d19774f929914936d"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6404d0d34934371f280a26bd2eddd896c41d76a1","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        dest: \"{{ BASE }}/log-start-timestamp.txt\""},{"line_number":16,"context_line":"        insertafter: EOF"},{"line_number":17,"context_line":"        line: \"{{ date_out.stdout }}\""},{"line_number":18,"context_line":"        create: yes"},{"line_number":19,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":20,"context_line":"  become: yes"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"1a6eadb0_c9fdf83b","line":18,"updated":"2016-12-14 00:04:57.000000000","message":"Sorry, I caught this back in patchset 10 but it got lost in all the noise since then. This will append to a file if run multiple times. The old code truncated and then added to the file. My reading of the code is that only a single timestamp in the file will be understood. I think we need to preserve that behavior as at least some people run devstack/devstack-gate on the same host more than once.\n\nDoesn\u0027t look like there is a way to tell lineinfile to truncate so I think we want to use the copy module with content set to {{ date_out.stdout }}.","commit_id":"dc65ad7955c8738c8e82e66d19774f929914936d"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"f3dd05907ef061c590ed93f1ebad295e86c445a4","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        dest: \"{{ BASE }}/log-start-timestamp.txt\""},{"line_number":16,"context_line":"        insertafter: EOF"},{"line_number":17,"context_line":"        line: \"{{ date_out.stdout }}\""},{"line_number":18,"context_line":"        create: yes"},{"line_number":19,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":20,"context_line":"  become: yes"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"1a6eadb0_2122e900","line":18,"in_reply_to":"1a6eadb0_4945686e","updated":"2016-12-14 12:48:12.000000000","message":"gah, apologies, I missed it due to the rebase churn, pushing new patchset","commit_id":"dc65ad7955c8738c8e82e66d19774f929914936d"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3d535ff3f904387ff3c37c913594bc48b479b344","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        dest: \"{{ BASE }}/log-start-timestamp.txt\""},{"line_number":16,"context_line":"        insertafter: EOF"},{"line_number":17,"context_line":"        line: \"{{ date_out.stdout }}\""},{"line_number":18,"context_line":"        create: yes"},{"line_number":19,"context_line":"  when: which_out.rc \u003d\u003d 0"},{"line_number":20,"context_line":"  become: yes"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"1a6eadb0_291cac76","line":18,"in_reply_to":"1a6eadb0_c9fdf83b","updated":"2016-12-14 00:05:29.000000000","message":"Apparently when you try to comment in the middle of gerrit db backups this happens. Sorry for the noise but the comment is valid :)","commit_id":"dc65ad7955c8738c8e82e66d19774f929914936d"}]}