)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a1f14aac78d5caf6345db3297dd8f4c3471c95b2","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Merge note:"},{"line_number":19,"context_line":"This patch needs to be merged first."},{"line_number":20,"context_line":"Immediately after that, merge the kolla\u0027s Depends-On patch."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ia9312f247d20e4233c669599804a51bacf15a75c"},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/kolla/+/989849"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"1bbe3588_2b3c18f0","line":20,"updated":"2026-05-28 05:55:19.000000000","message":"It\u0027s not how Depends-On in Zuul works","commit_id":"3061ad88c84ac27a7abaa4cb42dba947c56fb7da"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"3485b992c79eb643822c16e1eb0dcd65f7450793","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Merge note:"},{"line_number":19,"context_line":"This patch needs to be merged first."},{"line_number":20,"context_line":"Immediately after that, merge the kolla\u0027s Depends-On patch."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ia9312f247d20e4233c669599804a51bacf15a75c"},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/kolla/+/989849"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"b8d0f081_865741ce","line":20,"in_reply_to":"1bbe3588_2b3c18f0","updated":"2026-05-28 07:40:29.000000000","message":"This is one of those cross-repo dependency cases where the passing change\nneeds to be merged first.\nAnd actually, this is exactly how Depends-On works in Zuul, as described in the documentation:\n\nhttps://zuul-ci.org/docs/zuul/latest/gating.html?#cross-project-dependencies\nhttps://zuul-ci.org/docs/zuul/11.0.0/developer/specs/circular-dependencies.html\n\nSo what exactly did you mean by saying Zuul does not work like this?\n\nWas the -1 about the wording of the merge note, or do you disagree with what ?","commit_id":"3061ad88c84ac27a7abaa4cb42dba947c56fb7da"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e9f7aa4613e7b41bca89931fe031b0590ac68a1a","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Merge note:"},{"line_number":19,"context_line":"This patch needs to be merged first."},{"line_number":20,"context_line":"Immediately after that, merge the kolla\u0027s Depends-On patch."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ia9312f247d20e4233c669599804a51bacf15a75c"},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/kolla/+/989849"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"bfe3b93f_c039f03b","line":20,"in_reply_to":"b8d0f081_865741ce","updated":"2026-05-29 16:57:06.000000000","message":"Done","commit_id":"3061ad88c84ac27a7abaa4cb42dba947c56fb7da"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"bf67b921425533c0b0018822dfcf390673ab76e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2a31cd0b_86e114c5","updated":"2026-05-25 19:30:54.000000000","message":"recheck new set of images","commit_id":"bd907bb733015675df632e9baf9211181c1c93c0"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4db0c1b0bb85c57ecd97d41584d72bb693777f9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"be133365_1365b9eb","updated":"2026-05-26 09:06:48.000000000","message":"recheck unrelated","commit_id":"35b1ebfc2bb15f6c1b42170d721e1aa254387296"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"449942d620141da255d035def2b9acf534be812b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"9682dcc0_d9859223","updated":"2026-05-28 07:46:58.000000000","message":"Hmm, maybe I see on the kolla side there are probably patches which needs to be squashed together and depends-on against that squashed only one commit...\n\nIs it that ?","commit_id":"3061ad88c84ac27a7abaa4cb42dba947c56fb7da"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"2ddec1ea3a442f44402d8d9b6a14c4cfa1ddb726","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"a76b76ca_2655f37c","updated":"2026-05-28 09:29:50.000000000","message":"Maybe a short summary to clarify the situation. This kolla-ansible patch relies on two key changes in kolla itself:\n\n1. - perm: \"2775\" / perm: \"2557\" support (SGID bit handling)\n   - Fixed in kolla: https://review.opendev.org/c/openstack/kolla/+/988849\n   - Since it\u0027s only a fix in kolla, this can be merged independently right now\n   \n\n2. - files: [] feature\n   - Adds the ability to create/touch specific files and set ownership/permissions via config.json.\n   - Proposed in kolla: https://review.opendev.org/c/openstack/kolla/+/989936\n   - Since it\u0027s only a feature in kolla,  this can be merged independently right now alo\n\n\nSo, my question is - should I add both of these as Depends-On for this kolla-ansible patch?\n\n\n\n\nNow to the problematic part:\n\n3. - Removal of /var/log/kolla creation logic from container images\nhttps://review.opendev.org/c/openstack/kolla/+/989849\n\nThat change will naturally fail in CI until the related kolla-ansible patch is merged, because it removes the existing image-side directory and files creation logic while tests are still using the old kolla-ansible behavior (without the new config.json-based directory handling).\n\nAccording to Zuul documentation, cross-repository circular Depends-On relationships between are not supported, Because of that, I added a note in the commit message explaining that the kolla-ansible patch must be merged first before patch [3] can be properly verified by Zuul.\n\nIs it clearer now what workflow/proposal I was trying to describe?\n\n\nMaybe there is a question what will happen if i remove current depends-on ? \nWell, zuul will test \n\n- This revision of kolla-ansible\n- with kolla-images creation logic\n\nSo, on the end kolla-ansible config.json will work, but in second step kolla-extend-start in all of those files will rewrite sometimes owner, sometimes permissions ...etc which will cause fail of other patches stacked in this relation chain...\n\nSo, as I said, the best is:\n\n1. Merge https://review.opendev.org/c/openstack/kolla/+/988849\n2. Merge https://review.opendev.org/c/openstack/kolla/+/989936 \n3. Merge this kolla-ansible patch which handling directories and files https://review.opendev.org/c/openstack/kolla-ansible/+/989850\n4. Recheck and merge kolla patch (which will start to pass as kolla-ansible now handle all of those files/dirs) https://review.opendev.org/c/openstack/kolla/+/989849","commit_id":"3061ad88c84ac27a7abaa4cb42dba947c56fb7da"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"ad337a46db7fd092da4335d67cbae301448f50b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"a5edd9a0_7cd937ef","in_reply_to":"9682dcc0_d9859223","updated":"2026-05-28 07:57:14.000000000","message":"Or what ?","commit_id":"3061ad88c84ac27a7abaa4cb42dba947c56fb7da"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"59436a2b277710e1b2465bc465390eeec38797a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"9102173b_c2af2ea2","updated":"2026-05-30 10:05:12.000000000","message":"recheck unrelated","commit_id":"165015cb6158866603bd63c8f43cdf7b33e24246"}]}