)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Sharpz7 \u003cadam.mcarthur62@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-05 21:31:19 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Multi-transport support for routing backend"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is currently no way to multiplex oslo notifications to multiple"},{"line_number":10,"context_line":"backend queues, which greatly limits operator flexibility in cases where"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fd7e2028_baaed28d","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":15},"updated":"2024-12-06 09:33:58.000000000","message":"We normally indicate the name of the deliverable related to this proposal, in your case oslo.messaging I guess.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"decbebe6923cdd397bf78c0d430b5b06416a1177","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sharpz7 \u003cadam.mcarthur62@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-05 21:31:19 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Multi-transport support for routing backend"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is currently no way to multiplex oslo notifications to multiple"},{"line_number":10,"context_line":"backend queues, which greatly limits operator flexibility in cases where"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3cdeeb51_83a3750d","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":15},"in_reply_to":"f5ae04aa_32faf306","updated":"2024-12-23 12:21:19.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"fe3c5618b128bcfc315275e613ee155536d7153d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Sharpz7 \u003cadam.mcarthur62@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-05 21:31:19 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Multi-transport support for routing backend"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is currently no way to multiplex oslo notifications to multiple"},{"line_number":10,"context_line":"backend queues, which greatly limits operator flexibility in cases where"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"f5ae04aa_32faf306","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":15},"in_reply_to":"fd7e2028_baaed28d","updated":"2024-12-06 09:38:04.000000000","message":"Example: `oslo.messaging: Multi-transport support for routing backend`","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d189d6ac_748bdf88","updated":"2024-12-06 09:33:58.000000000","message":"Hey guys, thanks for this proposal.\nit looks interesting.\nHere are my first comments.\n\nI\u0027ll ping John Eckersberg to get is messaging expert eyes on it.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"6b98d474b8273e8af61c4decd7b38675ace49daf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"13fbf914_602f2090","updated":"2024-12-12 22:01:22.000000000","message":"Thanks for the initial review, myself or Adam will be revising soon.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"decbebe6923cdd397bf78c0d430b5b06416a1177","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f580ef16_0561fd11","in_reply_to":"13fbf914_602f2090","updated":"2024-12-23 12:21:19.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"267e5e92fb1acb8804f5513c41fff2ecde39751e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"86ade2fb_36626d1a","updated":"2025-02-06 20:27:53.000000000","message":"Adam and I are actively working on a prototype of this so we can make sure we have the right shape for the feature.","commit_id":"b4d711e9b5545092343c36e8a3061b92fe8867fc"}],"specs/dalmatian/multi-store-with-notification-routing-backend.rst":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"``oslo.messaging`` allows to specify the drivers and notifications transports"},{"line_number":20,"context_line":"in each components config files like below For example, in nova."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":".. code-block:: ini"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"d4d6e679_e38cb7d6","line":20,"range":{"start_line":20,"start_character":43,"end_line":20,"end_character":46},"updated":"2024-12-06 09:33:58.000000000","message":"A dot is missing?","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"d1b270c2aef1f72f5a7d1e4a776b7897a28ce21c","unresolved":false,"context_lines":[{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"``oslo.messaging`` allows to specify the drivers and notifications transports"},{"line_number":20,"context_line":"in each components config files like below For example, in nova."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":".. code-block:: ini"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"948bf0e3_9aa3bedc","line":20,"range":{"start_line":20,"start_character":43,"end_line":20,"end_character":46},"in_reply_to":"d4d6e679_e38cb7d6","updated":"2024-12-23 12:10:48.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":37,"context_line":"      log:"},{"line_number":38,"context_line":"          accepted_events: [\u0027compute.instance.exists\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"While this allows routing different events to different oslo.messaging"},{"line_number":41,"context_line":"backends, it does not provide a mechanism for configuring different versions"},{"line_number":42,"context_line":"of the same backend -- such as different ``transport_urls`` to support"},{"line_number":43,"context_line":"multiple different notification locations."}],"source_content_type":"text/x-rst","patch_set":3,"id":"66b8985d_3fe048ab","line":40,"range":{"start_line":40,"start_character":1,"end_line":40,"end_character":42},"updated":"2024-12-06 09:33:58.000000000","message":"Your example above only reflect one event, right?","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"d1b270c2aef1f72f5a7d1e4a776b7897a28ce21c","unresolved":false,"context_lines":[{"line_number":37,"context_line":"      log:"},{"line_number":38,"context_line":"          accepted_events: [\u0027compute.instance.exists\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"While this allows routing different events to different oslo.messaging"},{"line_number":41,"context_line":"backends, it does not provide a mechanism for configuring different versions"},{"line_number":42,"context_line":"of the same backend -- such as different ``transport_urls`` to support"},{"line_number":43,"context_line":"multiple different notification locations."}],"source_content_type":"text/x-rst","patch_set":3,"id":"f235ed1d_80681146","line":40,"range":{"start_line":40,"start_character":1,"end_line":40,"end_character":42},"in_reply_to":"66b8985d_3fe048ab","updated":"2024-12-23 12:10:48.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"54caa42cc26af6300c88d9099c0f74b106cc431a","unresolved":true,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":".. code-block:: yaml"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"  group_1:"},{"line_number":80,"context_line":"      messaging_rabbit001:"},{"line_number":81,"context_line":"          accepted_priorities: [\u0027info\u0027, \u0027warn\u0027, \u0027error\u0027]"},{"line_number":82,"context_line":"      messaging_kafka001:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"2dd66cbe_50f6b1ff","line":79,"range":{"start_line":79,"start_character":2,"end_line":79,"end_character":9},"updated":"2024-12-23 01:12:01.000000000","message":"Could you describe how it should behave if the corresponding key does not exist in routing_transport_urls ? (I think it should cause the process to exit)","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"d1b270c2aef1f72f5a7d1e4a776b7897a28ce21c","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":".. code-block:: yaml"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"  group_1:"},{"line_number":80,"context_line":"      messaging_rabbit001:"},{"line_number":81,"context_line":"          accepted_priorities: [\u0027info\u0027, \u0027warn\u0027, \u0027error\u0027]"},{"line_number":82,"context_line":"      messaging_kafka001:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1fd9e0bc_f482ddb4","line":79,"range":{"start_line":79,"start_character":2,"end_line":79,"end_character":9},"in_reply_to":"2dd66cbe_50f6b1ff","updated":"2024-12-23 12:10:48.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":110,"context_line":"Both of these solutions are significantly more complex for operators and"},{"line_number":111,"context_line":"add additional points of failure. For this reason, they are not acceptable."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Impact on Existing APIs"},{"line_number":114,"context_line":"-----------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"No impact."}],"source_content_type":"text/x-rst","patch_set":3,"id":"daf387f0_96745e53","line":113,"range":{"start_line":113,"start_character":10,"end_line":113,"end_character":23},"updated":"2024-12-06 09:33:58.000000000","message":"I think we need more details about connections.\noslo.messaging comes with various options related to rpc connections, is one pool fit all our needs?\nIs things from various backend will be mixed in one pool?\n\nhttps://docs.openstack.org/oslo.messaging/latest/configuration/opts.html#default","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"decbebe6923cdd397bf78c0d430b5b06416a1177","unresolved":true,"context_lines":[{"line_number":110,"context_line":"Both of these solutions are significantly more complex for operators and"},{"line_number":111,"context_line":"add additional points of failure. For this reason, they are not acceptable."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Impact on Existing APIs"},{"line_number":114,"context_line":"-----------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"No impact."}],"source_content_type":"text/x-rst","patch_set":3,"id":"e680f140_cf16c6b9","line":113,"range":{"start_line":113,"start_character":10,"end_line":113,"end_character":23},"in_reply_to":"3f27a8ef_96495876","updated":"2024-12-23 12:21:19.000000000","message":"Definitely agree that the API will change - sorry, that is us having our ironic-hats on.\n\nI feel that one pool might be sufficient - but lets continue discussing in the \"Performance Impact\" section - my question there will hopefully help me understand better.\n\nOn the config point, I think that makes a lot of sense, to make it easier to make sure you have defined all the transport_urls you need in every routing_config. Its a bit chicken-and-egg like at present.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"54caa42cc26af6300c88d9099c0f74b106cc431a","unresolved":true,"context_lines":[{"line_number":110,"context_line":"Both of these solutions are significantly more complex for operators and"},{"line_number":111,"context_line":"add additional points of failure. For this reason, they are not acceptable."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Impact on Existing APIs"},{"line_number":114,"context_line":"-----------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"No impact."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3f27a8ef_96495876","line":113,"range":{"start_line":113,"start_character":10,"end_line":113,"end_character":23},"in_reply_to":"daf387f0_96745e53","updated":"2024-12-23 01:12:01.000000000","message":"I have the same question here. If we aim to tune a few other options for different messaing backends separately then we may need a different mechanism such as having dedicated config sections to define transport_url specific backend.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":113,"context_line":"Impact on Existing APIs"},{"line_number":114,"context_line":"-----------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"No impact."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"Security impact"},{"line_number":119,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"0f4ae5d2_bb46f24b","line":116,"range":{"start_line":116,"start_character":0,"end_line":116,"end_character":10},"updated":"2024-12-06 09:33:58.000000000","message":"I\u0027m not convinced that this new concept will have no impact on the existing API, by example what will happen to `get_rpc_transport`?\n\nIt seems to me that this function is designed to only work with one backend, meaning that if we introduce this kind of change, we will surely need some adaptation of this logic.\n\nhttps://opendev.org/openstack/oslo.messaging/src/branch/master/oslo_messaging/rpc/transport.py#L24-L52","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"decbebe6923cdd397bf78c0d430b5b06416a1177","unresolved":false,"context_lines":[{"line_number":113,"context_line":"Impact on Existing APIs"},{"line_number":114,"context_line":"-----------------------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"No impact."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"Security impact"},{"line_number":119,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"23f1bdca_3d4bd8dd","line":116,"range":{"start_line":116,"start_character":0,"end_line":116,"end_character":10},"in_reply_to":"0f4ae5d2_bb46f24b","updated":"2024-12-23 12:21:19.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":123,"context_line":"Performance Impact"},{"line_number":124,"context_line":"------------------"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"Maintaining a connection to multiple different messaging backends, and sending"},{"line_number":127,"context_line":"notifications to each, may cause a small performance impact for those utilizing"},{"line_number":128,"context_line":"this feature."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Configuration Impact"},{"line_number":131,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bd5a7bfe_69bd9917","line":128,"range":{"start_line":126,"start_character":0,"end_line":128,"end_character":13},"updated":"2024-12-06 09:33:58.000000000","message":"At the process layer, I suppose we will have a number of threads equal to the number of backend connections we have, exact?\n\nPlease can you develop this aspect.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"bdfbdec0797a7b6e903eabae2ad6e0288166122a","unresolved":true,"context_lines":[{"line_number":123,"context_line":"Performance Impact"},{"line_number":124,"context_line":"------------------"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"Maintaining a connection to multiple different messaging backends, and sending"},{"line_number":127,"context_line":"notifications to each, may cause a small performance impact for those utilizing"},{"line_number":128,"context_line":"this feature."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Configuration Impact"},{"line_number":131,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"95aed1e1_647de890","line":128,"range":{"start_line":126,"start_character":0,"end_line":128,"end_character":13},"in_reply_to":"252a3b6e_7b827099","updated":"2025-01-30 08:43:08.000000000","message":"I guess that yes, 1 thread by backend connection.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"decbebe6923cdd397bf78c0d430b5b06416a1177","unresolved":true,"context_lines":[{"line_number":123,"context_line":"Performance Impact"},{"line_number":124,"context_line":"------------------"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"Maintaining a connection to multiple different messaging backends, and sending"},{"line_number":127,"context_line":"notifications to each, may cause a small performance impact for those utilizing"},{"line_number":128,"context_line":"this feature."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Configuration Impact"},{"line_number":131,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"252a3b6e_7b827099","line":128,"range":{"start_line":126,"start_character":0,"end_line":128,"end_character":13},"in_reply_to":"bd5a7bfe_69bd9917","updated":"2024-12-23 12:21:19.000000000","message":"At present, based on this comment, I am assuming that there is only one thread per backend/driver combination at present? (i.e one thread for oslo_messaging_notify with messagingv2 driver, one for oslo_messaging_notify with the routing driver).\n\nWe would likely want to extend this to also be one thread per transport_url, which I think is the same as saying threads \u003d\u003d number of backend connections?","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Configuration Impact"},{"line_number":131,"context_line":"--------------------"},{"line_number":132,"context_line":"Adds ``[oslo_messaging_notifications]/routing_transport_urls`` as described"},{"line_number":133,"context_line":"above."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Developer Impact"},{"line_number":136,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"b907882a_3853bf5c","line":133,"range":{"start_line":132,"start_character":1,"end_line":133,"end_character":6},"updated":"2024-12-06 09:33:58.000000000","message":"What will happen to `transport_url`?\nDo we want to keep it or do we want to delegate everything at `routing_transport_urls`, and hence remove it?\n\nBy example if user give only one entry to `routing_transport_urls` then, we could consider it as the equivalent of `transport_url`.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"d1b270c2aef1f72f5a7d1e4a776b7897a28ce21c","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Configuration Impact"},{"line_number":131,"context_line":"--------------------"},{"line_number":132,"context_line":"Adds ``[oslo_messaging_notifications]/routing_transport_urls`` as described"},{"line_number":133,"context_line":"above."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Developer Impact"},{"line_number":136,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"f55c5bd0_6f7379b4","line":133,"range":{"start_line":132,"start_character":1,"end_line":133,"end_character":6},"in_reply_to":"12db9f0c_ac2cf7b9","updated":"2024-12-23 12:10:48.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"54caa42cc26af6300c88d9099c0f74b106cc431a","unresolved":true,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Configuration Impact"},{"line_number":131,"context_line":"--------------------"},{"line_number":132,"context_line":"Adds ``[oslo_messaging_notifications]/routing_transport_urls`` as described"},{"line_number":133,"context_line":"above."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Developer Impact"},{"line_number":136,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"12db9f0c_ac2cf7b9","line":133,"range":{"start_line":132,"start_character":1,"end_line":133,"end_character":6},"in_reply_to":"b907882a_3853bf5c","updated":"2024-12-23 01:12:01.000000000","message":"I guess we can keep `transport_url` for existing deployment with messaging driver. This may have minimum impact on existing deployments.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":140,"context_line":"Testing Impact"},{"line_number":141,"context_line":"--------------"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Additional unit tests will be required to cover the added functionality."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Implementation"},{"line_number":146,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9bccf8f0_a8ee2b7c","line":143,"range":{"start_line":143,"start_character":11,"end_line":143,"end_character":21},"updated":"2024-12-06 09:33:58.000000000","message":"And beyond that, I think we mostly need functional tests.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"54caa42cc26af6300c88d9099c0f74b106cc431a","unresolved":true,"context_lines":[{"line_number":140,"context_line":"Testing Impact"},{"line_number":141,"context_line":"--------------"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Additional unit tests will be required to cover the added functionality."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Implementation"},{"line_number":146,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"dee79685_8b1583ba","line":143,"range":{"start_line":143,"start_character":11,"end_line":143,"end_character":21},"in_reply_to":"9bccf8f0_a8ee2b7c","updated":"2024-12-23 01:12:01.000000000","message":"+1","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":36102,"name":"Adam McArthur","display_name":"Adam McArthur","email":"adam@mcaq.me","username":"adamcarthur","status":"PhD Candidate in Computational Imaging. Open Source Software Engineer at G-Research."},"change_message_id":"d1b270c2aef1f72f5a7d1e4a776b7897a28ce21c","unresolved":false,"context_lines":[{"line_number":140,"context_line":"Testing Impact"},{"line_number":141,"context_line":"--------------"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Additional unit tests will be required to cover the added functionality."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Implementation"},{"line_number":146,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"192b5d37_54116e05","line":143,"range":{"start_line":143,"start_character":11,"end_line":143,"end_character":21},"in_reply_to":"dee79685_8b1583ba","updated":"2024-12-23 12:10:48.000000000","message":"Done","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"a9acf856e4c693b45b335ae7a6191848253376b5","unresolved":true,"context_lines":[{"line_number":159,"context_line":"Milestones"},{"line_number":160,"context_line":"----------"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"* Bring routing driver documentation up to date"},{"line_number":163,"context_line":"* Add new feature and configuration option as described"},{"line_number":164,"context_line":"* Update routing driver documentation to reflect new feature"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"781cb0dd_3b544e75","line":162,"range":{"start_line":162,"start_character":0,"end_line":162,"end_character":47},"updated":"2024-12-06 09:33:58.000000000","message":"Many kafka option are already deprecated but not yet removed, do not we want a clean first to simplify the implementation step?\n\nExample https://docs.openstack.org/oslo.messaging/latest/configuration/opts.html#oslo_messaging_kafka.pool_size","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"54caa42cc26af6300c88d9099c0f74b106cc431a","unresolved":false,"context_lines":[{"line_number":159,"context_line":"Milestones"},{"line_number":160,"context_line":"----------"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"* Bring routing driver documentation up to date"},{"line_number":163,"context_line":"* Add new feature and configuration option as described"},{"line_number":164,"context_line":"* Update routing driver documentation to reflect new feature"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"5f83d043_e3aca4c0","line":162,"range":{"start_line":162,"start_character":0,"end_line":162,"end_character":47},"in_reply_to":"781cb0dd_3b544e75","updated":"2024-12-23 01:12:01.000000000","message":"This was addressed by https://review.opendev.org/c/openstack/oslo.messaging/+/937548 , I think.","commit_id":"ca283ea642fc16cc9b66bcbf35562d2bbba4fc6b"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"86dfb3c8925333888deac642ea6c0c219d68cfe1","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"There will be backwards-compatible changes to different functions within oslo-messaging."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"This will depend mostly on implementation choices."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Security impact"},{"line_number":128,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"990b1743_8a463755","line":125,"updated":"2025-01-24 20:18:07.000000000","message":"We will need to discover and enumerate those implementation choices. It\u0027s OK if we need to prototype to know for sure.","commit_id":"b4d711e9b5545092343c36e8a3061b92fe8867fc"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"4d99b4dbeb377ebaf54965cef843c222734653d3","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"There will be backwards-compatible changes to different functions within oslo-messaging."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"This will depend mostly on implementation choices."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Security impact"},{"line_number":128,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"2fb3b2ee_3922c1fc","line":125,"in_reply_to":"990b1743_8a463755","updated":"2025-01-30 08:52:38.000000000","message":"Where? In this specification?","commit_id":"b4d711e9b5545092343c36e8a3061b92fe8867fc"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"86dfb3c8925333888deac642ea6c0c219d68cfe1","unresolved":true,"context_lines":[{"line_number":142,"context_line":"above."},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"The existing ``[oslo_messaging_notifications]/routing_transport_url`` will be kept, to maintain"},{"line_number":145,"context_line":"backwards compatibility."},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"Developer Impact"},{"line_number":148,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"880d9386_5eab0897","line":145,"updated":"2025-01-24 20:18:07.000000000","message":"It\u0027s worth noting as well that routing_transport_urls will *only apply to the routing backend*. transport_url will still be the path used by 99.99% of users not using the routing backend.","commit_id":"b4d711e9b5545092343c36e8a3061b92fe8867fc"}]}