Latest demo version with ZSA support #58
Draft
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
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>
dmidem
force-pushed
the
zsa-integration-demo
branch
from
11dcfc2 to
1e07ba0
Compare
## Motivation We want to know post running the CD step what exact version was deployed, in a single and centralized step. ### Specifications & References The PR adds an information print step in the CD (push to ecr) pipeline of ECS. ## Solution Add a print step. ### Tests I ran it to try. ### Follow-up Work Might need to add a similar test to tx-tool. ### PR Author's Checklist <!-- If you are the author of the PR, check the boxes below before making the PR ready for review. --> - [x] The PR name will make sense to users. - [ ] The PR provides a CHANGELOG summary. [Not needed] - [x] The solution is tested. - [x] The documentation is up to date. - [ ] The PR has a priority label. [Not needed] ### PR Reviewer's Checklist <!-- If you are a reviewer of the PR, check the boxes below before approving it. --> - [ ] The PR Author's checklist is complete. - [ ] The PR resolves the issue.
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.
The current version of the source branch (
zsa-integration-demo) is a copy of thezsa-integration-consensusbranch with several additional commits related to Docker configuration needed specifically to run the demo.