neondatabase / autoscaling

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

agent: Add scaling event reporting #1107

Open sharnoff opened 1 month ago

sharnoff commented 1 month ago

This is part 2 of 2; see #1078 for the ground work and neondatabase/cloud#15939 for the full context.

In short, this PR:


Notes for review:

I'd like to add minio-based S3 tests to this, but it seemed like it'd be non-trivial, particularly because scaling events actually require that there's scaling that happens ­— unlike the existing billing tests.

So I figured I'd open this for review in the meantime.

Also note: This PR builds on #1078 and must not be merged before it.

github-actions[bot] commented 1 month ago
No changes to the coverage. ### HTML Report Click to open
sharnoff commented 1 week ago

Remaining items for me, on this:

  1. Add more thorough e2e tests
  2. Test on staging

In the meantime, it should be ok to review.