)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"9b34e764be96ed7c3332067a339a11e1ff095852","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"39c44f02_eb9f7c7a","updated":"2021-11-29 21:43:18.000000000","message":"Hi, sorry to just pop in out of history and -1. I am triggered by gearman stuff. ;)\n\nThe options being removed will be hard to find on the web and even when people look at the docs in the git repo. The release note tells us that gearman was removed, so that\u0027s good.. but.. still seems like it will be confusing for current users reading the docs before they upgrade.","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"7d3ef32c8f153330a6a1f5e047c63817291aabf2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c57f690c_6b8fa1db","updated":"2021-12-03 17:59:55.000000000","message":"Spoke with Jim offline. I see that there are versioned docs available and that this is normal practice. It would be very helpful if somebody could add a version index to the zuul-ci.org website at some point. Until then, lists and reminders are what we have. Thanks for working on this!","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"}],"doc/source/discussion/components.rst":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"2e50064fb2f6a84a5b5d6c709917f341aa58be28","unresolved":false,"context_lines":[{"line_number":145,"context_line":"The following sections of ``zuul.conf`` are used by all Zuul components:"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":".. attr:: gearman"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"   Client connection information for Gearman."},{"line_number":151,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9ace0f16_432d203d","side":"PARENT","line":148,"updated":"2021-11-29 23:48:55.000000000","message":"We have versioned docs for this purpose.  So someone using 4.10.4 can visit https://zuul-ci.org/docs/zuul/4.10.4/discussion/components.html for their version.","commit_id":"35ffde958ad3362b48deadf52061bb212224c251"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"9b34e764be96ed7c3332067a339a11e1ff095852","unresolved":true,"context_lines":[{"line_number":145,"context_line":"The following sections of ``zuul.conf`` are used by all Zuul components:"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":".. attr:: gearman"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"   Client connection information for Gearman."},{"line_number":151,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"43b7f3d4_13a6f2c9","side":"PARENT","line":148,"updated":"2021-11-29 21:43:18.000000000","message":"Wouldn\u0027t it make sense to keep these in the docs a while as \"removed after version X.XX\"? Just so that people can still find the docs?","commit_id":"35ffde958ad3362b48deadf52061bb212224c251"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f74d2608ecb6b70216775ed3a2f077d8b2bdfd10","unresolved":false,"context_lines":[{"line_number":55,"context_line":"Each of the Zuul processes may run on the same host, or different"},{"line_number":56,"context_line":"hosts."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Zuul requires an external ZooKeeper cluster, and all Zuul and Nodepool"},{"line_number":59,"context_line":"components need to be able to connect to the hosts in that cluster on"},{"line_number":60,"context_line":"a TLS-encrypted TCP port, typically 2281."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a9b23cd_866a6a0b","line":58,"updated":"2022-01-17 20:58:10.000000000","message":"Agreed; I think you fixed this in a different patch so doesn\u0027t need to be updated here, just rebased.","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"d24c9fb83ee1f58feaaa876c70294a9451d3246f","unresolved":true,"context_lines":[{"line_number":55,"context_line":"Each of the Zuul processes may run on the same host, or different"},{"line_number":56,"context_line":"hosts."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Zuul requires an external ZooKeeper cluster, and all Zuul and Nodepool"},{"line_number":59,"context_line":"components need to be able to connect to the hosts in that cluster on"},{"line_number":60,"context_line":"a TLS-encrypted TCP port, typically 2281."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7195d261_0eba6f6b","line":58,"range":{"start_line":58,"start_character":26,"end_line":58,"end_character":43},"updated":"2022-01-13 00:35:40.000000000","message":"Nit maybe call out it needs to be ZK \u003e\u003d 3.5.1. This is related to some feedback we got via storyboard saying ubuntu packages are older and this has tripped people up.","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f271bbaa9295d35d452cfd020de288c69006e1f4","unresolved":false,"context_lines":[{"line_number":55,"context_line":"Each of the Zuul processes may run on the same host, or different"},{"line_number":56,"context_line":"hosts."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Zuul requires an external ZooKeeper cluster, and all Zuul and Nodepool"},{"line_number":59,"context_line":"components need to be able to connect to the hosts in that cluster on"},{"line_number":60,"context_line":"a TLS-encrypted TCP port, typically 2281."},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"95f794f5_2dfacc0f","line":58,"updated":"2022-01-17 21:01:47.000000000","message":"Oh, actually, the components didn\u0027t get updated in that patch; I\u0027ll update it here.","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"}],"releasenotes/notes/gearmanectomy-ecec32b3764eafc1.yaml":[{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"5475171d21c17486b1198b99508a3e42818704a0","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Zuul no longer uses gearman.  It is no longer to required to run a"},{"line_number":5,"context_line":"    Gearman server, and the Zuul scheduler no longer does so.  All"},{"line_number":6,"context_line":"    gearman-related settings in zuul.conf files are now ignored and"},{"line_number":7,"context_line":"    may be removed."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"da2b8d5e_2f754540","line":4,"updated":"2021-12-08 15:31:39.000000000","message":"... no longer required ... ?","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"146402404d442ded4f5a48cfb5847b41eac4dc5c","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Zuul no longer uses gearman.  It is no longer to required to run a"},{"line_number":5,"context_line":"    Gearman server, and the Zuul scheduler no longer does so.  All"},{"line_number":6,"context_line":"    gearman-related settings in zuul.conf files are now ignored and"},{"line_number":7,"context_line":"    may be removed."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d38e7ae6_578a0fb9","line":4,"in_reply_to":"da2b8d5e_2f754540","updated":"2022-01-25 23:54:07.000000000","message":"Yes \"no longer required\" would be correct. I think it is worth fixing this since this is a major part of the zuulv5 upgrade.","commit_id":"303da8ed083db4b75484dac92aa6ae2fac2a7a6a"}]}