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

Transition Mithril Era Reader adapter to CardanoChain in devnet #1428

Closed
3 tasks done
Tracked by #1482
jpraynaud opened this issue Jan 11, 2024 · 0 comments · Fixed by #1431
Closed
3 tasks done
Tracked by #1482

Transition Mithril Era Reader adapter to CardanoChain in devnet #1428

jpraynaud opened this issue Jan 11, 2024 · 0 comments · Fixed by #1431
Assignees
Labels
devX 🌞 Developer experience

Comments

@jpraynaud
Copy link
Member

jpraynaud commented Jan 11, 2024

Why

We want to include Mithril era markers stored on the Cardano chain in the end to end test. This will help thoroughly test this mechanism. Moreover, the issue #1315 is replacing the CardanoCliChainObserver and we want to test the new PallasChainObserver before merging the PR #1403.

What

Make the signer and aggregator nodes use read the era markers from the Cardano chain (instead of relying on the dummy adapter) in the end to end test.

How

  • Upgrade era generate-tx-datum command of the aggregator to export datum to a file
  • Upgrade the devnet with the generation of a script that writes era markers to the Cardano chain
  • Upgrade the end to end test so that signer and aggregator nodes use the CardanoChain era reader adapter
@jpraynaud jpraynaud self-assigned this Jan 11, 2024
@jpraynaud jpraynaud added the devX 🌞 Developer experience label Jan 12, 2024
@jpraynaud jpraynaud changed the title Transition Era Reader adapter to CardanoChain in devnet Transition Mithril Era Reader adapter to CardanoChain in devnet Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devX 🌞 Developer experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant