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

services/horizon: Add flag to customize storage directory for Captive Core data #3479

Merged
merged 14 commits into from
Mar 24, 2021

Conversation

Shaptic
Copy link
Contributor

@Shaptic Shaptic commented Mar 22, 2021

What

This introduces the --captive-core-storage-path=./ parameter, which allows users to control where Horizon stores Captive Core's data. This includes the auto-generated configuration, the log, and most-importantly Core's bucket data.

It defaults to the current directory rather than /tmp in order to raise visibility of the large amount of data Core stores in that directory. Internally, /captive-core is appended to it since we wipe the directory on shutdown.

Why

Closes #3437.

Known limitations

n/a

@Shaptic Shaptic added this to the Horizon 2.1.0 milestone Mar 22, 2021
@Shaptic Shaptic requested a review from a team March 22, 2021 18:51
@Shaptic Shaptic self-assigned this Mar 22, 2021
@Shaptic Shaptic force-pushed the custom-bucket-dir branch from b76d848 to d0cbf96 Compare March 22, 2021 19:43
ingest/ledgerbackend/stellar_core_runner.go Outdated Show resolved Hide resolved
services/horizon/internal/flags.go Outdated Show resolved Hide resolved
services/horizon/internal/flags.go Outdated Show resolved Hide resolved
@Shaptic Shaptic force-pushed the custom-bucket-dir branch from f3a677e to 32e72c6 Compare March 24, 2021 18:27
@Shaptic
Copy link
Contributor Author

Shaptic commented Mar 24, 2021

I swear if I need to merge master and pass tests before merging one more friggin' time...

@Shaptic Shaptic merged commit 8880520 into stellar:master Mar 24, 2021
@Shaptic Shaptic deleted the custom-bucket-dir branch March 24, 2021 20:09
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.

Allow setting BUCKET_DIR_PATH when starting Stellar-Core in captive mode
3 participants