search

algorand / conduit

Algorand's data pipeline framework.
MIT License
37 stars 26 forks source link

Set conduit data dir through env var in entrypoint script #154

Closed cshintov closed 10 months ago

cshintov commented 11 months ago

Use CONDUIT_DATA_DIR in the entrypoint script for docker.

CLAassistant commented 11 months ago

CLA assistant check
All committers have signed the CLA.

codecov[bot] commented 11 months ago

Codecov Report

Merging #154 (d3d5767) into master (442791a) will increase coverage by 5.23%. Report is 64 commits behind head on master. The diff coverage is 82.80%.

@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
+ Coverage   67.66%   72.89%   +5.23%     
==========================================
  Files          32       38       +6     
  Lines        1976     2789     +813     
==========================================
+ Hits         1337     2033     +696     
- Misses        570      646      +76     
- Partials       69      110      +41
Files Coverage Δ
conduit/data/block_export_data.go 100.00% <ø> (+92.30%) :arrow_up:
conduit/init_provider.go 100.00% <100.00%> (ø)
conduit/loggers/loggers.go 91.89% <100.00%> (+0.22%) :arrow_up:
conduit/metrics/metrics.go 100.00% <ø> (ø)
conduit/pipeline/common.go 90.62% <100.00%> (+65.62%) :arrow_up:
conduit/pipeline/errors.go 100.00% <100.00%> (ø)
conduit/pipeline/logging.go 100.00% <100.00%> (ø)
conduit/pipeline/metadata.go 73.52% <ø> (+4.41%) :arrow_up:
conduit/plugins/config.go 100.00% <ø> (ø)
conduit/plugins/exporters/exporter_factory.go 100.00% <100.00%> (ø)
... and 22 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

cshintov commented 10 months ago

Yup. Saw it. Was thinking to raise a PR for that as well. Thanks for the merge 🙂 !

cshintov commented 10 months ago

Done. https://github.com/algorand/indexer/pull/1583