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

Test2 bcf 2887 context propagation #538

Closed
wants to merge 1,990 commits into from
Closed

Test2 bcf 2887 context propagation #538

wants to merge 1,990 commits into from

Conversation

augustbleeds
Copy link
Collaborator

@augustbleeds augustbleeds commented Oct 10, 2024 •
edited
Loading

core ref: f4c9b9edec10085f629888ac6b50efca3fcd054b

archseer and others added 30 commits March 22, 2024 13:58
@archseer
Remove amarna workflow, it hasn't been compatible for a long time
de58f83
@augustbleeds @cfal
initial
0475442
@augustbleeds @cfal
add comment
ed3334d
@augustbleeds @cfal
retry on estimation failure if nonce error
74917cb
@augustbleeds @cfal
check for nonce error in confirmer
234bbf1
@augustbleeds @cfal
fix txm tests
b712cb0
@augustbleeds @cfal
fix function invoke
81ba1a9
@augustbleeds @cfal
fix err string handling
544287d
@augustbleeds @cfal
(debug) nonce (#369)
b512af4 
* debug logs

* publickey is for nonce manager, not contract address

* add skip validate to estimate

* change tag to pending

* make deep copy in txstore

* deep copy fix

* set x-apikey option for juno rpc

* fix bug

* add api key in few places

* fix debug err

* log

* tidy up logging

* remove nonce tracking from txstore

* clean up deep copy

* remove test
@augustbleeds @cfal
add delay to handling nonce
78ba80a
@augustbleeds @cfal
set apikey to empty string if none
25d3256
@augustbleeds @cfal
set l2 instead of l1 gas info
d175007
@augustbleeds @cfal
add key
b61787a
@augustbleeds @cfal
switch back to L1Gas
f856ea7
@augustbleeds @cfal
higher gas pad
5d4adbc
@augustbleeds @cfal
remove chainid
4120559
@augustbleeds @cfal
remove invalidate flag + fix tests
15ac8ae
@augustbleeds @cfal
simplify validate
364f1ec
@augustbleeds @cfal
add back skip validate flags for good
083b9c3
@augustbleeds @cfal
adjust token amount + add exp timer (#370)
1a685cd 
* adjust token amount + add exp timer

* add back skip_validate flag

* modify config
@augustbleeds @cfal
bubble up sync error
f968237
@augustbleeds @cfal
verbose err logging
11a2cb5
@smickovskid @cfal
Image bump
a1f4aec
@augustbleeds @cfal
pass key not address into validate
1b89658
@augustbleeds @cfal
dont error if no pending txs are found
941f2d5
@cfal
relayer/pkg/chainlink/txm/txm.go: print broadcast errors correctly
ee4ad5e
@cfal
relayer/pkg/chainlink/txm/txm.go: use smallest unconfirmed nonce for …
29d954e 
…estimation
@cfal
relayer/pkg/chainlink/txm/txm.go: print fee estimation error
24f4e47
@cfal
relayer/pkg/chainlink/txm/txstore.go: add GetSmallestUnconfirmedNonce
0acc71f
@cfal
relayer/pkg/starknet/client.go: dont create an account just to get nonce
a802fee
@jmank88 jmank88 force-pushed the BCF-2887-context-propagation branch from cb8283f to 957923c Compare October 10, 2024 18:28
@cl-sonarqube-production
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
8.3% Coverage on New Code (required ≥ 75%)

See analysis details on SonarQube

@jmank88 jmank88 force-pushed the BCF-2887-context-propagation branch 6 times, most recently from 9f331a3 to c0a9058 Compare October 11, 2024 00:22
@andrevmatos andrevmatos force-pushed the BCF-2887-context-propagation branch from e3ed7ce to 4283a0b Compare October 11, 2024 10:12
@jmank88 jmank88 force-pushed the BCF-2887-context-propagation branch 3 times, most recently from 0a982af to d7c803d Compare October 11, 2024 12:54
Base automatically changed from BCF-2887-context-propagation to develop October 11, 2024 13:29
@chray-zhang chray-zhang force-pushed the test2-BCF-2887-context-propagation branch from 2db52d5 to 16b948b Compare November 25, 2024 21:49
@chray-zhang chray-zhang requested review from a team as code owners November 25, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.