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.

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

backport: Integrate Packet Forward Middleware #4281

Merged
merged 2 commits into from
Feb 10, 2023
Merged

Conversation

nicolaslara
Copy link
Contributor

What is the purpose of the change

Backport #3911

Brief Changelog

Backport #3911

Testing and Verifying

all tests pass

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
  • Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
  • How is the feature or change documented? (not applicable / specification (x/<module>/spec/) / Osmosis docs repo / not documented)

* added initial packet forward middleware integration

* gofumpt

* updated go.mod

* correct value

* updating to match upstream

* added params

* gofumpt

* added PFM to upgrades

* go imports

* added e2e test for PFM+wasm hoks

* refactor

* refactored for reusability and added comments

* updated to latest version that allows native json

* removed unnecessary param

* reordering imports after merge

* added denom check

* added changelog

* Update tests/e2e/e2e_test.go

Co-authored-by: Roman <[email protected]>

* requiring the txs to succeed

* fixed e2e tests

---------

Co-authored-by: Roman <[email protected]>
@github-actions github-actions bot added the C:app-wiring Changes to the app folder label Feb 9, 2023
@nicolaslara
Copy link
Contributor Author

closing in favor of mergify

@nicolaslara
Copy link
Contributor Author

Reoppening this because tagging didn't seem to do anything

@nicolaslara nicolaslara reopened this Feb 10, 2023
@nicolaslara nicolaslara self-assigned this Feb 10, 2023
@ValarDragon
Copy link
Member

wdym tagging? You just add a backport label

@ValarDragon
Copy link
Member

Got the backport to work #4289 , but merging this one since it has the merge conflict fix :)

@ValarDragon ValarDragon merged commit 4062d2f into v15.x Feb 10, 2023
@ValarDragon ValarDragon deleted the nicolas/backport-pfm branch February 10, 2023 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:app-wiring Changes to the app folder
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants