)]}'
{"tools/test-setup.sh":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b88d186772685474092ffb7e28dab63c90cf9d3b","unresolved":false,"context_lines":[{"line_number":30,"context_line":"function set_conf_line { # file regex value"},{"line_number":31,"context_line":"    sudo sh -c \"grep -q -e \u0027$2\u0027 $1 \u0026\u0026 \\"},{"line_number":32,"context_line":"            sed -i \u0027s|$2|$3|g\u0027 $1 || \\"},{"line_number":33,"context_line":"            echo \u0027$3\u0027 \u003e\u003e $1\""},{"line_number":34,"context_line":"}"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"if is_rhel7 || is_rhel8; then"}],"source_content_type":"text/x-sh","patch_set":5,"id":"49aecc59_14d57036","line":33,"updated":"2021-06-25 11:30:16.000000000","message":"Wonderful. So this replaces the first (and presumably only) entry of $2 with $3 in file $1. Seeing as we\u0027re busting out a separate shell, maybe it would have made sense to do this in Python, but I tested this and it does what it says on the tin so...\n\nA function comment would have been helpful though, mind","commit_id":"085de864db31631ead22f9c23488ed05b6bd5f88"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b88d186772685474092ffb7e28dab63c90cf9d3b","unresolved":false,"context_lines":[{"line_number":34,"context_line":"}"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"if is_rhel7 || is_rhel8; then"},{"line_number":37,"context_line":"    # mysql needs to be started on centos/rhel"},{"line_number":38,"context_line":"    sudo systemctl restart mariadb.service"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    # postgres setup for centos"}],"source_content_type":"text/x-sh","patch_set":5,"id":"878c97d2_64a4b492","line":37,"updated":"2021-06-25 11:30:16.000000000","message":"reasoning would be useful so we can evaluate things if/when CentOS Stream 9/RHEL9 rolls around","commit_id":"085de864db31631ead22f9c23488ed05b6bd5f88"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b88d186772685474092ffb7e28dab63c90cf9d3b","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    set_conf_line $PG_HBA \u0027^local[ \\t]*all[ \\t]*all.*\u0027 \u0027local all all peer\u0027"},{"line_number":47,"context_line":"    set_conf_line $PG_HBA \u0027^host[ \\t]*all[ \\t]*all[ \\t]*127.0.0.1\\/32.*\u0027 \u0027host all all 127.0.0.1/32 scram-sha-256\u0027"},{"line_number":48,"context_line":"    set_conf_line $PG_HBA \u0027^host[ \\t]*all[ \\t]*all[ \\t]*::1\\/128.*\u0027 \u0027host all all ::1/128 scram-sha-256\u0027"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    sudo systemctl restart postgresql.service"},{"line_number":51,"context_line":"fi"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-sh","patch_set":5,"id":"0463b6ff_f91c3504","line":49,"updated":"2021-06-25 11:30:16.000000000","message":"Took me a bit to figure out what was happening here. So we\u0027re enabling password-based encryption rather than ident or whatever postgres defaults to, and we\u0027re using SCRAM rather than md5 or password because it\u0027s more secure (or possibly all that\u0027s supported in RHEL 8? idk).\n\nAgain, a quick comment would help future readers.","commit_id":"085de864db31631ead22f9c23488ed05b6bd5f88"}]}