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

fix: refetch missing packet #168

Draft
wants to merge 66 commits into
base: main
Choose a base branch
from
Draft

fix: refetch missing packet #168

wants to merge 66 commits into from

Conversation

viveksharmapoudel
Copy link
Collaborator

  • Implements packetHeights and missingPacketReceipt method in icon & wasm ibc module.
  • Refractor code wasm types.
  • Implements flush logic for contract based IBC module.

@viveksharmapoudel
fix: add missing mutex in query proof
2c07f05
@viveksharmapoudel
chore: add retry logic in wasm query method
244d363
@viveksharmapoudel
fix: add delay for pathprocessor sync
fb281b4
@viveksharmapoudel
fix: remove tp when getting client state
5c72d19
@viveksharmapoudel
fix: adding defer in queryWasmProof
df20a84
@viveksharmapoudel
fix: change to processheight
afbcf30
@viveksharmapoudel
fix: limit max block fetch at a time in wasm processor
f581195
@viveksharmapoudel
fix: btp block update miss
a2b9187
@viveksharmapoudel
fix:add values
9c3d3bd
@viveksharmapoudel
fix: the logic
4660bc8
@viveksharmapoudel
fix: queue logic
95396e6
@viveksharmapoudel
fix:refractor
6b8db31
@viveksharmapoudel
fix: implement mutex in queue
5145d20
@viveksharmapoudel
fix: add prevconsensusstateheight method and use it to fetch trusted …
87a9477 
…height
@viveksharmapoudel
fix: add method missing receipt methods
68f8257
@viveksharmapoudel
Merge branch 'main' of github.com:icon-project/ibc-relay into fix/ref…
ef7b18c 
…etch-missing-packet
@viveksharmapoudel
build fix
2b90d8b
@viveksharmapoudel
feat: flush logic for contract based chains
ed33f74
@viveksharmapoudel
feat: add query method in chain provider for flush logic
aed90fe
@viveksharmapoudel
feat: implement additional query method in cosmos & penumber
544277e
@viveksharmapoudel
feat: implement additional queries in icon & wasm chain
73613e6
@viveksharmapoudel
feat: getNextSequence Improvize logic
c05f1c6
@viveksharmapoudel
refract: wasm types code optimize
7efd902
@viveksharmapoudel
feat: implement next sequence send method
e08da36
@viveksharmapoudel
refract: unmarshal of contract byte response
6776c85
@viveksharmapoudel
feat: add querySendPacketByHeight wasm chain
c7e0534
@viveksharmapoudel
feat: query send packet by height query added
cb5cd9b
@viveksharmapoudel
fix: get next seq based on height
60ef21f
@viveksharmapoudel
fix: remove comment
807283c
@viveksharmapoudel
feat: new btp queue structure
2e9818c
@izyak izyak linked an issue Oct 2, 2023 that may be closed by this pull request
Add mechanism to flush packets #157
Open
@izyak
Copy link
Collaborator

izyak commented Oct 2, 2023 •
edited
Loading

If I have more than 10 packets to be flushed, it seems to be flushing only the first 10 packets.
Even on the next flush cycle, it's not sending the missing packets

@viveksharmapoudel
Copy link
Collaborator Author

If I have more than 10 packets to be flushed, it seems to be flushing only the first 10 packets. Even on the next flush cycle, it's not sending the missing packets

@viveksharmapoudel viveksharmapoudel added enhancement New feature or request and removed feature labels Oct 5, 2023
@viveksharmapoudel
chore: fix merge conflict
07f4dbd
@viveksharmapoudel
feat: add btp proof context change flag
f92cede
@viveksharmapoudel
chore: remove unwanted
3daee45
@viveksharmapoudel
chore: add flag for proof context from height
539c435
@viveksharmapoudel
feat: update proof context change logic and remove timer to save height
5d06c2d
@viveksharmapoudel
feat: add proof context update logic in start only
7183883
@viveksharmapoudel
feat: iconchainprocessor startheight fetch from context
274cd87
@viveksharmapoudel
feat: remove start height from IconproviderConfig-> always run from l…
6a83caa 
…atest height
@viveksharmapoudel
feat: icon find proofContextChange from height
683a508
@viveksharmapoudel
feat: wasm chain add concurrency and remove start height
e857e48
@viveksharmapoudel
chore: add constant
e3a21f8
@viveksharmapoudel
chore: helper method added
a902765
@viveksharmapoudel
chore: removed HeightSnapshot logic
b3b44e8
@viveksharmapoudel
chore: update example config
c584766
@viveksharmapoudel
chore: reduce sleep time
c222afa
@viveksharmapoudel
fix: add btp height of only it doesn't exist
436d733
@pragyanshrestha-ibriz pragyanshrestha-ibriz marked this pull request as draft March 25, 2024 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@izyak izyak Awaiting requested review from izyak

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add mechanism to flush packets
2 participants
@viveksharmapoudel @izyak