Skip to content

Update state service get block #819

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
Merged
nachog00 merged 157 commits into from
Feb 5, 2026
Merged

Update state service get block #819

nachog00 merged 157 commits into from
Feb 5, 2026

Conversation

@idky137
Copy link
Contributor

@idky137 idky137 commented Feb 4, 2026

Pullis updates in #690 and fixes conflicts.

nachog00 and others added 30 commits December 7, 2025 07:06
@nachog00
Migrate get_block_nullifiers to use indexer instead of deprecated blo…
6ab46a6 
…ck_cache

- Added ChainIndex trait and chain_types imports
- Use indexer.get_compact_block() then apply compact_block_to_nullifiers()
- Simplified height conversion with map_err
@nachog00
fmt
ccd3ba1
@nachog00
Merge branch 'state_service_chain_index_integration' into 677/695-get…
6b9e008 
…_block_nullifiers
@nachog00
Merge branch 'state_service_chain_index_integration' into 677/695-get…
01f20d2 
…_block_nullifiers
@AloeareV
generate 12-block chain
68460b0
@AloeareV
make prev_hash-consistent mock branching chain
89e1fcd
@AloeareV
mockchain source todo impl
8605f6b
@AloeareV
first draft BlockchainSource mock impl
dd6c43b
@AloeareV
some part of the block commitment hack is failing
86feb70
@AloeareV
wip
f6d50b9
@AloeareV
sync loop failure now triggers shutdown
dc1d00c
@AloeareV
test now apparently functions! Next step: actually assserting things...
971f418
@AloeareV
tidy make_chain
08359ed
@AloeareV
get compiling after zebra_3 update
71d0c27
@AloeareV
bit more info
45c9b0f
@AloeareV
fix after cherry-pick
5b3a590
@AloeareV
Unify ChainSegment type
e585152
@AloeareV
test currently shows with a 50% chance of a reorg on any request for …
713b224 
…a block by height, zaino does not sync in the current waiting period
@AloeareV
zaino attempting to reorg below genesis: investigate
2d3fc5e
@nachog00
Merge branch 'state_service_chain_index_integration' into 677/695-get…
97b74f2 
…_block_nullifiers
@AloeareV
sync differently. Main chain path complete-ish, reorg path todoed
f0861c9
@AloeareV
cleanup before reorg handling
0576cea
@AloeareV
start reimplementing handle reorg
32c7fdb
@AloeareV
reorg management complete untested
4b6aa22
@AloeareV
revert to previous proptest reorg logic, finish nfs sync
312dc9d
@nachog00
Merge branch 'state_service_chain_index_integration' into 677/695-get…
245327d 
…_block_nullifiers
@AloeareV
remove unused imports
5e0b9ac
@AloeareV
add minimal find fork point assertion
00fc05d
@AloeareV
ignore failing test
1d2e4e9
@nachog00
Merge branch 'state_service_chain_index_integration' into 677/695-get…
9a20b75 
…_block_nullifiers
idky137 and others added 25 commits January 28, 2026 11:44
@idky137 @pacu
added db compact block streamer tests
1aecb9c
@idky137 @pacu
updated db changelog
ad23141
@idky137 @pacu
review comment changes
c3682e3
@nachog00
Merge branch 'state_service_chain_index_integration' into 677/status-…
f238d37 
…trait
@nachog00
Merge pull request zingolabs#776 from zingolabs/677/status-trait
b1afd8f 
Introduce Status trait for service readiness probing
@idky137 @pacu
fix v1::get_compact_blocks test
d3c727a
@idky137 @pacu
fixed test_pool_type_filter_t_z_o
3048401
@idky137 @pacu
fixed get_transparent_data_from_compact_block_when_requested
863a9cd
@idky137 @pacu
clippy fix
32b7104
@pacu
move zaino_state::utils::blockid_to_hashorheight to zaino-proto
29474bd
@pacu
remove rustfmt TOML to avoid fmt warnings
e82fbe1
@pacu
cargo fmt
86244b1
@pacu
cargo clippy
c21edf7
@pacu
Create new CHANGELOG files and update existing ones
6a41ffa
@idky137
Merge pull request zingolabs#788 from pacu/feature/lightclient-protoc…
da2717f 
…ol-v0.4.0

Feature/lightclient protocol v0.4.0
@idky137
fixed zaino-state/src/chain_index/tests/proptest_blockgen.rs
e38f122
@idky137
merge conflicts
811d04f
@idky137
updated ChainIndex::get_compact_block
d7661b0
@idky137
added compact block streamer to chain index, switched fetchservice to…
e685085 
… use new method
@idky137
test and bug fixes
e340d17
@idky137
merge conflicts
b752840
@idky137
fixed get_transparent_data_from_compact_block_when_requested
08ffe1e
@idky137
Merge remote-tracking branch 'idky137/chain_index_integration' into u…
48c91e9 
…pdate_state_service_chain_index_integration
@idky137
remove unused import
35b4475
@idky137
merge conflicts
8a82a99
@idky137
Copy link
Contributor Author

idky137 commented Feb 4, 2026

all tests pass locally.

Note
8a82a99 is the only "real" commit (all others are from the parent pr).

@idky137 idky137 marked this pull request as ready for review February 4, 2026 15:44
@idky137 idky137 requested review from AloeareV and nachog00 February 4, 2026 15:44
@nachog00 nachog00 merged commit 829350e into zingolabs:677/694-get_block Feb 5, 2026
18 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AloeareV AloeareV Awaiting requested review from AloeareV

@nachog00 nachog00 Awaiting requested review from nachog00

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@idky137 @nachog00 @AloeareV @gustavovalverde @zancas @fluidvanadium @pacu @LarryRuane