-
Notifications
You must be signed in to change notification settings - Fork 106
all: rewrite bfgd and popmd, remove bssd #396
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.
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
326 commits
Select commit
Hold shift + click to select a range
e852b62
remove more bss mentions
marcopeereboom 0590732
Oops missed this prometheus panic
marcopeereboom 27f2431
Basic framework for handling keystone request
marcopeereboom 43289a7
Add some nice error handling
marcopeereboom ded229a
Slight cleanup
marcopeereboom 54cb215
Do not love the go in new
marcopeereboom 366e1ce
rig up gozer in bfg
marcopeereboom 02d1c95
add dumpkeystones to btctool
marcopeereboom e6314d0
Add method not allowed
marcopeereboom 38a0315
Grab keystone from tbc
marcopeereboom f38e59b
fixup rpc
marcopeereboom 779e1a2
err
marcopeereboom a9371ec
Return custom type for abbreviated keystone and fix various nits in w…
marcopeereboom 1da1251
linter fixes
AL-CT 8c1c9fa
skip old test
AL-CT 76ab3b0
add tx creation to wallet
AL-CT e33732f
add (untested) broadcasting to gozers
AL-CT dd3639a
fix blockstream broadcast
AL-CT 002ed36
cleanup rebase mess
AL-CT 10814a5
Only print errors at startup then print only one reconnect message
marcopeereboom 5ac2a7d
Fix some error messages
marcopeereboom 8c06e78
fix tx creation change amount
AL-CT b2161a0
fix test fee estimation
AL-CT f261e1a
add multi input tx create
AL-CT 94587f7
skip Tx Execution test on OP_RETURN pkscripts
AL-CT 5b9a065
dumb fee calculator
marcopeereboom 0a727ec
Rename vbytes since witness is bakes in
marcopeereboom 961f5d2
Start bringing back mempool
marcopeereboom f4dd96d
Create mempoolTx type to have a bit more info in mempool
marcopeereboom 6d9cd90
it's ok to ignore error on getinv since the randomness of the network…
marcopeereboom ad5eaf8
Unexport several footgun functions
marcopeereboom 3f8b853
meh mempool shit
marcopeereboom 4576e48
add mempool fee estimation
AL-CT 6771fee
Add ability to pull tx' from blockstream to test
marcopeereboom 78a97e8
Ask for witness data
marcopeereboom 55fe998
Calculate fees as we go, this is dumb but share with antonio
marcopeereboom 4ab6ecd
dump mempool
marcopeereboom af29c49
don't panic, from antonio
marcopeereboom d48e9f2
Don't dump fees for now; it is time to add an rpc call
marcopeereboom 9caeb18
cleanup mempool fees
AL-CT 8186348
change mempool fee format and ramp up fees
AL-CT 436a771
fix fee estimate rpc call from tbc
AL-CT 5877f72
fix linter and replace feesByBlockHash
AL-CT 9eb21ca
skip coinbase in feesbyblockhash
AL-CT 0ef028b
add mempool info rpc call
AL-CT f5390b8
add popm geth connection
AL-CT c925527
remove spew
AL-CT 00fb86d
add fake opgeth test
AL-CT 132c03c
cleanup
AL-CT e64f8a7
add mock keystone subscription and test
AL-CT a06b0aa
linter fix
AL-CT d1c5b82
add btc height to gozer
AL-CT ba80202
popm: add test mining of keystones
AL-CT d44d6e6
popm: re-add proper keystone mining handling
AL-CT 76e0601
popm: fix subscription lock
AL-CT a60f724
popm: re-add remaining tests
AL-CT a23e4fe
bfg: initial finality logic implementation
AL-CT aa7cb2a
bfg: add multiple queries to bfg test
AL-CT f3fb0a5
bfg: handle opgeth call and connection race
AL-CT 26b76a8
bfg: add batch tbc call, op-geth sends full keystone
AL-CT 397d6f7
bfg: test using free port
AL-CT 823212e
linter: header fixes post-rebase
AL-CT 5e9feb3
linter: remove yamllinter for removed files
AL-CT 83f909d
Make code less wide and start cleaning up old pop miner logic
marcopeereboom b3e9112
bfg: fix old test using retry threshold
AL-CT 7627fbe
Comments
marcopeereboom 0ee24e3
Make l2keystones key of hash
marcopeereboom 07fc3bf
unjoin adding and processing keystones
marcopeereboom 256267c
unjoin adding and processing keystones
marcopeereboom 16b147e
some comments and cleanup
marcopeereboom 07355dc
fuck a channel off forever
marcopeereboom 17b2c6c
get rid of more functions that do one thing
marcopeereboom 635b5b9
Deal with XXX
marcopeereboom 16cf7f9
Whitespace
marcopeereboom 9c7630e
move some code to avoid dragons
marcopeereboom c82ef9d
bit of cleanup
marcopeereboom a3ac1af
Show antonio
marcopeereboom cf020ae
remove debug
marcopeereboom e9b056a
A bit more cleanup
marcopeereboom 956e5a8
constify
marcopeereboom 772fa97
A bit more cleanup
marcopeereboom d735034
Get rid of superfluous structure field
marcopeereboom a73bd1d
doco
marcopeereboom 9ff1a2b
Rename New to Run to be consistent
marcopeereboom f98243c
Import opgeth branch
AL-CT bca8d0b
wallet: mock integration test gozer
AL-CT 5a6d339
start adding new keystone handling
marcopeereboom d5965b1
remove more stuff
marcopeereboom 992a70f
popm: refactor base test to reflect new changes
AL-CT 1d25816
make code more obvious
marcopeereboom 5221217
oops, antonio wanted this
marcopeereboom 720da1c
share keystones between mock tbc and opgeth
AL-CT 66493c6
Add test utility functions
AL-CT 818358f
oops forgot one
AL-CT c01da5d
connect mocked keystones using EPHash
AL-CT c2e979a
be loud fo verification
marcopeereboom 28befd5
fixup keystones generation mock
AL-CT fe6a685
Add first pass at keystones state cache
marcopeereboom a03b72f
make hashes more readable
AL-CT cf46ff3
globaly insert rehydrated cache map
marcopeereboom 258055d
first pass at testing new cache and flseh out cache
marcopeereboom bff675b
Show antonio
marcopeereboom a81f7cf
fix the dumbest bug in history
AL-CT deea138
Add state code and tests
marcopeereboom 14ed5c0
oops remove timer go routine for mining; it's now part of the main loop
marcopeereboom 4ef4b94
XXX
marcopeereboom f1dcf2e
use var for delay retry
marcopeereboom 5afca2a
fix test
AL-CT 38c8690
Fix cache expiry
marcopeereboom 942af4e
remove panic
marcopeereboom 6442637
Be more quiet
marcopeereboom acdd886
linter
marcopeereboom cf7ae94
linter
marcopeereboom 9d52301
Checked if we a re synced before starting to build mempool, also add …
marcopeereboom f3cdf6c
dedup copy paste
marcopeereboom a2ad6a2
more synced
marcopeereboom 7615c29
Fixup mempool
marcopeereboom fbf6a66
add expiry and keystone cache checks to test
AL-CT 35002db
XXX for antonio
marcopeereboom 1760925
Add a filter to the wallet to exclude mempool utxos
marcopeereboom 503f3ff
slight re-org of tests
AL-CT 6be9d2a
test fixes
AL-CT 39cb2fd
mempool reaping test
AL-CT 2e1aa73
improve test readability and logging
AL-CT 4092822
extra reaping test
AL-CT 209f7a6
mempool remove test
AL-CT 04172a0
add utxo filter test
AL-CT 72c3e58
oops
AL-CT bb8ae82
Travel utxos+index into mempool
marcopeereboom 32cbf7c
New filter mechanism for mempool
marcopeereboom 4ec8bc8
fix opgeth dependency and tests for utxo changes
AL-CT 63c58dc
prevent mempool filter if mempool not enabled
AL-CT e3f1578
add mempool utxo filter test
AL-CT b41ec52
Add mempool filter benchmark
AL-CT 2227914
limit size of utxos in benchmark
AL-CT 61f6c6a
Add utxo filtering integration with popm test
AL-CT 65365d6
fix some XXX
marcopeereboom 02a0747
Remove unused code
marcopeereboom fc48de8
add full mock integration test
AL-CT 3078dad
Commit this framework so that i can be tested over rpc
marcopeereboom c69ca8c
add a debug command to hemictl
marcopeereboom fcd24ee
implement canonical forward keystone grabber
marcopeereboom d7e3df2
Set height and return proper errors
marcopeereboom d0f746d
keep track of duration
marcopeereboom aaa8dc7
use proper tx index
marcopeereboom 088b190
govet shizzle
marcopeereboom 2cfcc9e
testing progress
ClaytonNorthey92 bc72f21
fix build
ClaytonNorthey92 321e825
revert test changes and unprivate keystoneTxs method
AL-CT 24240e8
Remove goto that read a bit difficult
marcopeereboom eae2bf9
make tbc block by abrev hash request batched
AL-CT 796ab77
merge back marcos changes to mempool
AL-CT 93f7d8d
fix test races and function names
AL-CT f6b4435
handle error in reconcileKeystones
AL-CT caf46a5
separate btc tip call in tbc BlockByl2AbrevHashes request
AL-CT e70f1cf
Shuffle order of operations to retain idempotence in the various rpc …
marcopeereboom b7b4be0
sigh typo
marcopeereboom 7a38792
Clayton's bandaid fixes, notes, and re-adding localnet tests
ClaytonNorthey92 384a731
check context
ClaytonNorthey92 24da872
removed _e2e_fixme folder
ClaytonNorthey92 30f33b4
Only have one RPC listen address env variable
marcopeereboom 376d039
add wait timeout
ClaytonNorthey92 38596e2
Reword some functions to make them slightly more readable
marcopeereboom c0ec563
latest op-geth
ClaytonNorthey92 2c7d102
change e2e popmd docker config to use opgeth websocket
AL-CT 93efd93
update op-geth
ClaytonNorthey92 7abef60
Add poll to keystone subscription loop
marcopeereboom 45ed9cb
NOPE
marcopeereboom 9b0371a
Complain about missing Bytes() call in btcutil
marcopeereboom 8cce296
debugging ci
ClaytonNorthey92 7241cae
mock test changes
AL-CT bd8aca5
remove unique utxo map from wallet
marcopeereboom 0062135
add logging to mock
AL-CT 62d076a
fix localnet test file
ClaytonNorthey92 e2ad2ce
add popm opgeth disconnect test
AL-CT 3a2785f
address fix
ClaytonNorthey92 a19f398
tsk tsk, please use gofmt on save
marcopeereboom 736a37f
Rewrite hydrateKeystone logic.
marcopeereboom 7500913
rebase fixes
AL-CT f3686e5
fix monitor deps
AL-CT f981671
add inheritance loop logic to bfg
AL-CT e2ab7fb
removed _e2e_fixme folder, removed extra prune in workflow file
ClaytonNorthey92 a61c8ef
re-added e2e tests file
ClaytonNorthey92 e0c3c63
all: fix testutil package usage, remove local service/testutil
joshuasing 12212f6
better way to getPort when not using httptest
ClaytonNorthey92 a96fb54
linter fixes maybe?
ClaytonNorthey92 e2bfd3a
let bfg start and walk chain
ClaytonNorthey92 b779c78
ignoring minor linting issues to get things working
ClaytonNorthey92 27c6f43
fix compilation error
ClaytonNorthey92 48e943a
fix some linting
ClaytonNorthey92 47ea0b6
use testutil opgeth + always used queried-for l2keystone in finality
ClaytonNorthey92 e0653bd
nil pointer check
ClaytonNorthey92 e35ce5c
Update api/bfgapi/bfgapi.go
marcopeereboom 82276d8
Apply suggestions from code review
marcopeereboom 6300634
uncomment testutil things
AL-CT 90836dc
fixed CI pipeline with test+code changes (#547)
ClaytonNorthey92 e27b896
un-skipped test testing effective height
ClaytonNorthey92 c78ebe3
add some more test debug logging
AL-CT 3d6d5ae
first attempt at short circuit
AL-CT e950f76
fix test
AL-CT 8866ea7
test error fix after rebase
AL-CT 7a64c09
prevent geth circular dependency
AL-CT fb6f63e
correct calculations for short circuit
AL-CT e6b8feb
re-add workflows
AL-CT 6de2182
yamllinter fix
AL-CT adf4b3a
remove mock logging
AL-CT 9df055e
limit descendant checks in finality query
AL-CT 8ec08d0
make short circuit optional
AL-CT c9c7a7b
only replace keystone in finality response once
AL-CT 40090fa
comment out short circuit
AL-CT fe252b0
make popm tests shorter
AL-CT e267aed
Merge branch 'main' into toni/bfg-rework
joshuasing 93df699
Merge branch 'main' into toni/bfg-rework
joshuasing 075c0aa
linter fix and add kss by height to mock tbc
AL-CT 28e5577
Merge branch 'main' into toni/bfg-rework
joshuasing ce9f48a
fix method signatures in popm post-merge
AL-CT c760e77
e2e: revert optimis-stack-legacy.Dockerfile changes
joshuasing c764a71
ci: add missing newline to localnet-test.yml
joshuasing ea598bd
golangci: revert adding go-ethereum to replace-allow-list
joshuasing 5c9044f
readme: remove bssd prerequisites section
joshuasing 7272dfa
service/bfg: rename HttpError -> HTTPError
joshuasing 6820f2a
service/bfg: use rand.Read in random
joshuasing abf361b
service/bfg: fix HTTP errors
joshuasing 885892a
service/bfg: use HTTP status 500 for internal errors
joshuasing c4808e1
service/bfg: tidy up InternalErrorf
joshuasing ab1a31f
service/bfg: tidy up logic in handleKeystoneFinality for loop
joshuasing 1b07bb8
service/bfg: more logic tidy up
AL-CT 0596adc
e2e/monitor: fix use of context.Background in tests
joshuasing eda8741
e2e/monitor: run golangci-lint
joshuasing 386a80c
cmd/bfgd: add opgeth url env variable and initial documentation
AL-CT 906aac8
service/bfg: make TBC default gozer type
AL-CT 767c4c4
reverted back to correctly using heminetwork directory in localnet
ClaytonNorthey92 2c4bc2b
use correct version of optimism+get rid of bss config
ClaytonNorthey92 53bb182
update op-geth commit
ClaytonNorthey92 6e6bb0e
increase popm test deadline
AL-CT 70c040f
updated localnet commits
ClaytonNorthey92 8e65ee4
service/popm: fix race in popm tests
AL-CT cc709a7
cmd/bfgd: change btc source env var description
AL-CT a81a64a
use uint64 to handle higher values
ClaytonNorthey92 1eb9f64
cmd/hemictl: add minimal hemictl documentation
AL-CT 914cac1
update test timeouts and remove unused function testing
AL-CT 67d6eda
service/bfg: linter fix
AL-CT 68207ff
update op-stack commits to latest in hemi branches
ClaytonNorthey92 3168781
goreleaser: remove bssd
joshuasing bdbb620
docker: remove bssd
joshuasing 9403e95
github/labeler: remove bss label
joshuasing 249ba6e
readme: remove bssd, reformat
joshuasing f6e4c5d
re-added localnode (#575)
ClaytonNorthey92 4bf16a6
gozer: fix tbcGozer fragile test
AL-CT 39475cf
Merge branch 'main' into toni/bfg-rework
joshuasing 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
cmd/bfgd: add opgeth url env variable and initial documentation
- Loading branch information
commit 386a80c5592d0e6c3ebadd4568f9329668524686
There are no files selected for viewing
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.