)]}'
{"tests/deploy.sh":[{"author":{"_account_id":27336,"name":"XiaojueGuan","email":"xiaojueguan@gmail.com","username":"XiaojueGuan"},"change_message_id":"3bf17adcb44bbf772a7244d5b4e64f5f2362e7b2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    #TODO(inc0): Post-deploy complains that /etc/kolla is not writable. Probably we need to include become there"},{"line_number":14,"context_line":"    sudo chmod -R 777 /etc/kolla"},{"line_number":15,"context_line":"    # generate self-signed certificates for the optional internal TLS tests"},{"line_number":16,"context_line":"    if [[ \"$TLS_ENABLED\" \u003d \"True\" ]]; then"},{"line_number":17,"context_line":"        tools/kolla-ansible -i ${RAW_INVENTORY} -vvv certificates \u003e /tmp/logs/ansible/certificates"},{"line_number":18,"context_line":"    fi"},{"line_number":19,"context_line":"    # Actually do the deployment"}],"source_content_type":"text/x-sh","patch_set":3,"id":"ff570b3c_d8051416","side":"PARENT","line":16,"range":{"start_line":16,"start_character":12,"end_line":16,"end_character":23},"updated":"2020-05-18 00:15:25.000000000","message":"sine we want to remove the check for this variable `TLS_ENABLED`, may we also remove the env in tests/run.yml","commit_id":"695b55f7c1c35c3f274c8466dae3b47a123d20fc"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"de3e4ff2817286d372065c81deb4db8478e9ca40","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    #TODO(inc0): Post-deploy complains that /etc/kolla is not writable. Probably we need to include become there"},{"line_number":14,"context_line":"    sudo chmod -R 777 /etc/kolla"},{"line_number":15,"context_line":"    # generate self-signed certificates for the optional internal TLS tests"},{"line_number":16,"context_line":"    if [[ \"$TLS_ENABLED\" \u003d \"True\" ]]; then"},{"line_number":17,"context_line":"        tools/kolla-ansible -i ${RAW_INVENTORY} -vvv certificates \u003e /tmp/logs/ansible/certificates"},{"line_number":18,"context_line":"    fi"},{"line_number":19,"context_line":"    # Actually do the deployment"}],"source_content_type":"text/x-sh","patch_set":3,"id":"ff570b3c_c6988ec1","side":"PARENT","line":16,"range":{"start_line":16,"start_character":12,"end_line":16,"end_character":23},"in_reply_to":"ff570b3c_d8051416","updated":"2020-05-18 09:04:33.000000000","message":"Thanks for the review, but DNM \u003d\u003d do not merge. It\u0027s just for testing.","commit_id":"695b55f7c1c35c3f274c8466dae3b47a123d20fc"}]}