)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8016b427_bbcdd17b","updated":"2025-12-09 17:14:19.000000000","message":"Great start on these much needed docs. I\u0027ve left a bunch of suggestions for improvements, happy to explain any of my comments in more detail if needed.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"91c6aa8d0ba5ac72eab62a3191c5488edec3df9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"883b88c4_3bb16177","updated":"2025-12-09 14:37:34.000000000","message":"recheck","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"96133382cdf9eb8efa1b337969deaa5e6e8bf2d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6cbbc09f_b5faeaa1","updated":"2025-12-31 09:31:31.000000000","message":"recheck","commit_id":"a8b62ab1aa11a2595dd08d63a592d1e2e780564b"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1af82477_6695a05d","updated":"2026-01-02 11:35:29.000000000","message":"Looks good thanks!  Some small typos to address.","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ed3b3682_e7482f9f","in_reply_to":"1af82477_6695a05d","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"cfd8d0d72f83ceec0a75a243cd7e89898c6314eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a04ec596_cd3aa9c9","updated":"2026-01-07 19:25:48.000000000","message":"Tidy work thanks Grzegorz","commit_id":"14d0b6506b2d191c575fbff1af490192cbffc9f4"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"7afcfae5e8e9fc47b8e09107d4cdfcb7ee754c4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d27e84d5_1b694f52","updated":"2026-01-07 14:44:54.000000000","message":"recheck","commit_id":"14d0b6506b2d191c575fbff1af490192cbffc9f4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"1bf02270bb693c5f0275c48eb0b00bc4999d489a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"66d832a1_e8594844","updated":"2026-01-08 09:49:31.000000000","message":"Can we move this to use oslo.config stanzas like other projects do? e.g. see https://docs.openstack.org/oslo.config/latest/reference/sphinxext.html","commit_id":"245c683f2a62d7d23ed3fa691e4ae2620586acf7"},{"author":{"_account_id":34076,"name":"Jakub Darmach","email":"jakub@stackhpc.com","username":"darmach"},"change_message_id":"4703031065ab1f25855e16054339d8c9cc691f86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"3c191a46_c909c6ae","updated":"2026-01-08 09:42:55.000000000","message":"Looks all right, thanks!","commit_id":"245c683f2a62d7d23ed3fa691e4ae2620586acf7"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"9ee63145f1433b346a3f680d0cd15129768ac9fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0f93174a_4ef39471","in_reply_to":"66d832a1_e8594844","updated":"2026-01-16 12:13:12.000000000","message":"Done","commit_id":"245c683f2a62d7d23ed3fa691e4ae2620586acf7"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"df2c2200c9be93883dd7e94f4dc31ca9f5aa27a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"86a089bd_7a507385","updated":"2026-01-19 13:14:50.000000000","message":"recheck","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"1be82027_7ad0ff29","updated":"2026-01-19 14:38:44.000000000","message":"tox -e docs fails silently (and generates an empty reference) on my machine with:\n[oslo_config.sphinxconfiggen] reading config generator instructions from /Users/mnasiadka/dev/openstack/magnum-capi-helm/docs/doc/source/../../etc/oslo-config-generator/capi_helm.conf\n[oslo_config.sphinxconfiggen] writing sample configuration to /Users/mnasiadka/dev/openstack/magnum-capi-helm/docs/doc/source/_static/capi_helm.conf.sample\nWARNING:stevedore.named:Could not load capi_helm\nbuilding [mo]: targets for 0 po files that are out of date\nwriting output...\nbuilding [html]: targets for 6 source files that are out of date\nupdating environment: [new config] 6 added, 0 changed, 0 removed\nloading config file etc/oslo-config-generator/capi_helm.conf\nWARNING:stevedore.named:Could not load capi_helm\n\nAfter changing skip_install \u003d False in docs section in tox.ini - this gets rendered correctly","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d750066f_63a9d22c","in_reply_to":"1be82027_7ad0ff29","updated":"2026-01-22 15:03:58.000000000","message":"I removed skip_install \u003d True. Apparently it shouldn\u0027t be here in the first place.","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"e092c8f447ccb60f1501b93a56f856db6de701f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"97ff4d97_b9cd5ee6","updated":"2026-01-23 08:44:43.000000000","message":"recheck","commit_id":"9d2540a86eadf7cb9aa29a08b7c077e50f8f5e7c"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"7c1ef8df3a13170455b0dfc73428a8b698a74c36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3a17b808_35a2ac42","updated":"2026-01-23 08:46:07.000000000","message":"I built the documentation locally and the new config reference looks good, thanks.","commit_id":"64f897093148c3e7d1ec5a0da53da80144031e63"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"b46c7f7fc2ba48e23399c8c417e36ee068644c2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"1f7940e2_fca1301d","updated":"2026-01-28 11:59:48.000000000","message":"I think my preference is to roll all these changes into my previous commit, so the feature is more complete as one commit, but happy to go with the consensus of what people wanting this approach (I haven\u0027t been able to be involved in those discussions, just wondering why).","commit_id":"64f897093148c3e7d1ec5a0da53da80144031e63"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"019b450ae91f68577babd90d9ff7a1d6cba6dd7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"c5fecdb0_a2e87f7c","in_reply_to":"1f7940e2_fca1301d","updated":"2026-01-28 12:03:26.000000000","message":"\u003e I think my preference is to roll all these changes into my previous commit, so the feature is more complete as one commit, but happy to go with the consensus of what people wanting this approach (I haven\u0027t been able to be involved in those discussions, just wondering why).\n\nAh, ignore me, I see this is generally just fixing the docs. I actually think this one should get rebased on the master branch, so we can get it merged independently from the other patch.","commit_id":"64f897093148c3e7d1ec5a0da53da80144031e63"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"63d3065b8fd9844edd313d4fc350de211e581c15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"8a69a615_89eca6e1","updated":"2026-05-27 09:48:06.000000000","message":"LGTM","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"07fa29a87d723c41d743ecb09c1881237a03d59a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"c8418e23_a21328ee","updated":"2026-05-27 09:56:45.000000000","message":"Looks good, a few comments on the labels","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c9fb374051fad4a7c18873f48b4c5032f3efdc86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"f2216ea4_0a52fbd2","updated":"2026-05-28 15:17:05.000000000","message":"LGTM for now","commit_id":"60388e2da858f8f6fa6695cceaec0440d946ee77"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"69f78763f04b2f508f1aa0ca02a8a7620effaf11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"4dd3d76e_5f9f7ff4","updated":"2026-05-28 14:27:01.000000000","message":"recheck fixes landed","commit_id":"60388e2da858f8f6fa6695cceaec0440d946ee77"}],"doc/source/conf.py":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":31,"context_line":"openstackdocs_pdf_link \u003d True"},{"line_number":32,"context_line":"openstackdocs_use_storyboard \u003d False"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"# sphinxcontrib.apidoc options"},{"line_number":36,"context_line":"config_generator_config_file \u003d ["},{"line_number":37,"context_line":"    (\"../../etc/oslo-config-generator/capi_helm.conf\", \"_static/capi_helm\"),"}],"source_content_type":"text/x-python","patch_set":10,"id":"a70c0fbc_55e37cf7","line":34,"updated":"2026-01-19 14:38:44.000000000","message":"extra not needed newline","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":31,"context_line":"openstackdocs_pdf_link \u003d True"},{"line_number":32,"context_line":"openstackdocs_use_storyboard \u003d False"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"# sphinxcontrib.apidoc options"},{"line_number":36,"context_line":"config_generator_config_file \u003d ["},{"line_number":37,"context_line":"    (\"../../etc/oslo-config-generator/capi_helm.conf\", \"_static/capi_helm\"),"}],"source_content_type":"text/x-python","patch_set":10,"id":"b74d292c_f429e2a6","line":34,"in_reply_to":"a70c0fbc_55e37cf7","updated":"2026-01-22 15:03:58.000000000","message":"Done","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":37,"context_line":"    (\"../../etc/oslo-config-generator/capi_helm.conf\", \"_static/capi_helm\"),"},{"line_number":38,"context_line":"]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"# autodoc generation is a bit aggressive and a nuisance when doing heavy"},{"line_number":42,"context_line":"# text edit cycles."},{"line_number":43,"context_line":"# execute \"export SPHINX_DEBUG\u003d1\" in your terminal to disable"}],"source_content_type":"text/x-python","patch_set":10,"id":"97c0cafc_1cdfe69e","line":40,"updated":"2026-01-19 14:38:44.000000000","message":"ditto","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"6c42b09ff10c248803b1e1fc855e647b398f2ad6","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    (\"../../etc/oslo-config-generator/capi_helm.conf\", \"_static/capi_helm\"),"},{"line_number":38,"context_line":"]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"# autodoc generation is a bit aggressive and a nuisance when doing heavy"},{"line_number":42,"context_line":"# text edit cycles."},{"line_number":43,"context_line":"# execute \"export SPHINX_DEBUG\u003d1\" in your terminal to disable"}],"source_content_type":"text/x-python","patch_set":10,"id":"6383cbc1_e8067b29","line":40,"in_reply_to":"97c0cafc_1cdfe69e","updated":"2026-01-22 15:04:05.000000000","message":"Done","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"}],"doc/source/configuration/config-reference.rst":[{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    **default**: https://azimuth-cloud.github.io/capi-helm-charts"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"     Reference to the helm chart repository for"},{"line_number":38,"context_line":"     the cluster API driver."},{"line_number":39,"context_line":"     Note that if helm_chart_name starts with oci://"},{"line_number":40,"context_line":"     you will want this to set this to the empty string."},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"49fcef48_881564a1","line":38,"updated":"2025-12-09 17:14:19.000000000","message":"```suggestion\n     the CAPI Helm driver.\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    **default**: https://azimuth-cloud.github.io/capi-helm-charts"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"     Reference to the helm chart repository for"},{"line_number":38,"context_line":"     the cluster API driver."},{"line_number":39,"context_line":"     Note that if helm_chart_name starts with oci://"},{"line_number":40,"context_line":"     you will want this to set this to the empty string."},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"b41a15b6_cbda6b02","line":38,"in_reply_to":"49fcef48_881564a1","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":37,"context_line":"     Reference to the helm chart repository for"},{"line_number":38,"context_line":"     the cluster API driver."},{"line_number":39,"context_line":"     Note that if helm_chart_name starts with oci://"},{"line_number":40,"context_line":"     you will want this to set this to the empty string."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"     Related options:"},{"line_number":43,"context_line":"        * helm_chart_name"}],"source_content_type":"text/x-rst","patch_set":3,"id":"29d276e8_039afdb3","line":40,"updated":"2025-12-09 17:14:19.000000000","message":"Should we be more direct here?\n```suggestion\n     this must be the empty string.\n```\nDo things break if it\u0027s not set to an empty string?","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":37,"context_line":"     Reference to the helm chart repository for"},{"line_number":38,"context_line":"     the cluster API driver."},{"line_number":39,"context_line":"     Note that if helm_chart_name starts with oci://"},{"line_number":40,"context_line":"     you will want this to set this to the empty string."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"     Related options:"},{"line_number":43,"context_line":"        * helm_chart_name"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3dbb31ac_66f12daf","line":40,"in_reply_to":"29d276e8_039afdb3","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    **default**: 2048"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    Minimum RAM for flavor used to create a Kubernetes node."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"**minimum_flavor_vcpus**"}],"source_content_type":"text/x-rst","patch_set":3,"id":"cf0617bd_93a498e0","line":99,"updated":"2025-12-09 17:14:19.000000000","message":"```suggestion\n    Minimum RAM (in MB units) for flavor used to create a Kubernetes node.\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    **default**: 2048"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    Minimum RAM for flavor used to create a Kubernetes node."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"**minimum_flavor_vcpus**"}],"source_content_type":"text/x-rst","patch_set":3,"id":"af549796_7fa284ca","line":99,"in_reply_to":"cf0617bd_93a498e0","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    **type**: string"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    **Default**: StorageClass volume type for persistent volumes."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"**csi_cinder_reclaim_policy**"}],"source_content_type":"text/x-rst","patch_set":3,"id":"21523879_f30d6464","line":115,"updated":"2025-12-09 17:14:19.000000000","message":"```suggestion\n    **Default**: Cinder volume type for default StorageClass.\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    **type**: string"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    **Default**: StorageClass volume type for persistent volumes."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"**csi_cinder_reclaim_policy**"}],"source_content_type":"text/x-rst","patch_set":3,"id":"86e66b1d_435c9950","line":115,"in_reply_to":"21523879_f30d6464","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"**csi_cinder_allowed_topologies**"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    default\u003d[]"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"     Select the Nodes where the application"},{"line_number":142,"context_line":"     Pods may be scheduled based on Node labels."}],"source_content_type":"text/x-rst","patch_set":3,"id":"b25d1b2c_8d155528","line":139,"updated":"2025-12-09 17:14:19.000000000","message":"Some config options use `default\u003d` others use `**default**:` - we should choose 1 and stick to it for consistency.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"**csi_cinder_allowed_topologies**"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    default\u003d[]"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"     Select the Nodes where the application"},{"line_number":142,"context_line":"     Pods may be scheduled based on Node labels."}],"source_content_type":"text/x-rst","patch_set":3,"id":"029174fd_a80c5b5f","line":139,"in_reply_to":"b25d1b2c_8d155528","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":151,"context_line":"    Filesystem type for persistent volumes."},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"**csi_cinder_volume_binding_mode**"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    **type**: string"},{"line_number":157,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1cfe33db_e0943f0e","line":154,"updated":"2025-12-09 17:14:19.000000000","message":"This config option won\u0027t actually do anything without https://review.opendev.org/c/openstack/magnum-capi-helm/+/970106","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"d23cfe1c98fbd8469b25c49305eca2508a01e30d","unresolved":false,"context_lines":[{"line_number":151,"context_line":"    Filesystem type for persistent volumes."},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"**csi_cinder_volume_binding_mode**"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    **type**: string"},{"line_number":157,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"37627f74_3bddcc06","line":154,"in_reply_to":"1cfe33db_e0943f0e","updated":"2025-12-22 10:19:55.000000000","message":"Merged on 19 Dec, so now it has more sense.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":205,"context_line":""},{"line_number":206,"context_line":"    **type**: list"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    **default**::"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"      ["},{"line_number":211,"context_line":"        MachinesReady,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bb8df8e6_f0259178","line":208,"updated":"2025-12-09 17:14:19.000000000","message":"```suggestion\n    **default**:\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":205,"context_line":""},{"line_number":206,"context_line":"    **type**: list"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    **default**::"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"      ["},{"line_number":211,"context_line":"        MachinesReady,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"0d2dc93b_59d4f8db","line":208,"in_reply_to":"bb8df8e6_f0259178","updated":"2025-12-19 14:46:54.000000000","message":"It is needed to properly render code block in rst , and have \":\" in text.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"All configuration options listed bellow are used in"},{"line_number":6,"context_line":"[capi_help] stanza in magnum.conf file."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"**kubeconfig_file**"},{"line_number":9,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"a5dc125b_8d7392f8","line":6,"updated":"2026-01-02 11:35:29.000000000","message":"typo - capi_helm","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"All configuration options listed bellow are used in"},{"line_number":6,"context_line":"[capi_help] stanza in magnum.conf file."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"**kubeconfig_file**"},{"line_number":9,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"74d4afca_a9f3ec9b","line":6,"in_reply_to":"a5dc125b_8d7392f8","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":5,"context_line":"All configuration options listed bellow are used in"},{"line_number":6,"context_line":"[capi_helm] stanza in magnum.conf file."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":".. show-options::"},{"line_number":10,"context_line":"   :config-file: etc/oslo-config-generator/capi_helm.conf"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"6ef3ae0b_2c9cb4b4","line":8,"updated":"2026-01-19 14:38:44.000000000","message":"extra not needed newline","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":5,"context_line":"All configuration options listed bellow are used in"},{"line_number":6,"context_line":"[capi_helm] stanza in magnum.conf file."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":".. show-options::"},{"line_number":10,"context_line":"   :config-file: etc/oslo-config-generator/capi_helm.conf"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"ea48a67c_f0a1fb3d","line":8,"in_reply_to":"6ef3ae0b_2c9cb4b4","updated":"2026-01-22 15:03:58.000000000","message":"Done","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"}],"doc/source/configuration/index.rst":[{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":27,"context_line":"| Label                   | Default     | Possible values |"},{"line_number":28,"context_line":"+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":29,"context_line":"| monitoring_enabled      | false       | false           |"},{"line_number":30,"context_line":"|                         |             |  true           |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"93cad3e9_20e96403","line":27,"updated":"2025-12-09 17:14:19.000000000","message":"The driver now supports a bunch more labels than the ones in this table. If you search for `self._label` in `magnum_capi_helm/driver.py` you should find all the other labels that are supported so that we can add them here.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":27,"context_line":"| Label                   | Default     | Possible values |"},{"line_number":28,"context_line":"+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":29,"context_line":"| monitoring_enabled      | false       | false           |"},{"line_number":30,"context_line":"|                         |             |  true           |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"c0d77e7a_aba2de1c","line":27,"in_reply_to":"93cad3e9_20e96403","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":41,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":42,"context_line":"| capi_helm_chart_version | 0.10.1      | existing chart  |"},{"line_number":43,"context_line":"|                         |             | version         |"},{"line_number":44,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":45,"context_line":"| helm_extra_values       | filepath    | accessible file |"},{"line_number":46,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"**TODO: Add more recently supported labels here.**"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7db78f7d_fb9b5eee","line":46,"range":{"start_line":44,"start_character":1,"end_line":46,"end_character":2},"updated":"2025-12-09 17:14:19.000000000","message":"Unless I\u0027m misunderstanding the linked patch, I don\u0027t think the user-facing label is a filepath. In your Zabbix example below, the user facing label on the cluster template (but not on the cluster) would just be `zabbix` and then the mapping of that label to a filepath for a values override file exists in magnum.conf (and is therefore not user-visible).","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":42,"context_line":"| capi_helm_chart_version | 0.10.1      | existing chart  |"},{"line_number":43,"context_line":"|                         |             | version         |"},{"line_number":44,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":45,"context_line":"| helm_extra_values       | filepath    | accessible file |"},{"line_number":46,"context_line":"+-------------------------+-------------+-----------------+"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"**TODO: Add more recently supported labels here.**"}],"source_content_type":"text/x-rst","patch_set":3,"id":"693bdffc_43b13365","line":46,"range":{"start_line":44,"start_character":1,"end_line":46,"end_character":2},"in_reply_to":"7db78f7d_fb9b5eee","updated":"2025-12-19 14:46:54.000000000","message":"Brain fog, my bad. Fixed.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":52,"context_line":"* capi_helm_chart_version: can only be set via template property"},{"line_number":53,"context_line":"  and CAN\u0027T be overrided by cli options. If not set in cluster template,"},{"line_number":54,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":55,"context_line":"  If not found there it defaults to 0.10.1"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* helm_extra_values: used to point key in dictionary"},{"line_number":58,"context_line":"  mapping file with value ovverrides for clusters with this label."}],"source_content_type":"text/x-rst","patch_set":3,"id":"a9a492dc_c7fc8e44","line":55,"updated":"2025-12-09 17:14:19.000000000","message":"The default is specified in the config file and now also in config-reference.rst above, maybe we should remove this line to give us one less place to update when the version get\u0027s bumped?","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":52,"context_line":"* capi_helm_chart_version: can only be set via template property"},{"line_number":53,"context_line":"  and CAN\u0027T be overrided by cli options. If not set in cluster template,"},{"line_number":54,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":55,"context_line":"  If not found there it defaults to 0.10.1"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* helm_extra_values: used to point key in dictionary"},{"line_number":58,"context_line":"  mapping file with value ovverrides for clusters with this label."}],"source_content_type":"text/x-rst","patch_set":3,"id":"975dd4e5_84df29e3","line":55,"in_reply_to":"a9a492dc_c7fc8e44","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":53,"context_line":"  and CAN\u0027T be overrided by cli options. If not set in cluster template,"},{"line_number":54,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":55,"context_line":"  If not found there it defaults to 0.10.1"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* helm_extra_values: used to point key in dictionary"},{"line_number":58,"context_line":"  mapping file with value ovverrides for clusters with this label."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Example:"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"b509d6d4_81a2d938","line":58,"range":{"start_line":56,"start_character":0,"end_line":58,"end_character":66},"updated":"2025-12-09 17:14:19.000000000","message":"Suggested rewording for clarity plus we should add John\u0027s warning\nfrom his patch description to Magnum operators about being careful\nto avoid breaking changes in the Helm values overrides which have\nalready been used on existing clusters.\n\n```suggestion\n* helm_extra_values: Allows operators to map arbitrary cluster \ntemplate labels to snippets of Helm value overrides. See example\nbelow for more details.\n\nAs an operator, you **must** be careful to not make changes\nto existing clusters that are not backwards compatible.\nAnd you must not remove any configuration that applies to\nexisting templates, unless that is backwards compatible.\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":54,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":55,"context_line":"  If not found there it defaults to 0.10.1"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* helm_extra_values: used to point key in dictionary"},{"line_number":58,"context_line":"  mapping file with value ovverrides for clusters with this label."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Example:"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fde4c655_29dabcdf","line":58,"range":{"start_line":57,"start_character":24,"end_line":58,"end_character":24},"updated":"2025-12-09 17:14:19.000000000","message":"Suggested rewording for clarity plus we should add John\u0027s warning\nfrom his patch description to Magnum operators about being careful\nto avoid breaking changes in the Helm values overrides which have\nalready been used on existing clusters.\n\n```suggestion\n* helm_extra_values: Allows operators to map arbitrary cluster \ntemplate labels to snippets of Helm value overrides. See example\nbelow for more details.\n\nAs an operator, you **must** be careful to not make changes\nto existing clusters that are not backwards compatible.\nAnd you must not remove any configuration that applies to\nexisting templates, unless that is backwards compatible.\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":53,"context_line":"  and CAN\u0027T be overrided by cli options. If not set in cluster template,"},{"line_number":54,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":55,"context_line":"  If not found there it defaults to 0.10.1"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* helm_extra_values: used to point key in dictionary"},{"line_number":58,"context_line":"  mapping file with value ovverrides for clusters with this label."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Example:"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"21f6e353_66e6e304","line":58,"range":{"start_line":56,"start_character":0,"end_line":58,"end_character":66},"in_reply_to":"b509d6d4_81a2d938","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":54,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":55,"context_line":"  If not found there it defaults to 0.10.1"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"* helm_extra_values: used to point key in dictionary"},{"line_number":58,"context_line":"  mapping file with value ovverrides for clusters with this label."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Example:"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fea82163_789f4da5","line":58,"range":{"start_line":57,"start_character":24,"end_line":58,"end_character":24},"in_reply_to":"fde4c655_29dabcdf","updated":"2025-12-19 14:46:54.000000000","message":"Done.\nAlso added example which describes this behaviour clearly","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Example:"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"We have cluster template with label and value: helm_extra_values \u0027zabbix\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"This should point to key which maps to file with values for chart,"},{"line_number":65,"context_line":"in variable helm_value_override_files in magnum.conf::"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bb18be32_782f3981","line":62,"updated":"2025-12-09 17:14:19.000000000","message":"```suggestion\nWe have cluster template with helm_extra_values \u003d \u0027zabbix\u0027\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Example:"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"We have cluster template with label and value: helm_extra_values \u0027zabbix\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"This should point to key which maps to file with values for chart,"},{"line_number":65,"context_line":"in variable helm_value_override_files in magnum.conf::"}],"source_content_type":"text/x-rst","patch_set":3,"id":"a130e1a1_738681d9","line":62,"in_reply_to":"bb18be32_782f3981","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"We have cluster template with label and value: helm_extra_values \u0027zabbix\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"This should point to key which maps to file with values for chart,"},{"line_number":65,"context_line":"in variable helm_value_override_files in magnum.conf::"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"  [capi_helm]"},{"line_number":68,"context_line":"  helm_value_override_files \u003d zabbix:/etc/magnum/zabbix.yml"}],"source_content_type":"text/x-rst","patch_set":3,"id":"e573f186_d09c06bb","line":65,"range":{"start_line":64,"start_character":7,"end_line":65,"end_character":7},"updated":"2025-12-09 17:14:19.000000000","message":"```suggestion\nThis should point to a key in the helm_value_override_files list in magnum.conf:\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"We have cluster template with label and value: helm_extra_values \u0027zabbix\u0027"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"This should point to key which maps to file with values for chart,"},{"line_number":65,"context_line":"in variable helm_value_override_files in magnum.conf::"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"  [capi_helm]"},{"line_number":68,"context_line":"  helm_value_override_files \u003d zabbix:/etc/magnum/zabbix.yml"}],"source_content_type":"text/x-rst","patch_set":3,"id":"b88d364d_bda00d70","line":65,"range":{"start_line":64,"start_character":7,"end_line":65,"end_character":7},"in_reply_to":"e573f186_d09c06bb","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Content of /etc/magnum/zabbix.yml file should look like this:"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    root@magnum-devstack:/etc/magnum# cat /etc/magnum/zabbix.yml"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    addons:"},{"line_number":78,"context_line":"      custom:"},{"line_number":79,"context_line":"        zabbix-helm-release:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"8e61ad1a_7dab263c","line":76,"range":{"start_line":72,"start_character":0,"end_line":76,"end_character":0},"updated":"2025-12-09 17:14:19.000000000","message":"We can probably remove this command line and the excessive whitespace I think?\n```suggestion\n```","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"Content of /etc/magnum/zabbix.yml file should look like this:"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    root@magnum-devstack:/etc/magnum# cat /etc/magnum/zabbix.yml"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    addons:"},{"line_number":78,"context_line":"      custom:"},{"line_number":79,"context_line":"        zabbix-helm-release:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a5d53ab_b00d0cee","line":76,"range":{"start_line":72,"start_character":0,"end_line":76,"end_character":0},"in_reply_to":"8e61ad1a_7dab263c","updated":"2025-12-19 14:46:54.000000000","message":"Done","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":36412,"name":"Scott Davidson","email":"sdavidson327@gmail.com","username":"sd109"},"change_message_id":"990f8b15131d5041aa2faf4b9ab1d07179d1c8a1","unresolved":true,"context_lines":[{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    root@magnum-devstack:/etc/magnum# cat /etc/magnum/zabbix.yml"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    addons:"},{"line_number":78,"context_line":"      custom:"},{"line_number":79,"context_line":"        zabbix-helm-release:"},{"line_number":80,"context_line":"          spec:"},{"line_number":81,"context_line":"            values:"},{"line_number":82,"context_line":"              zabbixProxy:"},{"line_number":83,"context_line":"                env:"},{"line_number":84,"context_line":"                  - name: ZBX_SERVER_HOST"},{"line_number":85,"context_line":"                    value: \"\u003cyour_zabbix_proxy_ip\u003e\""},{"line_number":86,"context_line":"                  - name: ZBX_HOSTNAME"},{"line_number":87,"context_line":"                    value: \"your_zabbix_proxy\""},{"line_number":88,"context_line":"                image:"},{"line_number":89,"context_line":"                  tag: alpine-7.4-latest"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"11bc0c5e_4ee8c976","line":89,"range":{"start_line":77,"start_character":9,"end_line":89,"end_character":23},"updated":"2025-12-09 17:14:19.000000000","message":"I don\u0027t think these are valid capi-helm-charts values? Doesn\u0027t the zabbix-helm-release section also need `namespace` and `chart` fields?\n\nIt might also be worth including a simpler example of Helm values overrides too since the \u0027Zabbix agent as a custom Helm release\u0027 example is quite unusual ;)\n\nMaybe something like overriding the pod and service CIDRs on all clusters using this config\n\n```yaml\nhelm_value_override_files \u003d all:/etc/magnum/all.yml\n```\nand this values snippet:\n```yaml\nkubeNetwork:\n  pods:\n    cidrBlocks:\n      - 172.30.0.0/16\n  services:\n    cidrBlocks:\n      - 172.31.0.0/16\n```\n\nOpen to other suggestions, but quite a lot of people are interested in this feature for their own customisations, it\u0027s probably worth taking the time to document it as well as we can :)","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"21f40c847ae63ab6224a1f04b3a3bab35973e23e","unresolved":false,"context_lines":[{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    root@magnum-devstack:/etc/magnum# cat /etc/magnum/zabbix.yml"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    addons:"},{"line_number":78,"context_line":"      custom:"},{"line_number":79,"context_line":"        zabbix-helm-release:"},{"line_number":80,"context_line":"          spec:"},{"line_number":81,"context_line":"            values:"},{"line_number":82,"context_line":"              zabbixProxy:"},{"line_number":83,"context_line":"                env:"},{"line_number":84,"context_line":"                  - name: ZBX_SERVER_HOST"},{"line_number":85,"context_line":"                    value: \"\u003cyour_zabbix_proxy_ip\u003e\""},{"line_number":86,"context_line":"                  - name: ZBX_HOSTNAME"},{"line_number":87,"context_line":"                    value: \"your_zabbix_proxy\""},{"line_number":88,"context_line":"                image:"},{"line_number":89,"context_line":"                  tag: alpine-7.4-latest"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3c7776bb_5fce6de9","line":89,"range":{"start_line":77,"start_character":9,"end_line":89,"end_character":23},"in_reply_to":"11bc0c5e_4ee8c976","updated":"2025-12-19 14:46:54.000000000","message":"I have added two more examples in spirit of this comment. And started with something simpler.","commit_id":"8eb6c1175cff298cf59f8626f23732484e3be98c"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"For complete reference of configuration options for this driver used in"},{"line_number":7,"context_line":"magnum.conf plesese refer to `config-reference.rst`"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Features"}],"source_content_type":"text/x-rst","patch_set":5,"id":"d81112a6_55691efc","line":7,"range":{"start_line":7,"start_character":30,"end_line":7,"end_character":50},"updated":"2026-01-02 11:35:29.000000000","message":"This link isn\u0027t working, perhaps add a TOC link instead like this:\n```\n.. toctree::\n  :maxdepth: 1\n\n  Configuration Reference \u003cconfig-reference\u003e\n```","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"a1427e671f1539e42ac615292f2ab1492f9f2f0f","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"For complete reference of configuration options for this driver used in"},{"line_number":7,"context_line":"magnum.conf plesese refer to `config-reference.rst`"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Features"}],"source_content_type":"text/x-rst","patch_set":5,"id":"0ad10bdf_3e066494","line":7,"range":{"start_line":7,"start_character":30,"end_line":7,"end_character":50},"in_reply_to":"63717c67_955bbbc7","updated":"2026-01-07 14:44:41.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":true,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"For complete reference of configuration options for this driver used in"},{"line_number":7,"context_line":"magnum.conf plesese refer to `config-reference.rst`"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Features"}],"source_content_type":"text/x-rst","patch_set":5,"id":"63717c67_955bbbc7","line":7,"range":{"start_line":7,"start_character":30,"end_line":7,"end_character":50},"in_reply_to":"d81112a6_55691efc","updated":"2026-01-07 14:44:21.000000000","message":"My rstviewer didn\u0027t render this properly, so I didn\u0027t catch that. Fixed.","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":14,"context_line":"as updates to node groups and their sizes."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The Kubernetes versions against which the CAPI Helm charts are currently being"},{"line_number":17,"context_line":"testedvcan be found `here \u003chttps://github.com/azimuth-cloud/capi-helm-charts/blob/main/.github/workflows/ensure-capi-images.yaml#L9\u003e`__."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"The driver respects the following cluster and template properties:"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"ae3aebdc_7b788950","line":17,"updated":"2026-01-02 11:35:29.000000000","message":"typo - testedvcan","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"as updates to node groups and their sizes."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The Kubernetes versions against which the CAPI Helm charts are currently being"},{"line_number":17,"context_line":"testedvcan be found `here \u003chttps://github.com/azimuth-cloud/capi-helm-charts/blob/main/.github/workflows/ensure-capi-images.yaml#L9\u003e`__."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"The driver respects the following cluster and template properties:"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"4ec7d586_0cf2fd6f","line":17,"in_reply_to":"ae3aebdc_7b788950","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":85,"context_line":"+-----------------------------------+---------------------+"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"**TODO: Add more recently supported labels here.**"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Two config options require a little bit more explanation:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"e4b68f48_06f3da7c","line":88,"updated":"2026-01-02 11:35:29.000000000","message":"Perhaps remove this TODO note now?","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":85,"context_line":"+-----------------------------------+---------------------+"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"**TODO: Add more recently supported labels here.**"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Two config options require a little bit more explanation:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"e0949e40_2122626c","line":88,"in_reply_to":"e4b68f48_06f3da7c","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"* capi_helm_chart_version: can only be set via template property"},{"line_number":95,"context_line":"  and CAN\u0027T be overrided by cli options. If not set in cluster template,"},{"line_number":96,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"e19af366_0b87ab79","line":95,"updated":"2026-01-02 11:35:29.000000000","message":"overridden","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"* capi_helm_chart_version: can only be set via template property"},{"line_number":95,"context_line":"  and CAN\u0027T be overrided by cli options. If not set in cluster template,"},{"line_number":96,"context_line":"  value is taken from magnum.config option default_helm_chart_version."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"269a321f_5bf83299","line":95,"in_reply_to":"e19af366_0b87ab79","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":103,"context_line":"  to existing clusters that are not backwards compatible."},{"line_number":104,"context_line":"  And you must not remove any configuration that applies to"},{"line_number":105,"context_line":"  existing templates, unless that is backwards compatible. Following"},{"line_number":106,"context_line":"  examples are presenting usage of this funcionality"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"25a2f74d_5610031b","line":106,"updated":"2026-01-02 11:35:29.000000000","message":"Spelling - functionality","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":103,"context_line":"  to existing clusters that are not backwards compatible."},{"line_number":104,"context_line":"  And you must not remove any configuration that applies to"},{"line_number":105,"context_line":"  existing templates, unless that is backwards compatible. Following"},{"line_number":106,"context_line":"  examples are presenting usage of this funcionality"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"2759cff1_242071c2","line":106,"in_reply_to":"25a2f74d_5610031b","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"**Exapmle #1** Using \"all\" clause to override cidrBlocks for pods and services"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":".. note::"},{"line_number":113,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"2f3297a1_3abc4712","line":110,"updated":"2026-01-02 11:35:29.000000000","message":"Consider adding the examples and tips in the remainder of this document as subheadings, to give the document an outline structure","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"**Exapmle #1** Using \"all\" clause to override cidrBlocks for pods and services"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":".. note::"},{"line_number":113,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7aef8c00_91bbfc3c","line":110,"in_reply_to":"2f3297a1_3abc4712","updated":"2026-01-07 14:44:21.000000000","message":"Done.","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":194,"context_line":"  [capi_helm]"},{"line_number":195,"context_line":"  helm_value_override_files \u003d cidr:/etc/magnum/cidr.yml"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"In /etc/magnum/magnum.conf we ned to add:::"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"  [capi_helm]"},{"line_number":200,"context_line":"  helm_value_override_files \u003d cidr:/etc/magnum/all.yml"}],"source_content_type":"text/x-rst","patch_set":5,"id":"82e279f4_ff2ff6af","line":197,"updated":"2026-01-02 11:35:29.000000000","message":"typo - need","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":194,"context_line":"  [capi_helm]"},{"line_number":195,"context_line":"  helm_value_override_files \u003d cidr:/etc/magnum/cidr.yml"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"In /etc/magnum/magnum.conf we ned to add:::"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"  [capi_helm]"},{"line_number":200,"context_line":"  helm_value_override_files \u003d cidr:/etc/magnum/all.yml"}],"source_content_type":"text/x-rst","patch_set":5,"id":"44965750_c2f218c0","line":197,"in_reply_to":"82e279f4_ff2ff6af","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":281,"context_line":"                  tag: alpine-7.4-latest"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"After spawning cluster your zabbix deployment will be there:"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"cc4fc398_598eb127","line":284,"updated":"2026-01-02 11:35:29.000000000","message":"Is there some example output missing here?","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":281,"context_line":"                  tag: alpine-7.4-latest"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"After spawning cluster your zabbix deployment will be there:"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"71fd54ae_d8c1e0cb","line":284,"in_reply_to":"cc4fc398_598eb127","updated":"2026-01-07 14:44:21.000000000","message":"Yup. I can\u0027t believe I missed that. (re)added.","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":334,"context_line":""},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"As a result, after spawning cluster, we will have zabix agent deployed with ip"},{"line_number":338,"context_line":"addresses from pool defined by us."},{"line_number":339,"context_line":""},{"line_number":340,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"d99380de_586485a4","line":337,"updated":"2026-01-02 11:35:29.000000000","message":"Typo - Zabbix","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":334,"context_line":""},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"As a result, after spawning cluster, we will have zabix agent deployed with ip"},{"line_number":338,"context_line":"addresses from pool defined by us."},{"line_number":339,"context_line":""},{"line_number":340,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"e550dd1e_c1c8f56a","line":337,"in_reply_to":"d99380de_586485a4","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"07e2530756645c52bc19003893b539f7f7b453be","unresolved":true,"context_lines":[{"line_number":357,"context_line":"  zabbix-helm-release-zabbix-helm-chart-agent   ClusterIP   172.24.142.131   \u003cnone\u003e        10050/TCP"},{"line_number":358,"context_line":"  zabbix-helm-release-zabbix-helm-chart-proxy   ClusterIP   172.27.127.148   \u003cnone\u003e        10051/TCP"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"**Tip \u0026 Tricks**"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"Currently, all clusters use the Calico CNI. While Cilium is also supported"},{"line_number":363,"context_line":"in the Helm charts, it is not currently regularly tested."}],"source_content_type":"text/x-rst","patch_set":5,"id":"28c1358b_f26b68fd","line":360,"updated":"2026-01-02 11:35:29.000000000","message":"Would be better as a heading - \u0027Tips and Tricks\u0027","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"ffbebba1c36de8a2e998826eb55b9c1d6b009f1d","unresolved":false,"context_lines":[{"line_number":357,"context_line":"  zabbix-helm-release-zabbix-helm-chart-agent   ClusterIP   172.24.142.131   \u003cnone\u003e        10050/TCP"},{"line_number":358,"context_line":"  zabbix-helm-release-zabbix-helm-chart-proxy   ClusterIP   172.27.127.148   \u003cnone\u003e        10051/TCP"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"**Tip \u0026 Tricks**"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"Currently, all clusters use the Calico CNI. While Cilium is also supported"},{"line_number":363,"context_line":"in the Helm charts, it is not currently regularly tested."}],"source_content_type":"text/x-rst","patch_set":5,"id":"30acf9ef_19559209","line":360,"in_reply_to":"28c1358b_f26b68fd","updated":"2026-01-07 14:44:21.000000000","message":"Done","commit_id":"7ba0f7e32d2cfbd4fa39e83d611a7178e4be81e1"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"+-----------------------------------+---------------------+"},{"line_number":37,"context_line":"| Label                             | Default             |"},{"line_number":38,"context_line":"+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":39,"context_line":"| monitoring_enabled                | false               |"}],"source_content_type":"text/x-rst","patch_set":10,"id":"b776a01e_97f7fc96","line":36,"updated":"2026-01-19 14:38:44.000000000","message":"Can we get labels from code and populate this similar to oslo_config options?","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"+-----------------------------------+---------------------+"},{"line_number":37,"context_line":"| Label                             | Default             |"},{"line_number":38,"context_line":"+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":39,"context_line":"| monitoring_enabled                | false               |"}],"source_content_type":"text/x-rst","patch_set":10,"id":"04037171_e1322f31","line":36,"in_reply_to":"b776a01e_97f7fc96","updated":"2026-01-22 15:03:58.000000000","message":"Good idea. I have explored this approach, unfortunately current implementation of labels in driver won\u0027t allow to do this without either writing our own sphinx extension or reorganising parts of code, which is out of scope of this patch. I propose to do this in follow up patch.","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"07fa29a87d723c41d743ecb09c1881237a03d59a","unresolved":true,"context_lines":[{"line_number":58,"context_line":"+-----------------------------------+---------------------+"},{"line_number":59,"context_line":"| etcd_blockdevice_type             | volume              |"},{"line_number":60,"context_line":"|                                   |                     |"},{"line_number":61,"context_line":"+-----------------------------------+---------------------+"},{"line_number":62,"context_line":"| etcd_blockdevice_volume_az        | \"\"                  |"},{"line_number":63,"context_line":"+-----------------------------------+---------------------+"},{"line_number":64,"context_line":"| csi_cinder_availability_zone      | \"\"                  |"}],"source_content_type":"text/x-rst","patch_set":15,"id":"adf7c548_448493f3","line":61,"updated":"2026-05-27 09:56:45.000000000","message":"also missing etcd_volume_type","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"314e3838219b204b13a32d0cc8e5b8ce30d0599a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"+-----------------------------------+---------------------+"},{"line_number":59,"context_line":"| etcd_blockdevice_type             | volume              |"},{"line_number":60,"context_line":"|                                   |                     |"},{"line_number":61,"context_line":"+-----------------------------------+---------------------+"},{"line_number":62,"context_line":"| etcd_blockdevice_volume_az        | \"\"                  |"},{"line_number":63,"context_line":"+-----------------------------------+---------------------+"},{"line_number":64,"context_line":"| csi_cinder_availability_zone      | \"\"                  |"}],"source_content_type":"text/x-rst","patch_set":15,"id":"f96e667d_7b8a8604","line":61,"in_reply_to":"adf7c548_448493f3","updated":"2026-05-28 10:33:55.000000000","message":"Added.","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"07fa29a87d723c41d743ecb09c1881237a03d59a","unresolved":true,"context_lines":[{"line_number":71,"context_line":"| csi_cinder_allow_volume_expansion | True                |"},{"line_number":72,"context_line":"|                                   |                     |"},{"line_number":73,"context_line":"+-----------------------------------+---------------------+"},{"line_number":74,"context_line":"| octavia_provider                  | amphora             |"},{"line_number":75,"context_line":"|                                   | ovn                 |"},{"line_number":76,"context_line":"+-----------------------------------+---------------------+"},{"line_number":77,"context_line":"| octavia_lb_algorithm              | ROUND_ROBIN,        |"}],"source_content_type":"text/x-rst","patch_set":15,"id":"0248e2ff_44de577d","line":74,"range":{"start_line":74,"start_character":2,"end_line":74,"end_character":18},"updated":"2026-05-27 09:56:45.000000000","message":"this is duplicated, see above","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"314e3838219b204b13a32d0cc8e5b8ce30d0599a","unresolved":false,"context_lines":[{"line_number":71,"context_line":"| csi_cinder_allow_volume_expansion | True                |"},{"line_number":72,"context_line":"|                                   |                     |"},{"line_number":73,"context_line":"+-----------------------------------+---------------------+"},{"line_number":74,"context_line":"| octavia_provider                  | amphora             |"},{"line_number":75,"context_line":"|                                   | ovn                 |"},{"line_number":76,"context_line":"+-----------------------------------+---------------------+"},{"line_number":77,"context_line":"| octavia_lb_algorithm              | ROUND_ROBIN,        |"}],"source_content_type":"text/x-rst","patch_set":15,"id":"3dd10c7a_271dba0c","line":74,"range":{"start_line":74,"start_character":2,"end_line":74,"end_character":18},"in_reply_to":"0248e2ff_44de577d","updated":"2026-05-28 10:33:55.000000000","message":"Fixed.","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"07fa29a87d723c41d743ecb09c1881237a03d59a","unresolved":true,"context_lines":[{"line_number":79,"context_line":"|                                   | octavia_provider is |"},{"line_number":80,"context_line":"|                                   | set to \"ovn\".       |"},{"line_number":81,"context_line":"+-----------------------------------+---------------------+"},{"line_number":82,"context_line":"| boot_volume_type                  | nvme                |"},{"line_number":83,"context_line":"+-----------------------------------+---------------------+"},{"line_number":84,"context_line":"| extra_network_name                | \"\"                  |"},{"line_number":85,"context_line":"|                                   |                     |"}],"source_content_type":"text/x-rst","patch_set":15,"id":"dcc6ce90_010ef55b","line":82,"range":{"start_line":82,"start_character":38,"end_line":82,"end_character":42},"updated":"2026-05-27 09:56:45.000000000","message":"I think the default here is \"\"","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"314e3838219b204b13a32d0cc8e5b8ce30d0599a","unresolved":false,"context_lines":[{"line_number":79,"context_line":"|                                   | octavia_provider is |"},{"line_number":80,"context_line":"|                                   | set to \"ovn\".       |"},{"line_number":81,"context_line":"+-----------------------------------+---------------------+"},{"line_number":82,"context_line":"| boot_volume_type                  | nvme                |"},{"line_number":83,"context_line":"+-----------------------------------+---------------------+"},{"line_number":84,"context_line":"| extra_network_name                | \"\"                  |"},{"line_number":85,"context_line":"|                                   |                     |"}],"source_content_type":"text/x-rst","patch_set":15,"id":"84ac7103_98a1b762","line":82,"range":{"start_line":82,"start_character":38,"end_line":82,"end_character":42},"in_reply_to":"dcc6ce90_010ef55b","updated":"2026-05-28 10:33:55.000000000","message":"Fixed.","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"07fa29a87d723c41d743ecb09c1881237a03d59a","unresolved":true,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"One config option require a little bit more explanation:"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"* capi_helm_chart_version: can only be set via template property"}],"source_content_type":"text/x-rst","patch_set":15,"id":"eb5e9c1d_09f4533f","line":90,"range":{"start_line":90,"start_character":18,"end_line":90,"end_character":25},"updated":"2026-05-27 09:56:45.000000000","message":"requires","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"314e3838219b204b13a32d0cc8e5b8ce30d0599a","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"One config option require a little bit more explanation:"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"* capi_helm_chart_version: can only be set via template property"}],"source_content_type":"text/x-rst","patch_set":15,"id":"b8862c4b_69fbeb8d","line":90,"range":{"start_line":90,"start_character":18,"end_line":90,"end_character":25},"in_reply_to":"eb5e9c1d_09f4533f","updated":"2026-05-28 10:33:55.000000000","message":"Good spot. Fixed.","commit_id":"7e3accf93f1707ff0dc4e93b6f39b967b1bcec1a"}],"etc/oslo-config-generator/capi_helm.conf":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"wrap_width \u003d 80"},{"line_number":3,"context_line":"output_file \u003d etc/capi_helm.conf.sample"},{"line_number":4,"context_line":"namespace \u003d capi_helm"}],"source_content_type":"text/plain","patch_set":10,"id":"ee14f533_17c3a67d","line":4,"updated":"2026-01-19 14:38:44.000000000","message":"no newline at the end of file","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"wrap_width \u003d 80"},{"line_number":3,"context_line":"output_file \u003d etc/capi_helm.conf.sample"},{"line_number":4,"context_line":"namespace \u003d capi_helm"}],"source_content_type":"text/plain","patch_set":10,"id":"acea774f_9b5faf27","line":4,"in_reply_to":"ee14f533_17c3a67d","updated":"2026-01-22 15:03:58.000000000","message":"Done","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"}],"magnum_capi_helm/conf.py":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from oslo_config import cfg"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"capi_helm_group \u003d cfg.OptGroup("},{"line_number":17,"context_line":"    name\u003d\"capi_helm\", title\u003d\"Helm Cluster API Driver configuration\""},{"line_number":18,"context_line":")"}],"source_content_type":"text/x-python","patch_set":10,"id":"6b2ee54e_adb3901f","line":15,"updated":"2026-01-19 14:38:44.000000000","message":"extra not needed newline","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from oslo_config import cfg"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"capi_helm_group \u003d cfg.OptGroup("},{"line_number":17,"context_line":"    name\u003d\"capi_helm\", title\u003d\"Helm Cluster API Driver configuration\""},{"line_number":18,"context_line":")"}],"source_content_type":"text/x-python","patch_set":10,"id":"ba382982_c5e77faf","line":15,"in_reply_to":"6b2ee54e_adb3901f","updated":"2026-01-22 15:03:58.000000000","message":"Done","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":155,"context_line":"            and plural names in string format."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"            Example::"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"                \u0027{"},{"line_number":161,"context_line":"                    \"K8sControlPlane\": {"}],"source_content_type":"text/x-python","patch_set":10,"id":"45ac99f8_1199291d","line":158,"range":{"start_line":158,"start_character":20,"end_line":158,"end_character":21},"updated":"2026-01-19 14:38:44.000000000","message":"do we need the extra colon?","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":155,"context_line":"            and plural names in string format."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"            Example::"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"                \u0027{"},{"line_number":161,"context_line":"                    \"K8sControlPlane\": {"}],"source_content_type":"text/x-python","patch_set":10,"id":"d938c579_e0c68364","line":158,"range":{"start_line":158,"start_character":20,"end_line":158,"end_character":21},"in_reply_to":"45ac99f8_1199291d","updated":"2026-01-22 15:03:58.000000000","message":"Depends. I know it looks weird in context of python code, but it is needed so sphinx can render nice code block in on-line documentation containing json snippet bellow and additonally add linters to the mix. It is rotten compromise between code clarity, linters and on-line documentation aesthetics.","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"}],"setup.cfg":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"53ce9c45bb247235ffafef382487321375558ce5","unresolved":true,"context_lines":[{"line_number":29,"context_line":"magnum.drivers \u003d"},{"line_number":30,"context_line":"    k8s_capi_helm_v1 \u003d magnum_capi_helm.driver:Driver"},{"line_number":31,"context_line":"oslo.config.opts \u003d"},{"line_number":32,"context_line":"    capi_helm \u003d magnum_capi_helm.conf:list_capi_opts"}],"source_content_type":"text/x-ttcn-cfg","patch_set":10,"id":"c503716d_7a71175c","line":32,"updated":"2026-01-19 14:38:44.000000000","message":"no newline at the end of the file","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"},{"author":{"_account_id":34213,"name":"Grzegorz Bialas","display_name":"Grzegorz Bialas","email":"grzegorz@stackhpc.com","username":"gbialas"},"change_message_id":"c51e202df57c56adc45e401193285c0b24d235e9","unresolved":false,"context_lines":[{"line_number":29,"context_line":"magnum.drivers \u003d"},{"line_number":30,"context_line":"    k8s_capi_helm_v1 \u003d magnum_capi_helm.driver:Driver"},{"line_number":31,"context_line":"oslo.config.opts \u003d"},{"line_number":32,"context_line":"    capi_helm \u003d magnum_capi_helm.conf:list_capi_opts"}],"source_content_type":"text/x-ttcn-cfg","patch_set":10,"id":"d86f55bd_ae674c5a","line":32,"in_reply_to":"c503716d_7a71175c","updated":"2026-01-22 15:03:58.000000000","message":"Done","commit_id":"bac84185a6a65e4de9cb9b804d53b2003c872da6"}]}