Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add safekeeper utilization endpoint #10429

Merged
merged 2 commits into from
Jan 17, 2025
Merged

Conversation

arpad-m
Copy link
Member

@arpad-m arpad-m commented Jan 16, 2025

Add an endpoint to obtain the utilization of a safekeeper. Future changes to the storage controller can use this endpoint to find the most suitable safekeepers for newly created timelines, analogously to how it's done for pageservers already.

Initially we just want to assign by timeline count, then we can iterate from there.

Part of #9011

Copy link

github-actions bot commented Jan 16, 2025

7326 tests run: 6949 passed, 0 failed, 377 skipped (full report)


Flaky tests (3)

Postgres 15

Code coverage* (full report)

  • functions: 33.7% (8429 of 25046 functions)
  • lines: 49.2% (70492 of 143362 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
26e3f61 at 2025-01-16T17:59:48.845Z :recycle:

@arpad-m arpad-m requested a review from jcsp January 16, 2025 19:03
@arpad-m arpad-m added this pull request to the merge queue Jan 17, 2025
Merged via the queue into main with commit b0f3409 Jan 17, 2025
84 checks passed
@arpad-m arpad-m deleted the arpad/safekeeper_utilization branch January 17, 2025 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants