search

neondatabase / autoscaling

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

neonvm-controller: Release leader election lease on exit #1091

Closed sharnoff closed 1 month ago

sharnoff commented 2 months ago

We currently aren't doing this, and it leads to 30s+ scaling operations when we deploy neonvm.

Notes for review: I want to also get rid of the srv stuff (I think that'll make this simpler), but that's probably best left for a follow-up PR :)

github-actions[bot] commented 2 months ago

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ :robot:
github.com/neondatabase/autoscaling/neonvm 0.00% (ø)
Coverage by file ### Changed files (no unit tests) | Changed File | Coverage Δ | Total | Covered | Missed | :robot: | |--------------|------------|-------|---------|--------|---------| | github.com/neondatabase/autoscaling/neonvm/main.go | 0.00% (ø) | 151 (-2) | 0 | 151 (-2) | | _Please note that the "Total", "Covered", and "Missed" counts above refer to ***code statements*** instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code._

HTML Report

Click to open