This repository was archived by the owner on Dec 17, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 51
Manage tx validation and propagation with its files #85
Merged
+1,802
−1,156
Merged
Changes from 1 commit
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
ece7b89
first implementation
musitdev 00831fd
correct file management. Work locally. Before remote test
musitdev 0056630
correct p2p peer list init
musitdev 53d5423
add some logs to follow tx
musitdev 30f9e48
correct rebase error
musitdev 557bc58
add checksum to VM file and test
musitdev 992de81
add type state to file and Tx
musitdev 4fc59b2
correct some issues in file path
musitdev 072235e
add some logs
musitdev 6428170
update proof generated file path
musitdev 3dd91cb
remove comments
musitdev 2a714a1
add comment, add timeout and error management to http download connec…
musitdev fdf618f
correct http url definition
musitdev ff9e291
add timeout to http send
musitdev c58ad9d
add better download error login and log file checksum
musitdev b19c14f
correct build
musitdev 423d573
desactivate checksum verification for downloaded files
musitdev 14bbd3c
add some logs for checksum
musitdev 4557493
change hasher for VM file and activate checksum verification
musitdev 48aec37
correct host detection for download
musitdev 32c5c0e
disable proof and verifcation workflow and remove some logs
musitdev 7bce8e6
rebase from master and correct download file issue
musitdev f8be0df
rebase from master and correct download file issue
musitdev 3b59b32
pass clippy and tests
musitdev 803cfe7
correct clippy
musitdev a9a4d87
do some cleaning
musitdev 3a1f086
correct PR remarks
musitdev 2a76a8e
Merge branch 'main' into manage_tx_consistency
tuommaki f422f14
add newline to be like the original file
musitdev a0216cc
change generated file name from uuid to checksum
musitdev 02a1347
Avoid to move download existing file
musitdev d45fb19
correct clippy
musitdev d68428b
remove asset manager call to test
musitdev d44e527
correct some path issue
musitdev ff72aed
add verify tx execution in the workflow.
musitdev ee8a376
add mark deploy tx as executed
musitdev 6d563e0
remove the dbg! that panic
musitdev e8cce96
Merge branch 'main' into manage_tx_consistency
tuommaki 29a99d8
change verif tx workflow management and change logs
musitdev a168896
Merge branch 'manage_tx_consistency' of https://github.com/gevulotnet…
musitdev 8766908
add some logs
musitdev 9952880
Merge branch 'main' into manage_tx_consistency
tuommaki 65b0b97
Terminate VM on get_task() when already assigned with Task (#96)
tuommaki e4481f6
revert .gitignore change
musitdev d9b2683
Merge branch 'manage_tx_consistency' of https://github.com/gevulotnet…
musitdev 80998df
correct mutex lock issue
musitdev 3aaab1f
add some logs
musitdev 37d7eb0
remove asset management and some logs
musitdev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
update proof generated file path
commit 6428170de4fb1e79f08fd6e61a6719cc105d0c90
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -119,13 +119,10 @@ impl File<ProofVerif> { | |||||
} | ||||||
|
||||||
pub fn get_relatif_path(&self, tx_hash: Hash) -> PathBuf { | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
let mut file_path = Path::new(&self.name); | ||||||
if file_path.is_absolute() { | ||||||
file_path = file_path.strip_prefix("/").unwrap(); //unwrap tested in is_absolute | ||||||
} | ||||||
|
||||||
//get uuid from file name | ||||||
let uuid = Path::new(&self.name).file_name().unwrap(); | ||||||
let mut path = PathBuf::from(tx_hash.to_string()); | ||||||
path.push(file_path); | ||||||
path.push(uuid); | ||||||
path | ||||||
} | ||||||
|
||||||
|
This file contains 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
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -447,12 +447,14 @@ impl Transaction<TxReceive> { | |||||||
.collect() | ||||||||
} | ||||||||
Payload::Proof { files, .. } | Payload::Verification { files, .. } => { | ||||||||
//generated file during execution has already been moved. No Download. | ||||||||
tracing::trace!( | ||||||||
"Payload::Proof tx:{} is_from_tx_exec_result:{}", | ||||||||
"Payload::Proof tx:{} is_from_tx_exec_result:{} files.len:{}", | ||||||||
self.hash.to_string(), | ||||||||
self.state.is_from_tx_exec_result() | ||||||||
self.state.is_from_tx_exec_result(), | ||||||||
files.len(), | ||||||||
); | ||||||||
|
||||||||
//generated file during execution has already been moved. No Download. | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
if self.state.is_from_tx_exec_result() { | ||||||||
Ok(vec![]) | ||||||||
} else { | ||||||||
|
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.