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.

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

Implement ChainObserver adapter configuration in aggregator #1396

Closed
5 tasks done
jpraynaud opened this issue Dec 11, 2023 · 0 comments · Fixed by #1423
Closed
5 tasks done

Implement ChainObserver adapter configuration in aggregator #1396

jpraynaud opened this issue Dec 11, 2023 · 0 comments · Fixed by #1423
Assignees
Labels
bug ⚠️ Something isn't working

Comments

@jpraynaud
Copy link
Member

jpraynaud commented Dec 11, 2023

Why

The stress test does not work anymore as PallasChainObserver is not compatible with the mock-cardano-cli. When starting the aggregator, the stress test panics with the following error:

Error: Dependencies Builder can not create aggregator runner

Caused by:
    Dependency initialization error: «cannot create aggregator runner: failed to retrieve current epoch.» with additional nested error: 'general error
    
    Caused by:
        0: PallasChainObserver Failed to create new client
        1: error connecting bearer
        2: Invalid argument (os error 22)'.

What

Make the aggregator use a ChainAdapter based on a configuration parameter and fix the stress test.

How

  • Add a configuration parameter that allows to select the ChainObserver in the aggregator (as in EraAdapter)
  • Make the aggregator of the stress test use CardanoChainCliObserver
  • Make the aggregator of end to end test use PallasChainObserver
  • Make the aggregator of the infrastructure use PallasChainObserver
  • Update documentation
@jpraynaud jpraynaud added the bug ⚠️ Something isn't working label Dec 11, 2023
@jpraynaud jpraynaud self-assigned this Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ⚠️ Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant