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.

Sign up for GitHub

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

Jump to bottom

feat(stats): wait until blockscout is indexed #1075

Merged
merged 24 commits into from
Oct 8, 2024
Merged

feat(stats): wait until blockscout is indexed #1075

merged 24 commits into from
Oct 8, 2024

Conversation

bragov4ik
Copy link
Contributor

Configurable thresholds for ratios returned from indexing-status endpoint; wait until they're satisfied & launch stats

@bragov4ik bragov4ik requested a review from sevenzing October 2, 2024 16:22
rimrakhimov
rimrakhimov approved these changes Oct 3, 2024
View reviewed changes
stats/stats-server/src/server.rs Outdated Show resolved Hide resolved
stats/stats-server/src/server.rs Outdated Show resolved Hide resolved
stats/stats-server/src/server.rs Outdated Show resolved Hide resolved
@bragov4ik
Copy link
Contributor Author

just for the record: the new IGNORE_​​BLOCKSCOUT_​API_​ABSENCE env is created with some upcoming changes in mind; we'll very likely have some charts depend on data from blockscout api, so this single env should simplify deployment without setting up the API

@bragov4ik bragov4ik requested a review from rimrakhimov October 4, 2024 17:25
bragov4ik added 22 commits October 8, 2024 15:13
@bragov4ik
add api config
d327453
@bragov4ik
bump to deduplicate deps in the future (when actix-prost is updated)
233ef43
@bragov4ik
wait for indexing
b24bbdf
@bragov4ik
test wait + fix
35dc9fc
@bragov4ik
better and more tests
3184a09
@bragov4ik
fix checks in tests to be easier to read on fail
88f3c03
@bragov4ik
tests again
255d430
@bragov4ik
clippy
4520236
@bragov4ik
adjust thresholds after checking hosted instances
a7c9c1e
@bragov4ik
fmt
b91615e
@bragov4ik
extract wait_config to fixture
6826db7
@bragov4ik
switch to f64
f50b819
@bragov4ik
fix envs couldn't disable thresholds
fd69498
@bragov4ik
improve description of template vars in readme
1239699
@bragov4ik
clippy
5bf68fb
@bragov4ik
env to ignore lack of provided api env and disable corresponding feat…
cc04d1a 
…ures (for ease of deployment if they're not needed)
@bragov4ik
better naming
5d1fdcc
@bragov4ik
remove excessive enabled setting
cac5fb5
@bragov4ik
add env gen to justfile
28d340f
@bragov4ik
rename variables after changing structure
bde7240
@bragov4ik
generate env docs + add zero-width spaces + write descriptions
e7c4ff4
@bragov4ik
fix test
f41e355
@bragov4ik bragov4ik force-pushed the bragov4ik/stats-wait-until-blockscout-is-indexed branch from 89ee8dc to e9714db Compare October 8, 2024 12:14
@bragov4ik bragov4ik merged commit 19dd633 into main Oct 8, 2024
5 checks passed
@bragov4ik bragov4ik deleted the bragov4ik/stats-wait-until-blockscout-is-indexed branch October 8, 2024 13:14
@bragov4ik bragov4ik mentioned this pull request Nov 5, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sevenzing sevenzing sevenzing left review comments

@rimrakhimov rimrakhimov Awaiting requested review from rimrakhimov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bragov4ik @rimrakhimov @sevenzing