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

proxy: add local-proxy to compute image #8823

Merged
merged 12 commits into from
Oct 4, 2024
Merged

Conversation

conradludgate
Copy link
Contributor

@conradludgate conradludgate commented Aug 23, 2024

  1. Adds local-proxy to compute image and vm spec
  2. Updates local-proxy config processing, writing PID to a file eagerly
  3. Updates compute-ctl to understand local proxy compute spec and to send SIGHUP to local-proxy over that pid.

closes https://github.com/neondatabase/cloud/issues/16867

Copy link

github-actions bot commented Aug 23, 2024

5085 tests run: 4878 passed, 0 failed, 207 skipped (full report)


Flaky tests (4)

Postgres 17

Postgres 16

Postgres 14

Code coverage* (full report)

  • functions: 31.4% (7509 of 23942 functions)
  • lines: 49.6% (60273 of 121590 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
f119dc4 at 2024-10-04T13:36:41.362Z :recycle:

Base automatically changed from local-proxy to main August 23, 2024 21:32
vm-image-spec.yaml Outdated Show resolved Hide resolved
@conradludgate conradludgate force-pushed the local-proxy-compute branch 3 times, most recently from 4571a5c to cbe0843 Compare September 20, 2024 18:37
@conradludgate conradludgate marked this pull request as ready for review September 23, 2024 09:45
@conradludgate conradludgate requested review from a team as code owners September 23, 2024 09:45
compute_tools/src/compute.rs Outdated Show resolved Hide resolved
proxy/src/bin/local_proxy.rs Outdated Show resolved Hide resolved
compute_tools/src/config.rs Outdated Show resolved Hide resolved
compute/vm-image-spec.yaml Outdated Show resolved Hide resolved
compute_tools/src/config.rs Outdated Show resolved Hide resolved
compute_tools/src/compute.rs Outdated Show resolved Hide resolved
@conradludgate conradludgate enabled auto-merge (squash) October 1, 2024 06:36
@conradludgate conradludgate merged commit 6c05f89 into main Oct 4, 2024
79 checks passed
@conradludgate conradludgate deleted the local-proxy-compute branch October 4, 2024 14:52
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.

3 participants