🐛 fix: autoscaling doesn't respect queued items

akozhuharov commented 6 days ago

Description

We encountered a bug where the autoscaling agentpool controller doesn't take into account generally queued items in TFC and hence the replicas stay at the minimum number set in the agent pool. Steps to reproduce:

Create an agent pool with autoscaling set to minimum 1
Trigger runs on several workspaces

Usage Example

I have attached a script using parts of the code in the controller which can highlight the difference(line 32 can be added/removed). main.go.zip

References

Depends on https://github.com/hashicorp/terraform-cloud-operator/pull/419

Community Note

Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
If you are interested in working on this issue or have submitted a pull request, please leave a comment.

hashicorp-cla-app[bot] commented 6 days ago

All committers have signed the CLA.

arybolovlev commented 23 hours ago

Hi @akozhuharov,

I was not able to reproduce this behavior. With no registered agents, runs transition to the plan_queued state.

Are there any other conditions for a run stuck in the queuing state?

Thanks.

hashicorp / terraform-cloud-operator