)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"4b517af9ea4dfb47cb1a8f231b8a92ab68b40efb","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Tim Beermann \u003cbeermann@osism.tech\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-04-29 08:54:50 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removed sanity checks"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"\"Smoke tests\" for barbican, cinder, glance and keystone have been removed as discussed in PTG April 2022."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"edd19927_8b57d839","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":8},"updated":"2022-04-29 11:36:24.000000000","message":"nit: by commit title guidelines, this should be in imperative so: \"Remove\"","commit_id":"f6c216871e2c8403efeab0a7cd8793ce36d50bf3"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"f86e8e989486f05368039d62ea5b65cff5c387e1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tim Beermann \u003cbeermann@osism.tech\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-04-29 08:54:50 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removed sanity checks"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"\"Smoke tests\" for barbican, cinder, glance and keystone have been removed as discussed in PTG April 2022."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"6dbc0379_a9c02cd1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":8},"in_reply_to":"edd19927_8b57d839","updated":"2022-05-13 17:40:20.000000000","message":"Done","commit_id":"f6c216871e2c8403efeab0a7cd8793ce36d50bf3"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"4b517af9ea4dfb47cb1a8f231b8a92ab68b40efb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"21076c95_b85d94a3","updated":"2022-04-29 11:36:24.000000000","message":"thanks, code change is good, need to tweak the reno though","commit_id":"f6c216871e2c8403efeab0a7cd8793ce36d50bf3"}],"releasenotes/notes/deprecate-sanity-checks-928ef3af2dc0f187.yaml":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"4b517af9ea4dfb47cb1a8f231b8a92ab68b40efb","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"deprecations:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    In the April 2022 PTG the deprecation and removal of the sanity"},{"line_number":5,"context_line":"    checks has been confirmed. Therefore the usage of"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    .. code-block:: bash"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"      kolla-ansible check"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"    is not possible any more."},{"line_number":12,"context_line":"features:"},{"line_number":13,"context_line":"  - |"},{"line_number":14,"context_line":"    Sanity checks have been removed. These \"smoke tests\" orignially"},{"line_number":15,"context_line":"    were implemented for barbican, cinder, glance and keystone."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1fb30948_fa1929be","line":15,"range":{"start_line":2,"start_character":0,"end_line":15,"end_character":63},"updated":"2022-04-29 11:36:24.000000000","message":"make a note of type \"upgrade\" and mention that they were removed because they were broken","commit_id":"f6c216871e2c8403efeab0a7cd8793ce36d50bf3"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"948cb329d873533caed66ac9c8b3605891b50ef6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"deprecations:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    In the April 2022 PTG the deprecation and removal of the sanity"},{"line_number":5,"context_line":"    checks has been confirmed. Therefore the usage of"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    .. code-block:: bash"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"      kolla-ansible check"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"    is not possible any more."},{"line_number":12,"context_line":"features:"},{"line_number":13,"context_line":"  - |"},{"line_number":14,"context_line":"    Sanity checks have been removed. These \"smoke tests\" orignially"},{"line_number":15,"context_line":"    were implemented for barbican, cinder, glance and keystone."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"531997b5_19a87d0e","line":15,"range":{"start_line":2,"start_character":0,"end_line":15,"end_character":63},"in_reply_to":"1fb30948_fa1929be","updated":"2022-05-13 17:39:40.000000000","message":"Done","commit_id":"f6c216871e2c8403efeab0a7cd8793ce36d50bf3"}],"tests/deploy.sh":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b1b3ae14f70b3c42215951159fdcc2b6bcd059dd","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv pull \u0026\u003e /tmp/logs/ansible/pull"},{"line_number":22,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv deploy \u0026\u003e /tmp/logs/ansible/deploy"},{"line_number":23,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv post-deploy \u0026\u003e /tmp/logs/ansible/post-deploy"},{"line_number":24,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv check \u0026\u003e /tmp/logs/ansible/check-deploy"},{"line_number":25,"context_line":"}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"4207cce9_54bdfbee","line":24,"updated":"2022-05-04 06:20:13.000000000","message":"Also need to drop this in reconfigure.sh and upgrade.sh","commit_id":"1e8c090f5cd86cce04bc82a476dcbb2927334df8"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"948cb329d873533caed66ac9c8b3605891b50ef6","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv pull \u0026\u003e /tmp/logs/ansible/pull"},{"line_number":22,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv deploy \u0026\u003e /tmp/logs/ansible/deploy"},{"line_number":23,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv post-deploy \u0026\u003e /tmp/logs/ansible/post-deploy"},{"line_number":24,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv check \u0026\u003e /tmp/logs/ansible/check-deploy"},{"line_number":25,"context_line":"}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"184fd42f_223fc1e6","line":24,"in_reply_to":"4207cce9_54bdfbee","updated":"2022-05-13 17:39:40.000000000","message":"Done","commit_id":"1e8c090f5cd86cce04bc82a476dcbb2927334df8"}],"tools/kolla-ansible":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e211c507a6f4ff0745f210f08f16198763553681","unresolved":true,"context_lines":[{"line_number":173,"context_line":"Commands:"},{"line_number":174,"context_line":"    install-deps         Install Ansible Galaxy dependencies"},{"line_number":175,"context_line":"    prechecks            Do pre-deployment checks for hosts"},{"line_number":176,"context_line":"    check                Do post-deployment smoke tests"},{"line_number":177,"context_line":"    mariadb_recovery     Recover a completely stopped mariadb cluster"},{"line_number":178,"context_line":"    mariadb_backup       Take a backup of MariaDB databases"},{"line_number":179,"context_line":"                             --full (default)"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"89bab59b_d8721d39","side":"PARENT","line":176,"updated":"2022-05-03 08:52:44.000000000","message":"I\u0027m not sure if we should remove this at once or just deprecate and make it a no-op, so that people\u0027s scripts don\u0027t break.\n\nYou also need to make sure that the check-deploy step in CI is no longer executed, cf. https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_199/839716/4/check/kolla-ansible-ubuntu-source/199b1fc/primary/logs/ansible/check-deploy","commit_id":"8ece7cce90ad08c94254217e763bf4b77b34d34e"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"6b7cbbb1f4ef706d70db3b5de74ed8b531c54db9","unresolved":true,"context_lines":[{"line_number":173,"context_line":"Commands:"},{"line_number":174,"context_line":"    install-deps         Install Ansible Galaxy dependencies"},{"line_number":175,"context_line":"    prechecks            Do pre-deployment checks for hosts"},{"line_number":176,"context_line":"    check                Do post-deployment smoke tests"},{"line_number":177,"context_line":"    mariadb_recovery     Recover a completely stopped mariadb cluster"},{"line_number":178,"context_line":"    mariadb_backup       Take a backup of MariaDB databases"},{"line_number":179,"context_line":"                             --full (default)"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"a4d0e605_c35827d6","side":"PARENT","line":176,"in_reply_to":"7a701f98_2b577420","updated":"2022-05-13 19:32:07.000000000","message":"Well, it did not do much so this new clear indication of \"failure on check\" might at least make users aware that it\u0027s a no-go.","commit_id":"8ece7cce90ad08c94254217e763bf4b77b34d34e"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"948cb329d873533caed66ac9c8b3605891b50ef6","unresolved":true,"context_lines":[{"line_number":173,"context_line":"Commands:"},{"line_number":174,"context_line":"    install-deps         Install Ansible Galaxy dependencies"},{"line_number":175,"context_line":"    prechecks            Do pre-deployment checks for hosts"},{"line_number":176,"context_line":"    check                Do post-deployment smoke tests"},{"line_number":177,"context_line":"    mariadb_recovery     Recover a completely stopped mariadb cluster"},{"line_number":178,"context_line":"    mariadb_backup       Take a backup of MariaDB databases"},{"line_number":179,"context_line":"                             --full (default)"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"7a701f98_2b577420","side":"PARENT","line":176,"in_reply_to":"89bab59b_d8721d39","updated":"2022-05-13 17:39:40.000000000","message":"Would prefer to simply remove it.","commit_id":"8ece7cce90ad08c94254217e763bf4b77b34d34e"}]}
