This repository has been archived by the owner on Dec 17, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 49
Manage tx validation and propagation with its files #85
Merged
Merged
Changes from 40 commits
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
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
34 changes: 0 additions & 34 deletions
34
...es/node/.sqlx/query-151f420a22124e7c5f7d6a868f06c71e6f6805787aae1dea47cb248d07689498.json
This file was deleted.
Oops, something went wrong.
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
16 changes: 16 additions & 0 deletions
16
crates/node/migrations/20240215174342_add_txfile_and_checksum.sql
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 |
---|---|---|
@@ -0,0 +1,16 @@ | ||
-- Add checksum to task files | ||
-- Add a new table to store Tx files. | ||
|
||
ALTER TABLE file ADD checksum VARCHAR(64) NOT NULL; | ||
|
||
DROP TABLE IF EXISTS txfile; | ||
|
||
CREATE TABLE txfile ( | ||
tx_id VARCHAR(64) NOT NULL, | ||
name VARCHAR(256) NOT NULL, | ||
url VARCHAR(2048) NOT NULL, | ||
checksum VARCHAR(64) NOT NULL, | ||
CONSTRAINT fk_tx | ||
FOREIGN KEY (tx_id) | ||
REFERENCES transaction (hash) ON DELETE CASCADE | ||
); |
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
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.
Hmm... Why is this here? AFAIK you need these files in order to be able to compile the code that uses
sqlx
. These also need to be updated regularly when changing DB (though the build usually fails when there's need).See /~https://github.com/launchbadge/sqlx/?tab=readme-ov-file#compile-time-verification and /~https://github.com/launchbadge/sqlx/blob/main/sqlx-cli/README.md#enable-building-in-offline-mode-with-query
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.
I thought it was sqlx temporary file folder. I've one deleted by sqlx, so I thought that it's a working directory of sqlx and not part of the code. I remove.