storcon: update db related dependencides

neondatabase / neon

Neon: Serverless Postgres. We separated storage and compute to offer autoscaling, code-like database branching, and scale to zero.

https://neon.tech

Apache License 2.0

14.48k stars 419 forks source link

storcon: update db related dependencides #8155

Closed VladLazar closed 3 months ago

VladLazar commented 3 months ago

Problem

Storage controller runs into memory corruption issue on the drain/fill code paths.

Summary of changes

Update db related depdencies in the unlikely case that the issue was fixed in diesel.

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

github-actions[bot] commented 3 months ago

2947 tests run: 2830 passed, 0 failed, 117 skipped (full report)

Flaky tests (2)

#### Postgres 16 - `test_wal_restore`: [debug](https://neon-github-public-dev.s3.amazonaws.com/reports/pr-8155/9662049822/index.html#suites/7ac407ba57d143e0d2d17655a1a207c1/3c4e27c81b9bc089/retries) #### Postgres 14 - `test_isolation[4]`: [debug](https://neon-github-public-dev.s3.amazonaws.com/reports/pr-8155/9662049822/index.html#suites/158be07438eb5188d40b466b6acfaeb3/25e7e13f58777f5b/retries)

Code coverage* (full report)

functions: 32.5% (6875 of 21135 functions)
lines: 50.2% (53654 of 106976 lines)

* collected from Rust tests only

_{The comment gets automatically updated with the latest test results
f8d2ee3bd0cdb75d3f0f4357c80b88630e6ce19c at 2024-06-25T13:29:07.680Z :recycle:}

jcsp commented 3 months ago

Hmm, I guess a lot of the underlying code for postgres is coming from https://github.com/neondatabase/rust-postgres -- so some of that might be a bit outdated, not sure when it was last reset from upstream

VladLazar commented 3 months ago

Hmm, I guess a lot of the underlying code for postgres is coming from https://github.com/neondatabase/rust-postgres -- so some of that might be a bit outdated, not sure when it was last reset from upstream

Good point. Let's try this and I'll look at those deps after.