)]}'
{".zuul.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"5f2b6a1b7983d7209ca1d97c3d107c6927070040","unresolved":true,"context_lines":[{"line_number":702,"context_line":"        - barbican-tempest-plugin-simple-crypto:"},{"line_number":703,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":704,"context_line":"            voting: false"},{"line_number":705,"context_line":"        - tempest-integrated-compute-centos-8-stream:"},{"line_number":706,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":707,"context_line":"        - tempest-centos8-stream-fips:"},{"line_number":708,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":709,"context_line":"            voting: false"},{"line_number":710,"context_line":"    gate:"},{"line_number":711,"context_line":"      jobs:"},{"line_number":712,"context_line":"        - nova-live-migration"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"43d46bcc_f62cae56","line":709,"range":{"start_line":705,"start_character":0,"end_line":709,"end_character":25},"updated":"2022-11-16 16:53:31.000000000","message":"these are failing constantly as I see, why not remove them then, to free up some CI resource?","commit_id":"343f01f7b922e9a54fed0c2796c47754beb44298"},{"author":{"_account_id":34120,"name":"Andre Aranha","display_name":"afariasa","email":"afariasa@redhat.com","username":"afariasa"},"change_message_id":"09d7573d6e62eae45e15b05e1331abdd0ebb26dc","unresolved":false,"context_lines":[{"line_number":702,"context_line":"        - barbican-tempest-plugin-simple-crypto:"},{"line_number":703,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":704,"context_line":"            voting: false"},{"line_number":705,"context_line":"        - tempest-integrated-compute-centos-8-stream:"},{"line_number":706,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":707,"context_line":"        - tempest-centos8-stream-fips:"},{"line_number":708,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":709,"context_line":"            voting: false"},{"line_number":710,"context_line":"    gate:"},{"line_number":711,"context_line":"      jobs:"},{"line_number":712,"context_line":"        - nova-live-migration"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f0024513_a2450168","line":709,"range":{"start_line":705,"start_character":0,"end_line":709,"end_character":25},"in_reply_to":"43d46bcc_f62cae56","updated":"2023-01-20 15:36:16.000000000","message":"Done","commit_id":"343f01f7b922e9a54fed0c2796c47754beb44298"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"c15c143398c1becf21464ccff566873e5a7f4ffd","unresolved":true,"context_lines":[{"line_number":728,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":729,"context_line":"        - openstacksdk-functional-devstack:"},{"line_number":730,"context_line":"            irrelevant-files: *nova-base-irrelevant-files"},{"line_number":731,"context_line":"    periodic-weekly:"},{"line_number":732,"context_line":"      jobs:"},{"line_number":733,"context_line":"        - tempest-centos9-stream-fips"},{"line_number":734,"context_line":"    experimental:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"d937d7ed_9bc85248","line":731,"updated":"2023-04-11 11:09:45.000000000","message":"Not a clean cherry-pick\nthis should be documented in commit msg","commit_id":"da03f38b0286cb5a2830a5daf4fdaeab76368f11"}],"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"bbcdd25c439f4f0081b56e885440c116f48d27e6","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Andre Aranha \u003cafariasa@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-01-20 15:36:05 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"a6956e3c_53159609","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"updated":"2023-02-03 16:49:11.000000000","message":"should be: \"[stable-only] Test setting the nova job to centos-9-stream\"\n\n(otherwise backport validator job will fail in gate queue)","commit_id":"87bd962a959bf7654d64d4664617f86deaf944e7"},{"author":{"_account_id":34120,"name":"Andre Aranha","display_name":"afariasa","email":"afariasa@redhat.com","username":"afariasa"},"change_message_id":"350186e59d1d6c9c79cf6b2cd2cd850879438758","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Andre Aranha \u003cafariasa@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-01-20 15:36:05 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"56df73c6_186eca96","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"in_reply_to":"a6956e3c_53159609","updated":"2023-02-10 13:32:19.000000000","message":"Done","commit_id":"87bd962a959bf7654d64d4664617f86deaf944e7"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"c9b4deffe7848670e90e0935e68bad04f8a2ee57","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2023-02-10 13:32:01 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"282ab6ba_4fb4ec80","line":8,"updated":"2023-02-10 14:54:43.000000000","message":"I\u0027d appreciate if you could document the difference between the master patch (which was merged in Zed) and this backport.","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":34120,"name":"Andre Aranha","display_name":"afariasa","email":"afariasa@redhat.com","username":"afariasa"},"change_message_id":"163efa8d5d2e1c5a0d756bbb72bac8497b8e30aa","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2023-02-10 13:32:01 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"c4beea65_17a14c3f","line":8,"in_reply_to":"282ab6ba_4fb4ec80","updated":"2023-02-13 16:53:43.000000000","message":"If I recall correctly, Zed patch was 2 patches, one was merged earlier and the one that was backported to this patch had just some fixes.\nBut the goal of both, this and the original, was to replace Centos 8 by Centos 9.","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6f506214a2ff9352c8e92f5797963b3f7623dc5d","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2023-02-10 13:32:01 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"050f6a89_38f4962d","line":8,"in_reply_to":"c4beea65_17a14c3f","updated":"2023-04-11 11:59:15.000000000","message":"this should either be called out explicitly as a squash merge fo the two patches in the commit or both patches should be backported togeter as indiviual patches.","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"c9b4deffe7848670e90e0935e68bad04f8a2ee57","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"99dcdb82_aa8911fd","line":9,"updated":"2023-02-10 14:54:43.000000000","message":"any reason why you\u0027re not following the stable guideline of mentioning the merge commit SHA1 of the original commit ?","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"2c957faefca3da62a2b7c45d8f3d1adf2df0808e","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5d3966d9_b33bbe5e","line":9,"in_reply_to":"582caff0_81c4b152","updated":"2023-02-14 15:22:03.000000000","message":"Just follow the stable guidelines as mentioned here: \nhttps://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes\n\nWhat\u0027s missing here is the mention of the master SHA1 automatically given by the -x flag but also some extra commit message paragraph describing the differences (and also why we have the specific stable-only tag if we *actually* backport from another patch.","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":34120,"name":"Andre Aranha","display_name":"afariasa","email":"afariasa@redhat.com","username":"afariasa"},"change_message_id":"db9380940fa20d188ee42b3b97d60156613854f5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"491f5a06_a7103456","line":9,"in_reply_to":"5d3966d9_b33bbe5e","updated":"2023-02-17 13:59:29.000000000","message":"Done","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":34120,"name":"Andre Aranha","display_name":"afariasa","email":"afariasa@redhat.com","username":"afariasa"},"change_message_id":"163efa8d5d2e1c5a0d756bbb72bac8497b8e30aa","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[stable-only] Test setting the nova job to centos-9-stream"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic8da6ee0313a911d742190ea5b0d4362cb6aef2f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"582caff0_81c4b152","line":9,"in_reply_to":"99dcdb82_aa8911fd","updated":"2023-02-13 16:53:43.000000000","message":"I`m not so sure, I think I was still figuring out how to backport when there was merge conflict, but I can\u0027t tell for sure.\nI can fix that, should I add the Zed SHA1 commit?","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"5f2b6a1b7983d7209ca1d97c3d107c6927070040","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4e984dc3_9d6b7d0d","updated":"2022-11-16 16:53:31.000000000","message":"I\u0027m OK to remove failing jobs that wastes CI resources, also I\u0027m OK to add the centos9 job to periodic-weekly instead (if someone is watching its result and act if it fails). my only question is: why not remove failing centos8 jobs completely (see my inline comment)? Giving -1 to highlight that.","commit_id":"343f01f7b922e9a54fed0c2796c47754beb44298"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"074aacc5680b365f901f8d52efe22a4261825daf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"677fb0ef_9a0c763b","updated":"2022-10-10 10:53:47.000000000","message":"im a little conflicted about this since we normal do not replace jobs or add new ones after we release a stable branch.\n\nwe kept python 3.6 support for one release longer then we planned to enable rdo to test centos 8 stream in yoga. so it partly feels wrong to add 9 now after we were forced to revers course and keep support for 8.\n\n9 stream is not a supported runtime for xena or wallaby so while yoga is boradarline we shoudl not backport this to older release regardless of what we decied to do here.\n","commit_id":"343f01f7b922e9a54fed0c2796c47754beb44298"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"be490d656f3801510bb02b25a16eada84ebf56d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5bc56ac4_4d51fade","updated":"2022-10-10 18:22:01.000000000","message":"looks ok to me.","commit_id":"343f01f7b922e9a54fed0c2796c47754beb44298"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"be490d656f3801510bb02b25a16eada84ebf56d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"67e44887_91ed2206","in_reply_to":"677fb0ef_9a0c763b","updated":"2022-10-10 18:22:01.000000000","message":"9 stream is there is testing runtime for yoga - https://governance.openstack.org/tc/reference/runtimes/yoga.html\n\nIt is ok to add the job and as TC discussed, to over its testing at our best effort, we can add centos stream jobs only in periodic pipeline which is this patch adding. Testing it periodically and debugging/reporting the issues to centos community is best testing effort on this distro.","commit_id":"343f01f7b922e9a54fed0c2796c47754beb44298"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"bbcdd25c439f4f0081b56e885440c116f48d27e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ed01acb_0a864ec9","updated":"2023-02-03 16:49:11.000000000","message":"Sorry, one minor update is needed to be able to merge this, see my inline comment.","commit_id":"87bd962a959bf7654d64d4664617f86deaf944e7"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"35f714563004c1ba2aeae187f2c1b312b8a4b0c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"116cd4a5_78c1dd35","updated":"2023-02-03 16:46:21.000000000","message":"Thanks for the update.","commit_id":"87bd962a959bf7654d64d4664617f86deaf944e7"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"c9b4deffe7848670e90e0935e68bad04f8a2ee57","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"129377dc_d991ad66","updated":"2023-02-10 14:54:43.000000000","message":"Sorry if this sounds procedural, but I\u0027d appreciate some sort of explanation behind this backport.","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":34120,"name":"Andre Aranha","display_name":"afariasa","email":"afariasa@redhat.com","username":"afariasa"},"change_message_id":"163efa8d5d2e1c5a0d756bbb72bac8497b8e30aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"20e1d647_49f0cfda","in_reply_to":"129377dc_d991ad66","updated":"2023-02-13 16:53:43.000000000","message":"Ack, Please, let me know if I should add the original commit SHA1 on the commit message, so I update it at the same commit.","commit_id":"046a8446f046bbb8f2731eeca5cf11ee22d31370"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"c15c143398c1becf21464ccff566873e5a7f4ffd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"575c26b9_8052ad0c","updated":"2023-04-11 11:09:45.000000000","message":"the patch seems okay to me, but as this is not clean cherry-pick, we should also add a note/document the conflicts or changes in commit msg.\n\nplease refer this example backport.\nhttps://review.opendev.org/c/openstack/nova/+/686292","commit_id":"da03f38b0286cb5a2830a5daf4fdaeab76368f11"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6f506214a2ff9352c8e92f5797963b3f7623dc5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"2e4c047b_d263f8c9","updated":"2023-04-11 11:59:15.000000000","message":"we should not do a stable only patch here this should be a cerry-pick form the orginal or  optionally as a squash merge if both master patches cant be added independently.\n\nkeeping both patches clean would be my preference.\n\nwe really only use stable only as a last resort.","commit_id":"da03f38b0286cb5a2830a5daf4fdaeab76368f11"}]}