Latest ZSA demo version with asset state and multiple action groups support #94
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…wap feature flag to enable)
We want to be able to run the recognized workflows (as they are on zsa1) also on this branch: <img width="1765" height="566" alt="image" src="https://github.com/user-attachments/assets/a0b0aa89-fd83-4d72-9974-f9ad680eca60" />
Add git commit and git tag (if exists) prints for the Zebra node startup.
We want to keep the dockerfile of zebra in the zebra repo, so adding them here with adjustments and removing from tx-tool in this PR: QED-it/zcash_tx_tool#36
If a user triggering the push to ECR Github workflow doesn't input any tag in the text box we'll use the current git tag, and if not available then the current git commit hash (Instead of failing)
Fix git info fetching of the git tag and commit hash by using docker arguments
Update the deploy ecs workflow on zsa-integration-demo by the latest file from zsa1 branch
* Defines and implements the issued asset state types
* Adds issued assets to the finalized state
* Validates issuance actions and burns before committing blocks to a non-finalized chain.
* Adds `issued_assets` fields on `ChainInner` and `ContextuallyValidatedBlock`
* Adds issued assets map to non-finalized chains
* adds new column family to list of state column families
* Updates AssetState, AssetStateChange, IssuedAssetsOrChange, & SemanticallyVerifiedBlock types, updates `IssuedAssetsChange::from_transactions()` method return type
* Fixes tests by computing an `IssuedAssetsChange` for conversions to CheckpointVerifiedBlock
* fixes finalization checks
* Adds documentation to types and methods in `asset_state` module, fixes several bugs.
* Fix compilation errors that appeared after the previous merge
* Avoid using NonEmpty in orchard_zsa/issuance
* Fix BurnItem serialization/deserializartioon errors (use LE instead of BE for amount, read amount after asset base)
* Make a minor fix and add FIXME comment in orchard_flavor_ext.rs
* Fix the sign of burn value in SupplyChange::add in orchard_zsa/asset_state, add a couple of FIXMEs
* Fix the 'transactions must have only one burn item per asset base' error in the function of the crate (this may not be a fully correct fix). Add a couple of FIXME comments explaining the problem.
* Use NoteValue from the orchard crate for BurnItem amount instead of u64 to prevent serialization errors and enable defining BurnItem in orchard, facilitating its reuse along with related functions
* Use BurnItem::from instead of try_from
* Fix a compilation error for the previous commit ('Use BurnItem::from instead of try_from')
* Fix a compilation error for the previous commit ('Use BurnItem::from instead of try_from') (2)
* Modify ValueCommitment::with_asset to accept value as a NoteValue instead of amount (with_asset is used to process orchard burn) - this allows avoiding the use of try_into for burn in binding_verification_key function
* Adds TODOs
* Adds state request/response variants for querying asset states
* Adds a `getassetstate` RPC method
* Adds snapshot test
* Addesses some FIXMEs and replaces a couple others with TODOs.
* Removes `issued_assets_change` field from `SemanticallyVerifiedBlock`
* Temporarily disable specific Clippy checks for Rust 1.83.0 compatibility
* Disable clippy warning about doc comment for empty line
* Update Orchard ZSA consensus tests to calculate and check asset supply
* Rename ZSA workflow tests (including file, constant and variable names) to Orchard ZSA
* Add amount method to BurnItem and make BurnItem pub (visible for other crates)
* Fix Orchard ZSA workflow tests to make it compilable with getblocktemplate-rpcs feature enabled
* Fix clippy error
* Add rust-toolchain.toml with Rust version 1.82.0 to avoid clippy errors came with Rust 1.83.0
* Fix revert_chain_with function in zebra-state to support V6/OrchardZSA
* Minor fix in comments
* Rename transaction_to_fake_v5 function to transaction_to_fake_min_v5 and panic if V6 passed into it
* Minor fixes in comments
* Revert "Minor fix in comments"
This reverts commit 59fec59.
* Revert "Rename transaction_to_fake_v5 function to transaction_to_fake_min_v5 and panic if V6 passed into it"
This reverts commit 2ce58ef.
* Revert " Minor fixes in comments"
This reverts commit fac3abd.
* Fix remaining merge conflicts
* Fix compilation erros
* Fix clippy warning
* Fix compilation errors appeared after the previous merge
* Fix compilation error
* Fix compilation errors in zebra-state happened without tx-v6 feature flag enabled
* Allow finalizing issued assets via the issue action when no notes are provided and the finalize flag is set to true
* Refactor orchard_workflow_blocks_zsa.rs (zebra-test crate) to read test data from files, introduce and use OrchardWorkflowBlock there
* Fix clippy errors
* Copy tests from zsa-issued-assets-tests here and fix compilation errors
* Temporarily comment out verify_issuance_blocks_test test (it fails now)
* Rename remaining tx-v6 flags to tx_v6
---------
Co-authored-by: Arya <aryasolhi@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Latest ZSA demo version with asset state and multiple action groups support.