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

Epoch information change #218

Closed
M4rc0Russ0 opened this issue Jul 24, 2023 · 4 comments · Fixed by #238
Closed

Epoch information change #218

M4rc0Russ0 opened this issue Jul 24, 2023 · 4 comments · Fixed by #238
Labels
review No actions expected, need to monitor post a dependency

Comments

@M4rc0Russ0
Copy link

Describe the bug
When I find an epoch using the Koios API, the tx_count, blk_count and out_sum values are different per request.

To Reproduce

  1. Go to 'Koios API'
  2. Click on _epoch_no string
  3. Write 348
  4. Click on TRY
  5. Save results
  6. Click on TRY again
  7. Compare results

Expected behavior
Every time someone make a request to the endpoint to get the epoch 348 should receive exactly the same information each time.

Screenshots
image
image
@rcmorano
Copy link
Contributor

This is probably related to this issue on cardano-db-sync, maybe some nodes are/were running different cardano-db-sync versions or restored from snapshots created by diff versions of it.

@rdlrt
Copy link
Contributor

rdlrt commented Jul 25, 2023

Even within a release, the state could differ depending on epoch cache...the instance providers have been instructed to run fixes - it should resolve in few hours

@rdlrt
Copy link
Contributor

rdlrt commented Jul 25, 2023

This should be sorted for now, but will keep the issue open until next release - as graceful fix will only be available on next dbsync release.
image

@rdlrt rdlrt added the review No actions expected, need to monitor post a dependency label Jul 25, 2023
@rdlrt rdlrt mentioned this issue Aug 11, 2023
Closed
@rdlrt
Copy link
Contributor

rdlrt commented Aug 28, 2023

Temporary patch to fix historical entries added to cron in #227 , part of upcoming release

@rdlrt rdlrt closed this as completed Aug 28, 2023
@rdlrt rdlrt mentioned this issue Oct 6, 2023
Merged
59 tasks
rdlrt added a commit that referenced this issue Oct 13, 2023
@rdlrt @hodlonaut @Scitz0
Koios v1.1.0rc (#238)
9b3460b 
## Description
<!--- Describe your changes -->
[Re-created from #227 post renaming base-branch]

- [x] pool_list: Return all fields from pool_info_cache (latest pool entry)
- [x] Remove filter for returning only 'registered' pools from pool_list, pool_relays and pool_metadata endpoints
- [x] Few more linting polishing (remove trailing spaces, move single column/table references onto same line)
- [x] Update pool_info to return same metadata as pool_list and pool_metadata (latest w/o fallback)
- [x] Update all tx_in joins to use the new tx_out.consumed_by_tx_in_id
- [x] Dont include epoch_params in epoch_info_cache, update references accordingly
- [x] Update `*_txs` as per #186 (point 4)
- [x] Update `*_utxos` as per #186 (point 3)
  - [x] credential_utxos
  - [x] address_utxos
  - [x] account_utxos
  - [x] utxo_info
  - [x] asset_utxos
  - [x] script_utxos
- [x] Add script_info and align all endpoints for scripts
- [x] Add temporary cron for fixing epoch count mismatch
- [x] Add pool_registrations/pool_deregistrations endpoint
- [x] Add retired txs to `pool_updates` (other fields for such transactions will return `null`)
- [x] Add reward_withdrawals/treasury_withdrawals endpoint
- [x] Simplify address_assets and account_assets heirarchy to return flat table (helps do horizontal filtering as desired)
- [x] API Spec updates:
  - [x] Add section for Authentication
  - [x] Bump Koios version
  - [x] pool_list
  - [x] pool_metadata
  - [x] pool_relay
  - [x] `*_txs`
  - [x] `*_utxos`
  - [x] `address_assets`
  - [x] `account_assets`
  - [x] `epoch_params`
  - [x] `pool_registrations`/`pool_deregistrations`
  - [x] `reward_withdrawals`/`treasury_withdrawals`
  - [x] All script-related endpoints
  - [x] Update examples
  - [x] Check re-usability
  - [x] Check schemathesis
    - [x] guild
    - [x] preview
    - [x] preprod
    - [x] mainnet
- [x] Add v0 vs v1 to monitoring
  - [x] guild
  - [ ] preview (post merge)
  - [ ] preprod (post merge)
  - [ ] mainnet (post 1.1.0 - instead of 1.1.0rc) release

- [x] Ogmios
  - [x] Add integration for tx Evaluation and submission using ogmios path
  - [x] Add health check based on /health (`version` and `networkSynchronization`)
- [x] Update CHANGELOG
- [x] Bump final specs version number to v1 

## Which issue it fixes?
<!--- Link to issue: Closes #issue-number -->
- [x] Closes #208
- [x] Closes #218
- [x] Closes #186 
- [x] Closes #221
- [x] Closes #224
- [x] Closes #191
- [x] Closes #232
- [x] Closes #240 #241

---------

Co-authored-by: KoT_B_KocMoce <[email protected]>
Co-authored-by: Ola [AHLNET] <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
review No actions expected, need to monitor post a dependency
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Koios v1.1.0rc cardano-community/koios-artifacts
3 participants
@rcmorano @rdlrt @M4rc0Russ0