[ResponseOps][TaskManager] followups from resource based scheduling PR

elastic / kibana

Your window into the Elastic Stack

https://www.elastic.co/products/kibana

Other

19.71k stars 8.13k forks source link

Open pmuellr opened 1 month ago

pmuellr commented 1 month ago

While reviewing https://github.com/elastic/kibana/pull/189626 , I noted a few potential issues, collected here to follow up on:

need to add new config to our docker and cloud allow lists: comment
change heap-to-capacity converter to take into account larger amounts of RAM comment
maxAllowedCost seems lazily initialized, so wondering if anything is sensitive to the initial value of 0 - comment

elasticmachine commented 1 month ago

Pinging @elastic/response-ops (Team:ResponseOps)