)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"5fa7dec6f0ea66c090c255a8bd9d53ba938b9945","unresolved":true,"context_lines":[{"line_number":20,"context_line":"by the Helm chart and results in a MachineDeployment with zero replicas."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"For autoscaling nodegroups with machineCountMin\u003d0, the driver now"},{"line_number":23,"context_line":"forwards the value to Helm. Note that the default chart version (0.10.1)"},{"line_number":24,"context_line":"rejects machineCountMin \u003c 1; a chart version that lifts this restriction"},{"line_number":25,"context_line":"is required to fully enable autoscaling scale-to-zero."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Add tests covering:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"59ecf8fa_ac834b51","line":24,"range":{"start_line":23,"start_character":28,"end_line":24,"end_character":27},"updated":"2026-05-22 15:05:29.000000000","message":"How do we make sure we run the required chart version?\nAnd why are we not bumping default chart version?\nWe at least need a reno stating what is the required chart version, but also we should not support scale to/from zero if the required dependencies (CAPO and Helm charts) are not in place.","commit_id":"f3f3e59b813e99532ee2d04fe863f8ccdf947853"},{"author":{"_account_id":27337,"name":"Mathieu GRZYBEK","email":"mathieu@stackhpc.com","username":"mgrzybek"},"change_message_id":"a6fcc4b3a6d9af229e2ce9e8f1a6b6cc71115335","unresolved":false,"context_lines":[{"line_number":20,"context_line":"by the Helm chart and results in a MachineDeployment with zero replicas."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"For autoscaling nodegroups with machineCountMin\u003d0, the driver now"},{"line_number":23,"context_line":"forwards the value to Helm. Note that the default chart version (0.10.1)"},{"line_number":24,"context_line":"rejects machineCountMin \u003c 1; a chart version that lifts this restriction"},{"line_number":25,"context_line":"is required to fully enable autoscaling scale-to-zero."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Add tests covering:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"117a49e7_587e607b","line":24,"range":{"start_line":23,"start_character":28,"end_line":24,"end_character":27},"in_reply_to":"59ecf8fa_ac834b51","updated":"2026-07-09 09:47:49.000000000","message":"I am working on a pull request on capi-helm-charts’s side (https://github.com/azimuth-cloud/capi-helm-charts/pull/745). I guess that the required version will be strictly higher than the current latest release (0.25.1). Some changes will be needed in `conf.py`.","commit_id":"f3f3e59b813e99532ee2d04fe863f8ccdf947853"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ca066e59b65bed61ca1bca159e7705afcae393b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1df99bf5_0259a218","updated":"2026-07-09 12:35:29.000000000","message":"Last reno nit and we\u0027re done.","commit_id":"dde36da9bd4028d502d78f66c72a5983ea9db129"}],"magnum_capi_helm/driver.py":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"5fa7dec6f0ea66c090c255a8bd9d53ba938b9945","unresolved":true,"context_lines":[{"line_number":756,"context_line":"        )"},{"line_number":757,"context_line":""},{"line_number":758,"context_line":"        if min_nodes is not None:"},{"line_number":759,"context_line":"            if min_nodes \u003c 0:"},{"line_number":760,"context_line":"                raise exception.NodeGroupInvalidInput("},{"line_number":761,"context_line":"                    message\u003d\"Min node count must be greater than \""},{"line_number":762,"context_line":"                    \"or equal to 0 for all node groups.\""}],"source_content_type":"text/x-python","patch_set":1,"id":"fe336c07_df8268d8","line":759,"updated":"2026-05-22 15:05:29.000000000","message":"How do we make sure CAPO is running at a version that will accept this?","commit_id":"f3f3e59b813e99532ee2d04fe863f8ccdf947853"},{"author":{"_account_id":27337,"name":"Mathieu GRZYBEK","email":"mathieu@stackhpc.com","username":"mgrzybek"},"change_message_id":"e58333dc9ef8309235043cf16598e8a6293cc8d0","unresolved":false,"context_lines":[{"line_number":756,"context_line":"        )"},{"line_number":757,"context_line":""},{"line_number":758,"context_line":"        if min_nodes is not None:"},{"line_number":759,"context_line":"            if min_nodes \u003c 0:"},{"line_number":760,"context_line":"                raise exception.NodeGroupInvalidInput("},{"line_number":761,"context_line":"                    message\u003d\"Min node count must be greater than \""},{"line_number":762,"context_line":"                    \"or equal to 0 for all node groups.\""}],"source_content_type":"text/x-python","patch_set":1,"id":"9aac0df6_245e115b","line":759,"in_reply_to":"66b13195_4a21b437","updated":"2026-07-09 09:54:41.000000000","message":"I added a release note https://review.opendev.org/c/openstack/magnum-capi-helm/+/989645/3/releasenotes/notes/scale-to-zero-workers-4f8a2e1c9b7d3f05.yaml","commit_id":"f3f3e59b813e99532ee2d04fe863f8ccdf947853"},{"author":{"_account_id":27337,"name":"Mathieu GRZYBEK","email":"mathieu@stackhpc.com","username":"mgrzybek"},"change_message_id":"a6fcc4b3a6d9af229e2ce9e8f1a6b6cc71115335","unresolved":false,"context_lines":[{"line_number":756,"context_line":"        )"},{"line_number":757,"context_line":""},{"line_number":758,"context_line":"        if min_nodes is not None:"},{"line_number":759,"context_line":"            if min_nodes \u003c 0:"},{"line_number":760,"context_line":"                raise exception.NodeGroupInvalidInput("},{"line_number":761,"context_line":"                    message\u003d\"Min node count must be greater than \""},{"line_number":762,"context_line":"                    \"or equal to 0 for all node groups.\""}],"source_content_type":"text/x-python","patch_set":1,"id":"66b13195_4a21b437","line":759,"in_reply_to":"fe336c07_df8268d8","updated":"2026-07-09 09:47:49.000000000","message":"We have several options: rely on CAPO’s helm version (if used) or have a look at the image tag used in the deployment if this feature is enabled by default it’s gonna be enough. Moreover, we can have a look at the feature flags enabled in the `deployment` object (command arguments or env variables).","commit_id":"f3f3e59b813e99532ee2d04fe863f8ccdf947853"}],"releasenotes/notes/scale-to-zero-workers-4f8a2e1c9b7d3f05.yaml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ca066e59b65bed61ca1bca159e7705afcae393b2","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Worker nodegroups can now be scaled to and from zero when autoscaling"},{"line_number":5,"context_line":"    is enabled. Set the ``min_node_count`` label to ``0`` to allow the"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"da2beb97_496a1602","line":2,"updated":"2026-07-09 12:35:29.000000000","message":"Need an upgrade section sating default capi helm charts version was updated to 0.26.0 (it\u0027s not the same as saying this feature requires this version).","commit_id":"dde36da9bd4028d502d78f66c72a5983ea9db129"},{"author":{"_account_id":27337,"name":"Mathieu GRZYBEK","email":"mathieu@stackhpc.com","username":"mgrzybek"},"change_message_id":"3dbb68f87a161349d698e9eb4a71ee289c629635","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Worker nodegroups can now be scaled to and from zero when autoscaling"},{"line_number":5,"context_line":"    is enabled. Set the ``min_node_count`` label to ``0`` to allow the"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5129d4ae_81d00d9d","line":2,"in_reply_to":"da2beb97_496a1602","updated":"2026-07-09 13:42:22.000000000","message":"I added an `upgrade` section. I hope that this is enough.","commit_id":"dde36da9bd4028d502d78f66c72a5983ea9db129"}]}