neondatabase / autoscaling

Postgres vertical autoscaling in k8s
Apache License 2.0
142 stars 16 forks source link

neonvm-controller: Set podName only while Pending #973

Closed sharnoff closed 2 weeks ago

sharnoff commented 2 weeks ago

Fixes #967, extracted from #970.

In short, the state machine becomes simpler if we only set the VM's pod name when it's in Pending. This gives us a couple things:

  1. When the VM is Failed or Succeeded, we can un-set the pod name and it will be stable. This is what fixes #967
  2. It gives us a consistent place to set status fields relating to a particular pod before creating the pod. This is why it's helpful for #970