geographybuff commented on issue #977: URL: https://github.com/apache/apisix-helm-chart/issues/977#issuecomment-6693006593
Agreed that the template belongs here, and it looks like a regression rather than a gap: this chart had an `hpa.yaml` until #837 removed it in the v2 update, which left the `autoscaling` values and the `pdb.yaml` branch behind. #1031 restores it. One difference from the `apisix` chart you pointed to: this controller uses leader election, so only the leader reconciles. In a k3d test with 3 replicas, only the leader started workers, and a standby took over about 30 s after the leader pod was deleted. So an HPA here gives failover rather than more capacity. A small range like your 2 to 3 fits that, and the PR defaults `maxReplicas` to 3 instead of the `apisix` chart's 100. Whether to offer an HPA on that basis is a maintainer call, and the PR lists removing the values as the alternative. AI disclosure: written with Claude Code. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
