)]}'
{"test-log-collect.sh":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"11466f34751dcdc51017f5fe9e7dae39cf43af7f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"  # NOTE(mhayden): We use sudo here to ensure that all logs are copied."},{"line_number":43,"context_line":"  sudo ${RSYNC_CMD} /var/log/ \"${WORKING_DIR}/logs/host\" || true"},{"line_number":44,"context_line":"  sudo ${RSYNC_CMD} /openstack/log/ \"${WORKING_DIR}/logs/openstack\" || true"},{"line_number":45,"context_line":"  sudo ${RSYNC_CMD} /etc/ \"${WORKING_DIR}/etc/host\" || true"},{"line_number":46,"context_line":"  sudo ${RSYNC_CMD} /openstack/etc/ \"${WORKING_DIR}/etc/openstack\" || true"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"  # NOTE(mhayden): All of the logs must be world-readable so that the log"}],"source_content_type":"text/x-sh","patch_set":2,"id":"9f436f4f_cffd7e09","line":45,"updated":"2017-08-18 10:19:49.000000000","message":"I\u0027d be super happy if you could add /usr/local/bin/openstack-ansible.rc file too.","commit_id":"e64ffe7d6c08b3bb00cae7d5ae09a4007313b837"},{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"f334781343c6cbcb558f3b27aa9f13b1f0dbff73","unresolved":false,"context_lines":[{"line_number":48,"context_line":"  sudo ${RSYNC_ETC_CMD} /etc/ \"${WORKING_DIR}/logs/etc/host/\" || true"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"  # Loop over each container and archive its /etc directory"},{"line_number":51,"context_line":"  for CONTAINER_NAME in `sudo lxc-ls`; do"},{"line_number":52,"context_line":"    CONTAINER_PID\u003d$(sudo lxc-info -p -n ${CONTAINER_NAME} | awk \u0027{print $2}\u0027)"},{"line_number":53,"context_line":"    ETC_DIR\u003d\"/proc/${CONTAINER_PID}/root/etc/\""},{"line_number":54,"context_line":"    sudo ${RSYNC_ETC_CMD} ${ETC_DIR} \"${WORKING_DIR}/logs/etc/openstack/${CONTAINER_NAME}/\" || true"}],"source_content_type":"text/x-sh","patch_set":8,"id":"7f287b81_09f5dacb","line":51,"range":{"start_line":51,"start_character":29,"end_line":51,"end_character":36},"updated":"2017-08-22 15:10:39.000000000","message":"I wonder if this should be \u0027lxc-ls -1\u0027 so you get one container per line. Does the current code work as expected?","commit_id":"8589ff201ffcd513cad99d022cbfe9f99983f3b7"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"5db17c4532ffa0dc634b388bc18da703e0a61ca8","unresolved":false,"context_lines":[{"line_number":48,"context_line":"  sudo ${RSYNC_ETC_CMD} /etc/ \"${WORKING_DIR}/logs/etc/host/\" || true"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"  # Loop over each container and archive its /etc directory"},{"line_number":51,"context_line":"  for CONTAINER_NAME in `sudo lxc-ls`; do"},{"line_number":52,"context_line":"    CONTAINER_PID\u003d$(sudo lxc-info -p -n ${CONTAINER_NAME} | awk \u0027{print $2}\u0027)"},{"line_number":53,"context_line":"    ETC_DIR\u003d\"/proc/${CONTAINER_PID}/root/etc/\""},{"line_number":54,"context_line":"    sudo ${RSYNC_ETC_CMD} ${ETC_DIR} \"${WORKING_DIR}/logs/etc/openstack/${CONTAINER_NAME}/\" || true"}],"source_content_type":"text/x-sh","patch_set":8,"id":"7f287b81_1ce3d707","line":51,"updated":"2017-08-22 17:23:57.000000000","message":"It does, but I\u0027ll add in the -1 anyway for clarity","commit_id":"8589ff201ffcd513cad99d022cbfe9f99983f3b7"}]}