)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"92f17e85f34db2dfd1f98e50571372df25d94039","unresolved":true,"context_lines":[{"line_number":11,"context_line":"CentOSStream instead of CentOS"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This breaks the is_fedora function in devstack"},{"line_number":14,"context_line":"preventing package installation and removeal."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This change make the comparison a substring"},{"line_number":17,"context_line":"check instead of equality to suppot both"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d718fa2d_15c026ef","line":14,"range":{"start_line":14,"start_character":36,"end_line":14,"end_character":44},"updated":"2020-12-10 01:52:29.000000000","message":"removal","commit_id":"b7d15122eb1aaffb821af06be151c2cb5151c61f"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"92f17e85f34db2dfd1f98e50571372df25d94039","unresolved":true,"context_lines":[{"line_number":14,"context_line":"preventing package installation and removeal."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This change make the comparison a substring"},{"line_number":17,"context_line":"check instead of equality to suppot both"},{"line_number":18,"context_line":"CentOS and CentOSStream."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I39ccefbd06f46adf5077f8d8001f37d3b190f040"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"499e2fe2_413dfcec","line":17,"range":{"start_line":17,"start_character":29,"end_line":17,"end_character":35},"updated":"2020-12-10 01:52:29.000000000","message":"support","commit_id":"b7d15122eb1aaffb821af06be151c2cb5151c61f"}],"functions-common":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c9dc6659c6c18437f2fae72212fdcc09c39eef87","unresolved":true,"context_lines":[{"line_number":389,"context_line":"    elif is_suse_linux_enterprise; then"},{"line_number":390,"context_line":"        # just use major release"},{"line_number":391,"context_line":"        DISTRO\u003d\"sle${os_RELEASE%.*}\""},{"line_number":392,"context_line":"    elif [[ \"$os_VENDOR\" \u003d~ (Red.*Hat) || \\"},{"line_number":393,"context_line":"        \"$os_VENDOR\" \u003d~ (CentOS) || \\"},{"line_number":394,"context_line":"        \"$os_VENDOR\" \u003d~ (Scientific) || \\"},{"line_number":395,"context_line":"        \"$os_VENDOR\" \u003d~ (OracleServer) || \\"},{"line_number":396,"context_line":"        \"$os_VENDOR\" \u003d~ (Virtuozzo) ]]; then"},{"line_number":397,"context_line":"        # Drop the . release as we assume it\u0027s compatible"},{"line_number":398,"context_line":"        # XXX re-evaluate when we get RHEL10"},{"line_number":399,"context_line":"        DISTRO\u003d\"rhel${os_RELEASE::1}\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"d5c8ebc8_3ea0bbf9","line":396,"range":{"start_line":392,"start_character":4,"end_line":396,"end_character":44},"updated":"2020-12-10 12:51:26.000000000","message":"specfically in GetDistro we use \u003d~ not exact matches\nso this passed fine.","commit_id":"b7d15122eb1aaffb821af06be151c2cb5151c61f"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"8c88d3afa3faade28efa0b198154c7c53186de7e","unresolved":true,"context_lines":[{"line_number":452,"context_line":"    [ \"$os_VENDOR\" \u003d \"Fedora\" ] || [ \"$os_VENDOR\" \u003d \"Red Hat\" ] || \\"},{"line_number":453,"context_line":"        [ \"$os_VENDOR\" \u003d \"RedHatEnterpriseServer\" ] || \\"},{"line_number":454,"context_line":"        [ \"$os_VENDOR\" \u003d \"RedHatEnterprise\" ] || \\"},{"line_number":455,"context_line":"        [[ \"$os_VENDOR\" \u003d~ \"CentOS\" ]] || [ \"$os_VENDOR\" \u003d \"OracleServer\" ] || \\"},{"line_number":456,"context_line":"        [ \"$os_VENDOR\" \u003d \"Virtuozzo\" ]"},{"line_number":457,"context_line":"}"},{"line_number":458,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"f7ef54f2_eb4d9433","line":455,"updated":"2020-12-10 08:03:47.000000000","message":"I\u0027d like to keep the match types consistent, otherwise we could integrate the RH matches above, too, so better add the new string explicitly.","commit_id":"b7d15122eb1aaffb821af06be151c2cb5151c61f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c9dc6659c6c18437f2fae72212fdcc09c39eef87","unresolved":true,"context_lines":[{"line_number":452,"context_line":"    [ \"$os_VENDOR\" \u003d \"Fedora\" ] || [ \"$os_VENDOR\" \u003d \"Red Hat\" ] || \\"},{"line_number":453,"context_line":"        [ \"$os_VENDOR\" \u003d \"RedHatEnterpriseServer\" ] || \\"},{"line_number":454,"context_line":"        [ \"$os_VENDOR\" \u003d \"RedHatEnterprise\" ] || \\"},{"line_number":455,"context_line":"        [[ \"$os_VENDOR\" \u003d~ \"CentOS\" ]] || [ \"$os_VENDOR\" \u003d \"OracleServer\" ] || \\"},{"line_number":456,"context_line":"        [ \"$os_VENDOR\" \u003d \"Virtuozzo\" ]"},{"line_number":457,"context_line":"}"},{"line_number":458,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"5550f2a9_56c3d521","line":455,"in_reply_to":"f7ef54f2_eb4d9433","updated":"2020-12-10 12:51:26.000000000","message":"sure i personally would merge the redhat ones the reason i prefer \u003d~ is that we do test like that elsewhere in the file that passed so i wanted to mirror that.","commit_id":"b7d15122eb1aaffb821af06be151c2cb5151c61f"}]}