)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"94fd495d7b7cc9a16b47f3b66862ab249b3d5e01","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Kevko \u003cmichal.arbet@ultimum.io\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-03-26 22:37:20 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extend init-runonce.sh script - create addiditonal resources"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For the upstream CI, we want to use Rally and specifically"},{"line_number":10,"context_line":"Tempest tests. However, we want to run Tempest through static"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"fc34be59_a2a467a9","line":7,"range":{"start_line":7,"start_character":39,"end_line":7,"end_character":50},"updated":"2024-04-02 13:53:15.000000000","message":"additional","commit_id":"5e0d3fb38a0f377fe4be3c1b7f48fbce0e71b4d2"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"7d355f3fec826d3caff40620cea795daaab3493a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":13,"id":"99732b09_9f8582be","updated":"2024-05-22 13:53:43.000000000","message":"I think we should not extend the bash parts here, can we just move those to ansible or python (whatever is simpler)?\n\nas said in kolla meeting: the end goal was to move to tempest to get rid of (most of) bash not to extend bash code base :)\n\nI really think it\u0027s good work being done here, just shuffle the stuff around from bash to python and/or ansible imho.\n\nThanks","commit_id":"8f65f77d856146ad9cf7b36d479c1bb1e8d65984"}],"tests/run.yml":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"82c6c94776d8058747e6c063639c149af1991e83","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":114,"context_line":"            chdir: \"{{ kolla_ansible_src_dir }}\""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        # Kevko\u0027s hack for now :)"},{"line_number":117,"context_line":"        # ! THIS IS ONLY TEMPORARY !"},{"line_number":118,"context_line":"        # ! THIS WILL BE REMOVED AFTER MERGE INTO STABLE/*"},{"line_number":119,"context_line":"        # ! :) !"},{"line_number":120,"context_line":"        - name: Download init-runonce"},{"line_number":121,"context_line":"          ansible.builtin.get_url:"},{"line_number":122,"context_line":"            url: \"https://raw.githubusercontent.com/keuko/kolla-ansible/stable/{{ previous_release | lower }}/tools/init-runonce\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"0de4215d_79d1e243","line":119,"range":{"start_line":116,"start_character":0,"end_line":119,"end_character":16},"updated":"2024-03-28 12:20:35.000000000","message":"not sure I quite understand this.\n\nis there a review somewhere containing these changes?\n\nhttps://github.com/openstack/kolla-ansible/commit/77b6e723293967efc43d8dcafbe169a92aa81c6c\n\nmight be worth it do add it as a dependency here then?","commit_id":"5e0d3fb38a0f377fe4be3c1b7f48fbce0e71b4d2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"12dbea48c323fb92616b150811feeec103345352","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":114,"context_line":"            chdir: \"{{ kolla_ansible_src_dir }}\""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        # Kevko\u0027s hack for now :)"},{"line_number":117,"context_line":"        # ! THIS IS ONLY TEMPORARY !"},{"line_number":118,"context_line":"        # ! THIS WILL BE REMOVED AFTER MERGE INTO STABLE/*"},{"line_number":119,"context_line":"        # ! :) !"},{"line_number":120,"context_line":"        - name: Download init-runonce"},{"line_number":121,"context_line":"          ansible.builtin.get_url:"},{"line_number":122,"context_line":"            url: \"https://raw.githubusercontent.com/keuko/kolla-ansible/stable/{{ previous_release | lower }}/tools/init-runonce\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"9463f7e3_db8cac84","line":119,"range":{"start_line":116,"start_character":0,"end_line":119,"end_character":16},"in_reply_to":"03f719c0_e0a10e3a","updated":"2024-04-06 12:00:10.000000000","message":"Yeah, of course ... I just need to see if everything is passing ..and I am not able to do it until previous release will adopt this also...","commit_id":"5e0d3fb38a0f377fe4be3c1b7f48fbce0e71b4d2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"ddbc8ea9a55c886f0843ec55a54e6c34d5f286f0","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":114,"context_line":"            chdir: \"{{ kolla_ansible_src_dir }}\""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        # Kevko\u0027s hack for now :)"},{"line_number":117,"context_line":"        # ! THIS IS ONLY TEMPORARY !"},{"line_number":118,"context_line":"        # ! THIS WILL BE REMOVED AFTER MERGE INTO STABLE/*"},{"line_number":119,"context_line":"        # ! :) !"},{"line_number":120,"context_line":"        - name: Download init-runonce"},{"line_number":121,"context_line":"          ansible.builtin.get_url:"},{"line_number":122,"context_line":"            url: \"https://raw.githubusercontent.com/keuko/kolla-ansible/stable/{{ previous_release | lower }}/tools/init-runonce\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5ca1b535_5bb883bb","line":119,"range":{"start_line":116,"start_character":0,"end_line":119,"end_character":16},"in_reply_to":"0de4215d_79d1e243","updated":"2024-04-02 12:01:44.000000000","message":"Even if you add a patch into dependency chain ... zuul is checkouting **previous** version of already **merged code** ... so it\u0027s impossible to make it work ...\n\nYou can\u0027t use depends-on as it is for cross project dependency  - kolla \u003c-\u003e kolla-ansible ...aaaand if you use dependency as a commit in chain ...you will test patchA -\u003e patchB -\u003e patchC ...but previous version will be *always* github merged previous version ...for now it is bobccat 2023.2 for normal upgrade and 2023.1 for slurp...\n\nGot it ?\n\n2023.1 - https://review.opendev.org/c/openstack/kolla-ansible/+/914860\n2023.2 - https://review.opendev.org/c/openstack/kolla-ansible/+/914881","commit_id":"5e0d3fb38a0f377fe4be3c1b7f48fbce0e71b4d2"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"572052fc89f02f696ebc6b4c33d8933488382024","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":114,"context_line":"            chdir: \"{{ kolla_ansible_src_dir }}\""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        # Kevko\u0027s hack for now :)"},{"line_number":117,"context_line":"        # ! THIS IS ONLY TEMPORARY !"},{"line_number":118,"context_line":"        # ! THIS WILL BE REMOVED AFTER MERGE INTO STABLE/*"},{"line_number":119,"context_line":"        # ! :) !"},{"line_number":120,"context_line":"        - name: Download init-runonce"},{"line_number":121,"context_line":"          ansible.builtin.get_url:"},{"line_number":122,"context_line":"            url: \"https://raw.githubusercontent.com/keuko/kolla-ansible/stable/{{ previous_release | lower }}/tools/init-runonce\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"03f719c0_e0a10e3a","line":119,"range":{"start_line":116,"start_character":0,"end_line":119,"end_character":16},"in_reply_to":"5ca1b535_5bb883bb","updated":"2024-04-02 17:31:00.000000000","message":"yeah, sorry, I got the technical part, so you didn\u0027t add it as a dependency because gerrit can\u0027t really resolve this (in a meaningful way).\n\nWhat is was more getting at is, that it is still useful information for me as a poor reviewer. This information should be present somewhere, that this review technically depends on other stuff being done. Because I\u0027m not gonna +1 something which randomly loads code from github, sorry 😊 Got it?","commit_id":"5e0d3fb38a0f377fe4be3c1b7f48fbce0e71b4d2"}],"tools/init-runonce":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c234ee0edbd899166a1f3c95b495c31ff903a25a","unresolved":true,"context_lines":[{"line_number":16,"context_line":"            ;;"},{"line_number":17,"context_line":"        --user)"},{"line_number":18,"context_line":"            KOLLA_USER\u003d\"$2\""},{"line_number":19,"context_line":"            shift # preskočí hodnotu argumentu"},{"line_number":20,"context_line":"            ;;"},{"line_number":21,"context_line":"        --pass)"},{"line_number":22,"context_line":"            KOLLA_PASS\u003d\"$2\""}],"source_content_type":"application/octet-stream","patch_set":7,"id":"7f86e185_d0d53648","line":19,"range":{"start_line":19,"start_character":20,"end_line":19,"end_character":46},"updated":"2024-03-26 14:10:52.000000000","message":"english comments pls ;)","commit_id":"47bc67b88c80738f5480fafa5a6f9a8ea522d199"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"5d5ac09e2a176aaf09163c4f34c60070f00e216c","unresolved":false,"context_lines":[{"line_number":16,"context_line":"            ;;"},{"line_number":17,"context_line":"        --user)"},{"line_number":18,"context_line":"            KOLLA_USER\u003d\"$2\""},{"line_number":19,"context_line":"            shift # preskočí hodnotu argumentu"},{"line_number":20,"context_line":"            ;;"},{"line_number":21,"context_line":"        --pass)"},{"line_number":22,"context_line":"            KOLLA_PASS\u003d\"$2\""}],"source_content_type":"application/octet-stream","patch_set":7,"id":"c93e89e2_172bc8c2","line":19,"range":{"start_line":19,"start_character":20,"end_line":19,"end_character":46},"in_reply_to":"7f86e185_d0d53648","updated":"2024-03-26 17:15:39.000000000","message":"Done","commit_id":"47bc67b88c80738f5480fafa5a6f9a8ea522d199"}]}