neondatabase / autoscaling

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

Use cargo-chef for vm-monitor builds #991

Open sharnoff opened 2 weeks ago

sharnoff commented 2 weeks ago

Before this PR, pg16-disk-test: Clean build took 266s. Rebuilding from main+1 took 201s.

With this PR, pg16-disk-test: Clean build took 303s. Rebuilding from main+1 took 101s.

(main+1 was jcsp/shard-split-logging; 75747cdbf -> 7f7f33f7e)

This is still quite a bit more time to rebuild than I'd like. The final step of building vm-monitor actually takes a surprisingly long time.

Also, there's this strange thing where the cargo chef image spends time installing clippy et al during prepare & cook (so, we actually end up doing it twice!). It doesn't take too long, but it's worth fixing that before merging, because there's no sense being wasteful.