)]}'
{"kuryr_kubernetes/config.py":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"a2cb10329864f9bfb313e2d8dd7421aa78e40159","unresolved":true,"context_lines":[{"line_number":253,"context_line":"                help\u003d_(\"A mapping of default subnets for certain driverType \""},{"line_number":254,"context_line":"                       \"in a form of \u003cdriverType\u003e:\u003cSUBNET-ID\u003e\"),"},{"line_number":255,"context_line":"                default\u003d{}),"},{"line_number":256,"context_line":"    cfg.ListOpt(\u0027resource_tags\u0027,"},{"line_number":257,"context_line":"                help\u003d_(\"List of tags that will be applied to all OpenStack \""},{"line_number":258,"context_line":"                       \"(Neutron and Octavia) resources created by Kuryr. \""},{"line_number":259,"context_line":"                       \"This can be used to identify and garbage-collect \""}],"source_content_type":"text/x-python","patch_set":2,"id":"886e05d5_1c15ef4e","line":256,"updated":"2022-05-11 09:41:57.000000000","message":"why couldn\u0027t we use this config to identify the cluster instead of adding a new one?","commit_id":"34f7508904a34a0851c284da8e89286fe3a6a902"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"7a9fedf2a6968ef6e93906657d79da1a69377b58","unresolved":true,"context_lines":[{"line_number":253,"context_line":"                help\u003d_(\"A mapping of default subnets for certain driverType \""},{"line_number":254,"context_line":"                       \"in a form of \u003cdriverType\u003e:\u003cSUBNET-ID\u003e\"),"},{"line_number":255,"context_line":"                default\u003d{}),"},{"line_number":256,"context_line":"    cfg.ListOpt(\u0027resource_tags\u0027,"},{"line_number":257,"context_line":"                help\u003d_(\"List of tags that will be applied to all OpenStack \""},{"line_number":258,"context_line":"                       \"(Neutron and Octavia) resources created by Kuryr. \""},{"line_number":259,"context_line":"                       \"This can be used to identify and garbage-collect \""}],"source_content_type":"text/x-python","patch_set":2,"id":"f708ac0b_c8816665","line":256,"in_reply_to":"3cb7fdcc_829af3c3","updated":"2022-05-12 16:32:52.000000000","message":"In general cluster_id is a cool, semantic idea, but obviously when it comes to configuration options the less, the better. The point is where we want it to be used. If it\u0027s to be used as some tag or even description field, then fine, anything goes. But if we want to put that into device_id, then we\u0027ll better have something that\u0027s resembling an ID.","commit_id":"34f7508904a34a0851c284da8e89286fe3a6a902"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"800048316da3c67b64534a47bf25913524126f2b","unresolved":true,"context_lines":[{"line_number":253,"context_line":"                help\u003d_(\"A mapping of default subnets for certain driverType \""},{"line_number":254,"context_line":"                       \"in a form of \u003cdriverType\u003e:\u003cSUBNET-ID\u003e\"),"},{"line_number":255,"context_line":"                default\u003d{}),"},{"line_number":256,"context_line":"    cfg.ListOpt(\u0027resource_tags\u0027,"},{"line_number":257,"context_line":"                help\u003d_(\"List of tags that will be applied to all OpenStack \""},{"line_number":258,"context_line":"                       \"(Neutron and Octavia) resources created by Kuryr. \""},{"line_number":259,"context_line":"                       \"This can be used to identify and garbage-collect \""}],"source_content_type":"text/x-python","patch_set":2,"id":"3cb7fdcc_829af3c3","line":256,"in_reply_to":"886e05d5_1c15ef4e","updated":"2022-05-11 11:52:47.000000000","message":"Technically we could use that, why not. Except default value is an empty list which can be even more confusing for besides the section is on neutron side, not on kuryr-kubernetes.","commit_id":"34f7508904a34a0851c284da8e89286fe3a6a902"}]}