)]}'
{"docker/heat/heat-base/extend_start.sh":[{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"1daa7eb7acb1f0b1b0d8abcedd1e8e07b79e04a9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    chmod 755 /var/log/kolla/heat"},{"line_number":8,"context_line":"fi"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Assume the service runs on top of Apache when user is root"},{"line_number":11,"context_line":"if [[ $USER \u003d\u003d \u0027root\u0027 ]]; then"},{"line_number":12,"context_line":"    # NOTE(pbourke): httpd will not clean up after itself in some cases which"},{"line_number":13,"context_line":"    # results in the container not being able to restart. (bug #1489676, 1557036)"},{"line_number":14,"context_line":"    if [[ \"${KOLLA_BASE_DISTRO}\" \u003d~ debian|ubuntu ]]; then"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dfeb2761_255ecd75","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":30},"updated":"2017-04-05 11:22:34.000000000","message":"Is this necessary? None of the other images bother with this check, I think it can be assumed apache is enabled","commit_id":"57ab0cabd54297d9334f6f6bc6da09144e0ff071"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"bfcef073c395b80663b52bb64a2d4d37a3d6360e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    chmod 755 /var/log/kolla/heat"},{"line_number":8,"context_line":"fi"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Assume the service runs on top of Apache when user is root"},{"line_number":11,"context_line":"if [[ $USER \u003d\u003d \u0027root\u0027 ]]; then"},{"line_number":12,"context_line":"    # NOTE(pbourke): httpd will not clean up after itself in some cases which"},{"line_number":13,"context_line":"    # results in the container not being able to restart. (bug #1489676, 1557036)"},{"line_number":14,"context_line":"    if [[ \"${KOLLA_BASE_DISTRO}\" \u003d~ debian|ubuntu ]]; then"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dfeb2761_7bea360f","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":30},"in_reply_to":"dfeb2761_255ecd75","updated":"2017-04-05 11:57:13.000000000","message":"Hey Paul, this is needed here because this container usually runs as the `heat` user to start the heat services. It\u0027s only in the case we want to run the services on top of apache that we need it to be the root user.","commit_id":"57ab0cabd54297d9334f6f6bc6da09144e0ff071"}]}