)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c3881022a94dd2f88c0ddc8cc2d03baacf0ecbb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"a07dec8e_362f5f8d","updated":"2024-07-18 15:03:09.000000000","message":"LGTM with one issue, see comment.","commit_id":"f1b3b0f07a5f43e0f4ca0180b1f8a6e34ec8c542"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"a3d816f18132120dbd53d9afa0a4d3d41f65b9e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"5df36122_2468953f","updated":"2024-07-29 10:02:16.000000000","message":"I don\u0027t think we want to backport this, do we?","commit_id":"a42b0b5ed1c428ba1795eedac40239e56008b258"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"236cc87eaaa821a940738a97f409505c6557dd8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"7cc5e3c8_dcff5e2d","updated":"2024-08-16 13:22:36.000000000","message":"reno missing ?","commit_id":"a42b0b5ed1c428ba1795eedac40239e56008b258"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6a8eabbf0ccf56fedbed1df26fb6f28ba2e7681e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":24,"id":"ea2f8f96_3f907c8d","in_reply_to":"7cc5e3c8_dcff5e2d","updated":"2024-08-16 13:26:40.000000000","message":"for a CI only patch?","commit_id":"a42b0b5ed1c428ba1795eedac40239e56008b258"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"3328460720ad5f344e4f85dd10735147f9fa0016","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":24,"id":"2f816fdc_e1295e1a","in_reply_to":"ea2f8f96_3f907c8d","updated":"2024-08-16 13:30:42.000000000","message":"Ah, sorry, I\u0027ve seen ansible/roles and didn\u0027t realized that\u0027s not part of kola-ansible or let\u0027s say supposed to be run in CI only.","commit_id":"a42b0b5ed1c428ba1795eedac40239e56008b258"}],"roles/cephadm/tasks/main.yml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2f72cfc743be3ef904bbfaba9998ad463c1202ec","unresolved":true,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Bootstrap cephadm"},{"line_number":35,"context_line":"  vars:"},{"line_number":36,"context_line":"    mon_ip: \"{{ hostvars[inventory_hostname][\u0027ansible_\u0027 + api_interface_name|replace(\u0027-\u0027,\u0027_\u0027)].ipv4.address }}\""},{"line_number":37,"context_line":"  command:"},{"line_number":38,"context_line":"    cmd: \u003e"},{"line_number":39,"context_line":"         cephadm"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"7503326f_729a32bd","line":36,"updated":"2024-06-24 13:35:32.000000000","message":"the fixes in this file should go into the parent patch?","commit_id":"ebf1a613e63184290f728d4a94f88fa865f015c3"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"b08ad715a051371d402fedcc748b76d79f614dfa","unresolved":true,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Bootstrap cephadm"},{"line_number":35,"context_line":"  vars:"},{"line_number":36,"context_line":"    mon_ip: \"{{ hostvars[inventory_hostname][\u0027ansible_\u0027 + api_interface_name|replace(\u0027-\u0027,\u0027_\u0027)].ipv4.address }}\""},{"line_number":37,"context_line":"  command:"},{"line_number":38,"context_line":"    cmd: \u003e"},{"line_number":39,"context_line":"         cephadm"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"a8c4ebcf_96e7847c","line":36,"in_reply_to":"7503326f_729a32bd","updated":"2024-06-24 14:42:47.000000000","message":"thanks for the hint, I found also more instances of this, see my comment in:\n\nhttps://review.opendev.org/c/openstack/kolla-ansible/+/922478/comments/c8f25dc8_e347e716\n\nI agree it should be fixed there.","commit_id":"ebf1a613e63184290f728d4a94f88fa865f015c3"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"d1a3ea691f6e10396d4999e639432acebf20596d","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Bootstrap cephadm"},{"line_number":35,"context_line":"  vars:"},{"line_number":36,"context_line":"    mon_ip: \"{{ hostvars[inventory_hostname][\u0027ansible_\u0027 + api_interface_name|replace(\u0027-\u0027,\u0027_\u0027)].ipv4.address }}\""},{"line_number":37,"context_line":"  command:"},{"line_number":38,"context_line":"    cmd: \u003e"},{"line_number":39,"context_line":"         cephadm"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"b348dca4_35e9e63f","line":36,"in_reply_to":"a8c4ebcf_96e7847c","updated":"2024-07-04 14:42:42.000000000","message":"Done","commit_id":"ebf1a613e63184290f728d4a94f88fa865f015c3"}],"tests/get_logs.sh":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c3881022a94dd2f88c0ddc8cc2d03baacf0ecbb4","unresolved":true,"context_lines":[{"line_number":48,"context_line":"    cp -r /etc/sudoers.d ${LOG_DIR}/system_logs/"},{"line_number":49,"context_line":"    cp /etc/sudoers ${LOG_DIR}/system_logs/sudoers.txt"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    cp /etc/hosts \u003e ${LOG_DIR}/system_logs/etc_hosts.txt"},{"line_number":52,"context_line":"    df -h \u003e ${LOG_DIR}/system_logs/df.txt"},{"line_number":53,"context_line":"    free  \u003e ${LOG_DIR}/system_logs/free.txt"},{"line_number":54,"context_line":"    lsblk \u003e ${LOG_DIR}/system_logs/lsblk.txt"}],"source_content_type":"text/x-sh","patch_set":22,"id":"5d839a70_fbaa5f95","line":51,"range":{"start_line":51,"start_character":0,"end_line":51,"end_character":2},"updated":"2024-07-18 15:03:09.000000000","message":"why is this needed? we already have the `/etc/hosts` file collected in our artifacts, retrievable via `logs/system_configs/hosts`.\n\nsee line 24 in this very same file:\n\nhttps://opendev.org/openstack/kolla-ansible/src/commit/4749b6067c578c2e32032948c64635bfb5f6a02a/tests/get_logs.sh#L24","commit_id":"f1b3b0f07a5f43e0f4ca0180b1f8a6e34ec8c542"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c49ccefff7e4f17b0abc635d1ea577b3df0175eb","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    cp -r /etc/sudoers.d ${LOG_DIR}/system_logs/"},{"line_number":49,"context_line":"    cp /etc/sudoers ${LOG_DIR}/system_logs/sudoers.txt"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    cp /etc/hosts \u003e ${LOG_DIR}/system_logs/etc_hosts.txt"},{"line_number":52,"context_line":"    df -h \u003e ${LOG_DIR}/system_logs/df.txt"},{"line_number":53,"context_line":"    free  \u003e ${LOG_DIR}/system_logs/free.txt"},{"line_number":54,"context_line":"    lsblk \u003e ${LOG_DIR}/system_logs/lsblk.txt"}],"source_content_type":"text/x-sh","patch_set":22,"id":"79a4e026_255099e4","line":51,"range":{"start_line":51,"start_character":0,"end_line":51,"end_character":2},"in_reply_to":"5d839a70_fbaa5f95","updated":"2024-07-18 18:49:11.000000000","message":"Yup, that was a brain fart, or similar situation.","commit_id":"f1b3b0f07a5f43e0f4ca0180b1f8a6e34ec8c542"}]}