)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"c49fd0d5954fec3d3f12cbfd0c4f87583987bd84","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Commit 260c968118934 broke the gate by causing jobs"},{"line_number":10,"context_line":"to not get run when it added RE2 compatibility for"},{"line_number":11,"context_line":"irrelevant files. Digging found that RE2 doesn\u0027t"},{"line_number":12,"context_line":"support negative lookahead (and won\u0027t ever). This"},{"line_number":13,"context_line":"fixes the issue by just not including the single"},{"line_number":14,"context_line":"file we care about in each section."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[0] https://github.com/google/re2/issues/156"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I3bba89ac14414c6b7d375072ae92d2e0b5497736"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"8604388f_ecdd1436","line":15,"range":{"start_line":11,"start_character":17,"end_line":15,"end_character":1},"updated":"2024-06-05 05:59:00.000000000","message":"yes that\u0027s a known bit and negate: true was added for to get this work. Until we get/fix the regex usage, should be fine to get this.","commit_id":"f989a287a370935c91051621d3067ac824882f1f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0b5523df149fc2f4d399ded3d73192ec9a0b4473","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c0204b46_8084a06c","updated":"2024-06-04 17:13:22.000000000","message":"Different, but still not running all jobs, previous PS was better but child patch not running jobs","commit_id":"834870385511a881cf675e9240651b2c6d7ef1da"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"1246bc5f00c215349b5b86068f2bc835fef18aa2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7b9dbe95_aa62a9d2","updated":"2024-06-04 18:53:10.000000000","message":"Think this is wrong, should be ignoring everything zuul.d except for the one file, this matches","commit_id":"9bd10ab936ab114396949b2608a4e358148930b1"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"d8140008abebd29a3a417f48402c9a52bc2eb3d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1f71611d_4a360970","updated":"2024-06-04 22:41:24.000000000","message":"I think it\u0027s a good path forward though probably we could list the files once and then refer to a variable, to avoid duplication. Whoever knows more about zuul could follow up with a cleanup.","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"528c325865ca3567e4b8b8e10d71845b109d64fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b5451268_fd384f73","updated":"2024-06-04 23:08:24.000000000","message":"This works for me for now. Seems like they need to come up with a syntax for combining a regex with a negative match and a positive match for `negate: true` to be of value in any but the simplest cases (where there is just a single negate regex rule).","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f0eab9fe9684907949fc19bee12fcd32d12c51a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f71522d7_70b60135","updated":"2024-06-05 07:37:20.000000000","message":"fingers crossed","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"5b95689d60b27d96ad321c1b87a7e9fa1829c2a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"76fff6a8_964360b9","updated":"2024-06-04 23:09:15.000000000","message":"recheck","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"b2422704edd11c5d1a4629142f573cfec4caf18c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c1ace154_f32c482b","updated":"2024-06-05 15:37:41.000000000","message":"recheck","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"8020dbd2211a5e23db211a786fb0f36c86162b2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"96f3f0d5_870d60ee","updated":"2024-06-05 08:23:14.000000000","message":"recheck POST_FAILURES","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"c49fd0d5954fec3d3f12cbfd0c4f87583987bd84","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bcc9562b_06c18ae8","updated":"2024-06-05 05:59:00.000000000","message":"recheck POST_FAILURES","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b29f55b22394c0d3e23e63b942867472d1158098","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f808e709_c6b1b1fb","updated":"2024-06-05 19:11:30.000000000","message":"recheck cover timeout","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"4652a83309071359df745dd230330e6c77fa1d7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"43db3927_a6f83c69","updated":"2024-06-05 20:57:13.000000000","message":"confirmed in zuul that cover job is now non-voting","commit_id":"202319d0d4ed7086c941b365fd735279fd9d4601"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"ba09b48a751d797a94b64941cace444e6f74828c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3c89f731_1abb4804","updated":"2024-06-05 21:08:59.000000000","message":"looks good to me. the commit message is a little confusing talking about RE2/negative lookahead when this patch doesn\u0027t really deal with negative lookahead (only the one that it is fixing removed those lookaheads). But not a big deal.\n\nwould be awesome if it made it through the check queue.","commit_id":"202319d0d4ed7086c941b365fd735279fd9d4601"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"4b2b32d523e8d11db99ad933e5e373b21d12d081","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5938d0df_b135498a","in_reply_to":"3c89f731_1abb4804","updated":"2024-06-05 21:20:36.000000000","message":"oh right, if this fails i\u0027ll update the commit message","commit_id":"202319d0d4ed7086c941b365fd735279fd9d4601"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"998904cc8e4a04241bdadac3b771bdf2a41d8c7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"3e1df8a0_586cde37","updated":"2024-06-06 00:40:11.000000000","message":"FYI the last time these two jobs timed out was ~mid-May. We seem to be on a roll today, catching all possible sporadic failures. (Alternatively, something merged that we are not aware of yet that made all jobs slower. I prefer to think that is not the case... yet.)","commit_id":"ba6e34180fa9cd3616c99d7c765eb874028b216d"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"5738b5367e4b8801997bbf1152d4a87d4e858213","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a1826734_ce06c8a5","updated":"2024-06-06 00:38:47.000000000","message":"Updated commit message, hopefully this is better. Immediately sending back to gate.","commit_id":"ba6e34180fa9cd3616c99d7c765eb874028b216d"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ff460dd4eea00f9ba773ae7c68c3d9d62e374936","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1842ff05_9b32c29f","updated":"2024-06-06 06:12:48.000000000","message":"Rebased on https://review.opendev.org/c/openstack/neutron/+/921420 as this failed and was conflicting, hope it goes through this time.","commit_id":"11027e3e1ef9a58d5b2faa575a3764bd33cd2a08"}],"zuul.d/base.yaml":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3b2f2c4457870d11dd69e8f39fe2b634e73df623","unresolved":true,"context_lines":[{"line_number":35,"context_line":"      - ^roles/add_mariadb_repo/.*$"},{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"27351f16_9b8f4a90","line":38,"updated":"2024-06-04 22:30:54.000000000","message":"I actually start questioning the original regexes. Why would we ignore \"everything that is not project.yaml\" for jobs defined in base.yaml? Shouldn\u0027t each \"ignoring everything that is not XXX\" have XXX \u003d the name of the yaml file it\u0027s in?","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3de104d1e646d20289e5f6c3aa2828aed2d0b051","unresolved":true,"context_lines":[{"line_number":35,"context_line":"      - ^roles/add_mariadb_repo/.*$"},{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"af50743b_bf035921","line":38,"updated":"2024-06-04 19:36:00.000000000","message":"I\u0027m not sure if we can just use a \u0027files:\u0027 section above this to target a single file and ignore the rest here.","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"60b9f7339364f10c6972c61d68c5caac782f46f5","unresolved":true,"context_lines":[{"line_number":35,"context_line":"      - ^roles/add_mariadb_repo/.*$"},{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"b1c1c740_0597596e","line":38,"in_reply_to":"27351f16_9b8f4a90","updated":"2024-06-04 22:33:03.000000000","message":"Nevermind, I think commit message in https://review.opendev.org/c/openstack/neutron/+/827615 explains why it\u0027s project.yaml everywhere (the actual jobs are listed there).","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"a3fae3628a2a83f0019a480f31ad72bab4fac84e","unresolved":true,"context_lines":[{"line_number":35,"context_line":"      - ^roles/add_mariadb_repo/.*$"},{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"2e702f5e_3c43a6e1","line":38,"in_reply_to":"af50743b_bf035921","updated":"2024-06-04 22:36:52.000000000","message":"Not sure. The doc says \"This indicates that the job should only run on changes where the specified files are modified.\" This \"only\" seems to say that if `files` is used, then unless one of the explicitly listed files is touched, the job will NOT run.","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"d8140008abebd29a3a417f48402c9a52bc2eb3d4","unresolved":true,"context_lines":[{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"},{"line_number":42,"context_line":"      - ^zuul.d/rally.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9cb399a3_91e40aef","line":39,"updated":"2024-06-04 22:41:24.000000000","message":"I think there\u0027s a way to define a variable and then reuse it with \u0026/* syntax, but I don\u0027t know much about zuul to come up with exact recipe how to write it.","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"e764624e208adb4a91e5cb04914c2e1cfc8a2746","unresolved":true,"context_lines":[{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"},{"line_number":42,"context_line":"      - ^zuul.d/rally.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"540e9614_5324ebad","line":39,"in_reply_to":"1f83d2e2_862ddf28","updated":"2024-06-05 14:18:17.000000000","message":"you can try but AFAIK that don\u0027t work with list items atleast when i last checked years back. May be things changed since then.","commit_id":"f989a287a370935c91051621d3067ac824882f1f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"47cf020f97214416381f0ba5c5fd086f108c0f65","unresolved":true,"context_lines":[{"line_number":36,"context_line":"      - ^roles/nftables/.*$"},{"line_number":37,"context_line":"      - ^rally-jobs/.*$"},{"line_number":38,"context_line":"      # Ignore everything except for zuul.d/project.yaml"},{"line_number":39,"context_line":"      - ^zuul.d/base.yaml"},{"line_number":40,"context_line":"      - ^zuul.d/grenade.yaml"},{"line_number":41,"context_line":"      - ^zuul.d/job-templates.yaml"},{"line_number":42,"context_line":"      - ^zuul.d/rally.yaml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1f83d2e2_862ddf28","line":39,"in_reply_to":"9cb399a3_91e40aef","updated":"2024-06-05 13:59:33.000000000","message":"Yes, and we do use the \u0026/* syntax even in this definition, we just always do it a whole section at a time. There is a way to insert something in a section assuming it\u0027s defined first. For example, if we had a base definition like:\n\nname: haleyb_base\n    haleyb_files: \u0026haleyb_files\n        - ^zuul.d/base.yaml\n\nWe could then make another job like this:\n\nname: haleyb_job\n    haleyb_files:\n        - ^zuul.d/grenade.yaml\n        \u003c\u003c: *haleyb_files\n\nIt doesn\u0027t look pretty but should work to clean things up a bit.\n\nI can try to do that after we fix the gate :)","commit_id":"f989a287a370935c91051621d3067ac824882f1f"}]}