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.

Sign up for GitHub

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

Jump to bottom

Add ethereum offchain worker #335

Merged
merged 12 commits into from
Mar 23, 2020
Merged

Add ethereum offchain worker #335

merged 12 commits into from
Mar 23, 2020

Conversation

yanganto
Copy link
Contributor

@yanganto yanganto self-assigned this Mar 12, 2020
AurevoirXavier
AurevoirXavier requested changes Mar 12, 2020
View reviewed changes
Copy link
Member

@AurevoirXavier AurevoirXavier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cargo fmt

@yanganto yanganto force-pushed the feature/crab-eth-offchain branch 2 times, most recently from f284efb to a36d11f Compare March 18, 2020 11:42
@yanganto yanganto force-pushed the feature/crab-eth-offchain branch 4 times, most recently from 8713399 to 7e86a9e Compare March 20, 2020 05:20
@yanganto
add: eth offchain workder
0f2d751 
- get current eth block hight for current time stamp
- get total difficulty, parent hash
- retry in request
- build eth header
- verify eth header
- poc version: pass api key when build
@yanganto yanganto changed the title Feature/crab eth offchain Add ethereum offchain worker Mar 20, 2020
@yanganto yanganto mentioned this pull request Mar 20, 2020
Closed
@AurevoirXavier
Merge remote-tracking branch 'upstream/develop' into feature/crab-eth…
fe6e4d6 
…-offchain
@AurevoirXavier
update: format
0baae34
@AurevoirXavier
update: info and deps
030bdf9
@AurevoirXavier
add: error type
90269df 
fix: retry logic
remove: alloc
update: use `Time` trait instead `pallet_timestamp` module
@AurevoirXavier
add: mock
c005ef5 
update: tests
@AurevoirXavier
update: deps
d605a33
@AurevoirXavier
Copy link
Member

If I don't use a VPN.

WARN wasm-runtime  Ran out of free WASM instances

@yanganto
Copy link
Contributor Author

yanganto commented Mar 23, 2020 •
edited
Loading

If I don't use a VPN.

WARN wasm-runtime  Ran out of free WASM instances

This issue may not exist, when the #370 implemented and removed the retry feature.
#372 may not be implemented, because of the exhaust issue of wasm runtime.

yanganto
yanganto commented Mar 23, 2020
View reviewed changes
Copy link
Contributor Author

@yanganto yanganto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the refactor.

@yanganto yanganto merged commit 1cd0f4b into darwinia-network:develop Mar 23, 2020
This was referenced Mar 23, 2020
Merged
Merged
@AurevoirXavier AurevoirXavier linked an issue Mar 24, 2020 that may be closed by this pull request
Offchain worker #289
Closed
boundless-forest pushed a commit that referenced this pull request Aug 1, 2023
@notlesh
Remove only() call in tests (#335)
01bcc25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AurevoirXavier AurevoirXavier AurevoirXavier approved these changes

@hackfisher hackfisher hackfisher approved these changes

Assignees

@yanganto yanganto

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Offchain worker
3 participants
@yanganto @AurevoirXavier @hackfisher