neondatabase / autoscaling

Postgres vertical autoscaling in k8s
Apache License 2.0
153 stars 21 forks source link

neonvm: Fix double prefix in image loader #993

Closed sharnoff closed 3 months ago

sharnoff commented 3 months ago

Basically, we were applying the 'neonvm-' prefix twice when we added neonvm-runner-image-loader into neonvm.yaml (see #988 for more).

The diff in neonvm.yaml from the #988 to now:

4007c4007
<   name: neonvm-neonvm-runner-image-loader
---
>   name: neonvm-runner-image-loader

(i.e., the name of the daemonset is corrected, as it should be)

And, there is no diff in neonvm-runner-image-loader.


Medium-term, we should probably remove the namespace prefix thing, but I'd rather just get this fixed ASAP.