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

Rename binary attachment_service -> storage_controller #7042

Merged
merged 3 commits into from
Mar 7, 2024

Conversation

jcsp
Copy link
Collaborator

@jcsp jcsp commented Mar 7, 2024

Problem

The storage controller binary still has its historic attachment_service name -- it will be painful to change this later because we can't atomically update this repo and the helm charts used to deploy.

Companion helm chart change: neondatabase/helm-charts#70

Summary of changes

  • Change the name of the binary to storage_controller
  • Skipping renaming things in the source right now: this is just to get rid of the legacy name in external interfaces.

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

@jcsp jcsp added a/tech_debt Area: related to tech debt c/storage/controller Component: Storage Controller labels Mar 7, 2024
Copy link

github-actions bot commented Mar 7, 2024

2490 tests run: 2369 passed, 0 failed, 121 skipped (full report)


Code coverage* (full report)

  • functions: 28.8% (6993 of 24311 functions)
  • lines: 47.3% (43012 of 90873 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
d84383e at 2024-03-07T13:34:03.114Z :recycle:

@jcsp jcsp force-pushed the jcsp/storcon-bin-name branch from 25b2ed7 to 588e536 Compare March 7, 2024 09:54
@jcsp jcsp marked this pull request as ready for review March 7, 2024 11:52
@jcsp jcsp requested a review from a team as a code owner March 7, 2024 11:52
@jcsp jcsp requested review from koivunej, arpad-m and VladLazar and removed request for koivunej March 7, 2024 11:52
@jcsp jcsp merged commit d3c583e into main Mar 7, 2024
53 checks passed
@jcsp jcsp deleted the jcsp/storcon-bin-name branch March 7, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a/tech_debt Area: related to tech debt c/storage/controller Component: Storage Controller
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants