)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"08cfb83fffad6a097327dbf1022131a1384bb7da","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    /zuul/\u003ctenant\u003e/pipeline/\u003cpipeline\u003e/queue/\u003clayout-uuid\u003e/\u003cqueue-uuid\u003e"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"TODO:"},{"line_number":14,"context_line":"* cleanup leaked queues for old layouts"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I0a64bd9adc7b9f8f7a775280bb7a01ace22baac4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"2195d8df_c1beece0","line":14,"updated":"2021-10-05 23:12:35.000000000","message":"Should we just do this at the end of pipeline processing?","commit_id":"335568ea7a454fe0d6164dca27991e6915b06b24"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"39bd64dbddfdf554c472aeab4e03a47de832ab0d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    /zuul/\u003ctenant\u003e/pipeline/\u003cpipeline\u003e/queue/\u003clayout-uuid\u003e/\u003cqueue-uuid\u003e"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"TODO:"},{"line_number":14,"context_line":"* cleanup leaked queues for old layouts"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I0a64bd9adc7b9f8f7a775280bb7a01ace22baac4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"a3752cd2_a868b03e","line":14,"updated":"2021-10-25 20:50:15.000000000","message":"I believe this is done in https://review.opendev.org/c/zuul/zuul/+/814773/8/zuul/scheduler.py#1269","commit_id":"b7d89ce2193cae4da526d2b41cb04c2937518026"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"d39baa76a2dcd4fc9e8dfcc93019c1aedae9f580","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    /zuul/\u003ctenant\u003e/pipeline/\u003cpipeline\u003e/queue/\u003clayout-uuid\u003e/\u003cqueue-uuid\u003e"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"TODO:"},{"line_number":14,"context_line":"* cleanup leaked queues for old layouts"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I0a64bd9adc7b9f8f7a775280bb7a01ace22baac4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"9972d09f_29141c8c","line":14,"in_reply_to":"a3752cd2_a868b03e","updated":"2021-10-26 05:07:16.000000000","message":"Yes, this is done. Forgot to remote the TODO on the last rebase.","commit_id":"b7d89ce2193cae4da526d2b41cb04c2937518026"}],"zuul/model.py":[{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"bbca636bddb07a34747f5e556ee87f92ae1e0188","unresolved":true,"context_lines":[{"line_number":544,"context_line":"        data.update({"},{"line_number":545,"context_line":"            \"_jobs\": set(data[\"_jobs\"]),"},{"line_number":546,"context_line":"            \"queue\": queue,"},{"line_number":547,"context_line":"            \"project_branches\": [tuple(pb) for pb in self.project_branches],"},{"line_number":548,"context_line":"        })"},{"line_number":549,"context_line":"        return data"},{"line_number":550,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"be54aa9f_1c400b6c","line":547,"updated":"2021-10-20 11:59:11.000000000","message":"This should be:\n\n  \"project_branches\": [tuple(pb) for pb in data[\"project_branches\"]],","commit_id":"1d0c01a1994db924a01a0a60e6e51eecefa133f6"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"2e5b06a96374a6774dcb975e36b6215772dee2fc","unresolved":false,"context_lines":[{"line_number":544,"context_line":"        data.update({"},{"line_number":545,"context_line":"            \"_jobs\": set(data[\"_jobs\"]),"},{"line_number":546,"context_line":"            \"queue\": queue,"},{"line_number":547,"context_line":"            \"project_branches\": [tuple(pb) for pb in self.project_branches],"},{"line_number":548,"context_line":"        })"},{"line_number":549,"context_line":"        return data"},{"line_number":550,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"f1ee8e7b_d4f3afb2","line":547,"in_reply_to":"be54aa9f_1c400b6c","updated":"2021-10-20 12:49:16.000000000","message":"Done","commit_id":"1d0c01a1994db924a01a0a60e6e51eecefa133f6"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"d928452a3a2907f7f1f789a7419e3d14ef73b731","unresolved":true,"context_lines":[{"line_number":550,"context_line":""},{"line_number":551,"context_line":"    def getPath(self):"},{"line_number":552,"context_line":"        pipeline_path \u003d self.pipeline.state.getPath()"},{"line_number":553,"context_line":"        layout_uuid \u003d self.pipeline.tenant.layout.uuid"},{"line_number":554,"context_line":"        return f\"{pipeline_path}/queue/{layout_uuid}/{self.uuid}\""},{"line_number":555,"context_line":""},{"line_number":556,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":6,"id":"18e3ac9a_dd9e229d","line":553,"updated":"2021-10-20 12:13:18.000000000","message":"I think there is no need to include the layout UUID in the path.","commit_id":"1d0c01a1994db924a01a0a60e6e51eecefa133f6"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"2e5b06a96374a6774dcb975e36b6215772dee2fc","unresolved":false,"context_lines":[{"line_number":550,"context_line":""},{"line_number":551,"context_line":"    def getPath(self):"},{"line_number":552,"context_line":"        pipeline_path \u003d self.pipeline.state.getPath()"},{"line_number":553,"context_line":"        layout_uuid \u003d self.pipeline.tenant.layout.uuid"},{"line_number":554,"context_line":"        return f\"{pipeline_path}/queue/{layout_uuid}/{self.uuid}\""},{"line_number":555,"context_line":""},{"line_number":556,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa19e8ab_97d23b29","line":553,"in_reply_to":"18e3ac9a_dd9e229d","updated":"2021-10-20 12:49:16.000000000","message":"Done","commit_id":"1d0c01a1994db924a01a0a60e6e51eecefa133f6"}]}
