Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

Wrong interval when orchestrator queries for data commitment #432

Closed
rach-id opened this issue Aug 9, 2023 · 0 comments · Fixed by #446
Closed

Wrong interval when orchestrator queries for data commitment #432

rach-id opened this issue Aug 9, 2023 · 0 comments · Fixed by #446
Assignees
Labels
bug Something isn't working orchestrator orchestrator related

Comments

@rach-id
Copy link
Member

rach-id commented Aug 9, 2023

After bumping the version of Celestia, we started having the following log in orchestrator:

core0-orch    | I[2023-08-09|10:37:07.366] processing nonce                             nonce=3
core0-orch    | I[2023-08-09|10:37:07.372] signed Valset                                nonce=3
core0-orch    | I[2023-08-09|10:37:07.372] processing nonce                             nonce=2
core0-orch    | I[2023-08-09|10:37:07.380] signed Valset                                nonce=2
core0-orch    | I[2023-08-09|10:37:07.380] processing nonce                             nonce=1
core0-orch    | I[2023-08-09|10:37:07.385] signed Valset                                nonce=1
core0-orch    | I[2023-08-09|10:37:10.847] enqueueing new attestation nonce             nonce=4
core0-orch    | I[2023-08-09|10:37:10.847] processing nonce                             nonce=4
core0-orch    | E[2023-08-09|10:37:10.849] failed to process nonce, retrying            nonce=4 err="error in json rpc client, with http response metadata: (Status: 200 OK, Protocol HTTP/1.1). RPC error -32603 - Internal error: last block 102 is higher than current chain height 101"
core0-orch    | I[2023-08-09|10:37:10.849] trying to recover from error...              
core0-orch    | I[2023-08-09|10:38:06.680] enqueueing new attestation nonce             nonce=5
core0-orch    | I[2023-08-09|10:38:10.850] retrying                                     retry_number=0 retries_left=6
core0-orch    | I[2023-08-09|10:38:10.862] signed commitment                            nonce=4 begin_block=1 end_block=102 data_root_tuple_root=0x8ab1f80ee0a89c54f27d57d4638ac200944da6bb2a4447de55577726d54a1913
core0-orch    | I[2023-08-09|10:38:10.862] succeeded                                    retries_number=0
core0-orch    | I[2023-08-09|10:38:10.862] processing nonce                             nonce=5
core0-orch    | I[2023-08-09|10:38:10.877] signed commitment                            nonce=5 begin_block=102 end_block=203 data_root_tuple_root=0x0aa87f2b4564bc2f06c8084ea5333332c37597c828186d018b2893e0ed276140
core0-orch    | I[2023-08-09|10:39:06.746] enqueueing new attestation nonce             nonce=6
core0-orch    | I[2023-08-09|10:39:06.747] processing nonce                             nonce=6
core0-orch    | I[2023-08-09|10:39:06.758] signed commitment                            nonce=6 begin_

This is due probably to mismatching ranges. We need to fix this

@rach-id rach-id added bug Something isn't working orchestrator orchestrator related labels Aug 9, 2023
@rach-id rach-id self-assigned this Aug 9, 2023
rach-id added a commit to celestiaorg/celestia-core that referenced this issue Aug 10, 2023
## Description

Fixes celestiaorg/orchestrator-relayer#432

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments

---------

Co-authored-by: Rootul P <rootulp@gmail.com>
rach-id added a commit to rach-id/celestia-core that referenced this issue Aug 10, 2023
## Description

Fixes celestiaorg/orchestrator-relayer#432

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments

---------

Co-authored-by: Rootul P <rootulp@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working orchestrator orchestrator related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant