Skip to content

[IBC] chore: integrate IBC packet forward middleware (PFM)#1568

Open
bryanchriswhite wants to merge 63 commits intomainfrom
feat/ibc/packet-forward-middleware
Open

[IBC] chore: integrate IBC packet forward middleware (PFM)#1568
bryanchriswhite wants to merge 63 commits intomainfrom
feat/ibc/packet-forward-middleware

Conversation

@bryanchriswhite
Copy link
Copy Markdown
Collaborator

@bryanchriswhite bryanchriswhite commented Jun 25, 2025 •
edited
Loading

Summary

Integrates IBC packet forward middleware.

Issue

Dependencies

Type of change

Select one or more from the following:

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Documentation
  • Other (specify)

Sanity Checklist

  • I have updated the GitHub Issue Metadata: assignees, reviewers, labels, project, iteration and milestone
  • For docs: make docusaurus_start
  • For small changes: make go_develop_and_test and make test_e2e
  • For major changes: devnet-test-e2e label to run E2E tests in CI
  • For migration changes: make test_e2e_oneshot
  • 'TODO's, configurations and other docs

@bryanchriswhite bryanchriswhite self-assigned this Jun 25, 2025
@bryanchriswhite bryanchriswhite added on-chain On-chain business logic ibc everything IBC related labels Jun 25, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Shannon Jun 25, 2025
github-actions[bot]
github-actions bot reviewed Jun 25, 2025
View reviewed changes
Comment thread app/ibc.go Outdated
@bryanchriswhite bryanchriswhite moved this from 📋 Backlog to 🏗 In progress in Shannon Jun 25, 2025
@bryanchriswhite bryanchriswhite linked an issue Jun 25, 2025 that may be closed by this pull request
[IBC] Kickoff IBC Support & Integrations with Axelar & ICF #1201
Closed
9 tasks
@bryanchriswhite bryanchriswhite force-pushed the feat/ibc/packet-forward-middleware branch from a0464fa to 6d51e4d Compare June 30, 2025 18:04
@bryanchriswhite bryanchriswhite mentioned this pull request Jun 30, 2025
Merged
okdas and others added 5 commits July 17, 2025 15:36
@okdas
defaults for new config structure
4ca75e9
@okdas
remove the default from ibc.tilt
7f1349d
@bryanchriswhite
docs: comprehensive IBC documentation overhaul
dc9196a 
  - Add new IBC documentation pages:
    - ica.md: Interchain Accounts implementation and usage
    - pfm.md: Packet Forward Middleware with client failure resilience
    - integration.md: IBC integration patterns and best practices
    - testing.md: Testing strategies for IBC functionality

  - Enhance existing documentation:
    - localnet.md: Fix make targets, improve troubleshooting formatting
    - overview.md: Updated structure and cross-references
    - _category_.json: Add new pages to navigation

  - Key improvements:
    - Consistent ASCII diagrams with proper character encoding
    - Comprehensive troubleshooting sections with proper formatting
    - Added PFM client failure scenarios and recovery strategies
    - Removed non-existent make targets from ICA documentation

  🤖 Generated with [Claude Code](https://claude.ai/code)

  Co-Authored-By: Claude <noreply@anthropic.com>
@bryanchriswhite
chore: integrate IBC packet forward middleware
7a6c33e
@bryanchriswhite
chore: add packet forward middleware flags
fc22a42
@bryanchriswhite bryanchriswhite force-pushed the feat/ibc/packet-forward-middleware branch from f0206db to fc22a42 Compare July 22, 2025 09:03
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 22, 2025

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks.

You may need to run make trigger_ci to submit an empty commit that'll trigger the tests.

GCP workloads (requires changing the namespace to 1568)
Grafana network dashboard for devnet-issue-1568

@github-actions github-actions bot added devnet push-image CI related - pushes images to ghcr.io labels Jul 22, 2025
@bryanchriswhite bryanchriswhite marked this pull request as ready for review July 22, 2025 09:04
@bryanchriswhite bryanchriswhite moved this from 🏗 In progress to 👀 In review in Shannon Jul 22, 2025
@github-actions github-actions bot removed the devnet label Jul 22, 2025
@bryanchriswhite
Merge remote-tracking branch 'pokt/main' into chore/localnet/tilt
838841b 
* pokt/main: (34 commits)
  [GOVERNANCE][PARAMS][CUTTM] Updating global CUTTM per Foundation requirement (#1727)
  Update docusaurus/docs/4_develop/upgrades/4_upgrade_list.md w/ v0.1.28 MainNet block
  Updating bulk_params_main/tokenomics_params.json so proposer gets rewards again
  Update upgrade_tx_v0.1.28_main.json
  [Morse Recovery] Add recovery allowlist entries for issues #1718 and #1719 (#1723)
  [Upgrade]: Update v0.1.28 upgrade list docs (#1722)
  [Upgrade] v0.1.28 upgrade txs (#1721)
  [Upgrade] Add v0.1.28 upgrade plan (#1720)
  [CometBFT] Use Pocket's fork temporarily (#1717)
  Use use_cometbft_fork_v038.17 (#1714)
  [RelayMiner] Add session lifetime caching (#1713)
  Update recovery allowlist for recovery-8aug2025 (#1716)
  [Tokenomics] Validator proper decoding (#1710)
  [Session Rollovers] First attempt at solving session rollovers from RelayMiner's POV (#1705)
  [RelayMiner] Add comprehensive relay timing and gRPC call metrics (#1703)
  [Shared Module] Increase `ComputeUnitsPerRelayMax` (#1709)
  [RelayMiner] Improve non-2xx errors visibility, do not mine 5xx (#1695)
  [Governance] - Updated Params to match the ones in the forum  (#1707)
  [RelayMiner] Improve RelayMiner timeout handling and test reliability (#1691)
  [Morse Recovery] Updated Allowlist for Recovery (#1696)
  ...
@bryanchriswhite
Merge branch 'chore/localnet/tilt' into chore/ibc-axelar
f2ec1f5 
* chore/localnet/tilt: (36 commits)
  [GOVERNANCE][PARAMS][CUTTM] Updating global CUTTM per Foundation requirement (#1727)
  Update docusaurus/docs/4_develop/upgrades/4_upgrade_list.md w/ v0.1.28 MainNet block
  Updating bulk_params_main/tokenomics_params.json so proposer gets rewards again
  Update upgrade_tx_v0.1.28_main.json
  [Morse Recovery] Add recovery allowlist entries for issues #1718 and #1719 (#1723)
  [Upgrade]: Update v0.1.28 upgrade list docs (#1722)
  [Upgrade] v0.1.28 upgrade txs (#1721)
  [Upgrade] Add v0.1.28 upgrade plan (#1720)
  [CometBFT] Use Pocket's fork temporarily (#1717)
  Use use_cometbft_fork_v038.17 (#1714)
  [RelayMiner] Add session lifetime caching (#1713)
  Update recovery allowlist for recovery-8aug2025 (#1716)
  [Tokenomics] Validator proper decoding (#1710)
  [Session Rollovers] First attempt at solving session rollovers from RelayMiner's POV (#1705)
  [RelayMiner] Add comprehensive relay timing and gRPC call metrics (#1703)
  [Shared Module] Increase `ComputeUnitsPerRelayMax` (#1709)
  [RelayMiner] Improve non-2xx errors visibility, do not mine 5xx (#1695)
  [Governance] - Updated Params to match the ones in the forum  (#1707)
  [RelayMiner] Improve RelayMiner timeout handling and test reliability (#1691)
  [Morse Recovery] Updated Allowlist for Recovery (#1696)
  ...

# Conflicts:
#	tiltfiles/ibc.tilt
@bryanchriswhite
Merge branch 'refs/heads/chore/ibc-axelar' into docs/ibc/localnet
361cd50 
* refs/heads/chore/ibc-axelar: (36 commits)
  [GOVERNANCE][PARAMS][CUTTM] Updating global CUTTM per Foundation requirement (#1727)
  Update docusaurus/docs/4_develop/upgrades/4_upgrade_list.md w/ v0.1.28 MainNet block
  Updating bulk_params_main/tokenomics_params.json so proposer gets rewards again
  Update upgrade_tx_v0.1.28_main.json
  [Morse Recovery] Add recovery allowlist entries for issues #1718 and #1719 (#1723)
  [Upgrade]: Update v0.1.28 upgrade list docs (#1722)
  [Upgrade] v0.1.28 upgrade txs (#1721)
  [Upgrade] Add v0.1.28 upgrade plan (#1720)
  [CometBFT] Use Pocket's fork temporarily (#1717)
  Use use_cometbft_fork_v038.17 (#1714)
  [RelayMiner] Add session lifetime caching (#1713)
  Update recovery allowlist for recovery-8aug2025 (#1716)
  [Tokenomics] Validator proper decoding (#1710)
  [Session Rollovers] First attempt at solving session rollovers from RelayMiner's POV (#1705)
  [RelayMiner] Add comprehensive relay timing and gRPC call metrics (#1703)
  [Shared Module] Increase `ComputeUnitsPerRelayMax` (#1709)
  [RelayMiner] Improve non-2xx errors visibility, do not mine 5xx (#1695)
  [Governance] - Updated Params to match the ones in the forum  (#1707)
  [RelayMiner] Improve RelayMiner timeout handling and test reliability (#1691)
  [Morse Recovery] Updated Allowlist for Recovery (#1696)
  ...
@bryanchriswhite
Merge branch 'docs/ibc/localnet' into feat/ibc/packet-forward-middleware
76194c2 
* docs/ibc/localnet: (36 commits)
  [GOVERNANCE][PARAMS][CUTTM] Updating global CUTTM per Foundation requirement (#1727)
  Update docusaurus/docs/4_develop/upgrades/4_upgrade_list.md w/ v0.1.28 MainNet block
  Updating bulk_params_main/tokenomics_params.json so proposer gets rewards again
  Update upgrade_tx_v0.1.28_main.json
  [Morse Recovery] Add recovery allowlist entries for issues #1718 and #1719 (#1723)
  [Upgrade]: Update v0.1.28 upgrade list docs (#1722)
  [Upgrade] v0.1.28 upgrade txs (#1721)
  [Upgrade] Add v0.1.28 upgrade plan (#1720)
  [CometBFT] Use Pocket's fork temporarily (#1717)
  Use use_cometbft_fork_v038.17 (#1714)
  [RelayMiner] Add session lifetime caching (#1713)
  Update recovery allowlist for recovery-8aug2025 (#1716)
  [Tokenomics] Validator proper decoding (#1710)
  [Session Rollovers] First attempt at solving session rollovers from RelayMiner's POV (#1705)
  [RelayMiner] Add comprehensive relay timing and gRPC call metrics (#1703)
  [Shared Module] Increase `ComputeUnitsPerRelayMax` (#1709)
  [RelayMiner] Improve non-2xx errors visibility, do not mine 5xx (#1695)
  [Governance] - Updated Params to match the ones in the forum  (#1707)
  [RelayMiner] Improve RelayMiner timeout handling and test reliability (#1691)
  [Morse Recovery] Updated Allowlist for Recovery (#1696)
  ...
Olshansk added a commit to pokt-network/helm-charts that referenced this pull request Sep 2, 2025
@bryanchriswhite @github-actions @Olshansk
chore: Osmosis validator chart (#70)
5bf32c3 
## Summary

Adds a chart for a localnet [Osmosis chain node](https://osmosis.zone/) for exercising IBC connectivity

```
Pocket cosmos-sdk: v0.53.x
Pocket ibc-go: v8.7.0
Osmosis cosmos-sdk: v0.50.x (fork; https://github.com/osmosis-labs/osmosis/blob/main/go.mod#L319) 
Osmosis ibc-go: v8.7.x (https://github.com/osmosis-labs/osmosis/blob/main/go.mod#L33)
```
## Dependents 

- pokt-network/poktroll#1272
- pokt-network/poktroll#1568
- pokt-network/poktroll#1609

## Issue

- pokt-network/poktroll#1201 


Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Daniel Olshansky <olshansky.daniel@gmail.com>
@github-actions github-actions bot added the consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. label Sep 3, 2025
@Olshansk Olshansk changed the base branch from docs/ibc/localnet to main September 3, 2025 00:11
@github-actions github-actions bot removed the consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. label Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@Olshansk Olshansk Awaiting requested review from Olshansk

@okdas okdas Awaiting requested review from okdas

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

Assignees

@bryanchriswhite bryanchriswhite

Labels

devex Developer experience related improvements. ibc everything IBC related on-chain On-chain business logic push-image CI related - pushes images to ghcr.io

Projects

Shannon
Status: 👀 In review

Milestone

Interoperability

Development

Successfully merging this pull request may close these issues.

[IBC] Kickoff IBC Support & Integrations with Axelar & ICF

3 participants

@bryanchriswhite @Olshansk @okdas