)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"e3621db1e786250858a4ef1d321e83cdb457334b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d6197ca7_ac4e802e","updated":"2021-10-28 07:57:34.000000000","message":"check-rdo\n\nSounds good, thanks for taking the time to answer!","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3c6e56d89e01b8b8f5d016219f757cb71e180818","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c6cc06fd_4e927fd6","updated":"2021-10-29 06:59:37.000000000","message":"recheck ","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0792707853bc5073ef487a16a7cc259015ac55a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8aeb0f91_158267f8","updated":"2021-10-28 07:19:38.000000000","message":"thanks for posting","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"}],"deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"2750864229b9854f1e809a84be7bc2068371816c","unresolved":true,"context_lines":[{"line_number":187,"context_line":"            - name: remove all OpenStack packages"},{"line_number":188,"context_line":"              shell: |"},{"line_number":189,"context_line":"                yum -y remove *el7ost* \\"},{"line_number":190,"context_line":"                  mariadb-server* -- \\"},{"line_number":191,"context_line":"                  -*openvswitch* \\"},{"line_number":192,"context_line":"                  -python2-babel \\"},{"line_number":193,"context_line":"                  -python2-dateutil \\"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b246cdac_7ca6f7ba","line":190,"updated":"2021-10-28 06:25:46.000000000","message":"care to provide a diff of the actually removed packages? Just to ensure we\u0027re not seeing anything sensitive being removed..\nSame goes for the other patch. I\u0027m always a bit cautious with wildcards.","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"},{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"5ff42ffbfc608ccabcad1a6d0711eda5ac2fce79","unresolved":true,"context_lines":[{"line_number":187,"context_line":"            - name: remove all OpenStack packages"},{"line_number":188,"context_line":"              shell: |"},{"line_number":189,"context_line":"                yum -y remove *el7ost* \\"},{"line_number":190,"context_line":"                  mariadb-server* -- \\"},{"line_number":191,"context_line":"                  -*openvswitch* \\"},{"line_number":192,"context_line":"                  -python2-babel \\"},{"line_number":193,"context_line":"                  -python2-dateutil \\"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e375b198_ba80eb94","line":190,"in_reply_to":"ab1babce_a489ce8d","updated":"2021-10-28 07:54:51.000000000","message":"Marios is right, the intention here is to get rid of all the mariadb-server* packages from the host. However, if interested on which packages this wildcard cover I took this from the CI execution:\n\nPackage mariadb-server.x86_64 3:10.1.20-3.el7ost will be erased\nPackage mariadb-server-galera.x86_64 3:10.1.20-3.el7ost will be erased\n\nThese would be the two packages matched by this wildcard, but I would leave the wildcard in case any other package would be present as according to Damien that would cause issues with the containerized mariadb.","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0792707853bc5073ef487a16a7cc259015ac55a3","unresolved":true,"context_lines":[{"line_number":187,"context_line":"            - name: remove all OpenStack packages"},{"line_number":188,"context_line":"              shell: |"},{"line_number":189,"context_line":"                yum -y remove *el7ost* \\"},{"line_number":190,"context_line":"                  mariadb-server* -- \\"},{"line_number":191,"context_line":"                  -*openvswitch* \\"},{"line_number":192,"context_line":"                  -python2-babel \\"},{"line_number":193,"context_line":"                  -python2-dateutil \\"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ab1babce_a489ce8d","line":190,"in_reply_to":"b246cdac_7ca6f7ba","updated":"2021-10-28 07:19:38.000000000","message":"by \u0027other patch\u0027 do you mean the train cherrypick which was posted first? \n\ni think the point here is to remove *all* mariadb server related packages from the host so the wildcard should be OK but sanity checking is also hard to argue with ;)\n\nbut then would you want to add a list of packages here instead of the wildcard? Or you just want to check what that wildcard is currently matching?","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"e3621db1e786250858a4ef1d321e83cdb457334b","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            - name: remove all OpenStack packages"},{"line_number":188,"context_line":"              shell: |"},{"line_number":189,"context_line":"                yum -y remove *el7ost* \\"},{"line_number":190,"context_line":"                  mariadb-server* -- \\"},{"line_number":191,"context_line":"                  -*openvswitch* \\"},{"line_number":192,"context_line":"                  -python2-babel \\"},{"line_number":193,"context_line":"                  -python2-dateutil \\"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e25de57f_cf02f81e","line":190,"in_reply_to":"e375b198_ba80eb94","updated":"2021-10-28 07:57:34.000000000","message":"Ack","commit_id":"d33865cded7766eb8d35ce56d5eb206c2dda1ad9"}]}