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 RL genesis #4406

Closed
wants to merge 2 commits into from
Closed

Backport RL genesis #4406

wants to merge 2 commits into from

Conversation

nicolaslara
Copy link
Contributor

Replaces #4404

What is the purpose of the change

Backport RL genesis

Brief Changelog

backport #4340

Testing and Verifying

all tests should 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 rate limits in upgrade

* added upgrade test

* refactor rate limiting tests and added rate limits pre-upgrade

* lint

* goimports

* experiments for E2E

* post merge lints

* added RL before upgrade

* just get the latest contract

* experiments

* updated wasm file

* remove unnecessary prints

* bad check

* experiments without e2e

* removed unnecessary fmt

* param space initialization for rate limiting (#4360)

* param space initialization for rate limiting

* lint

* better test

* properly initialize empty params

* fix typo

* adding RL to E2E

* removing RL again from E2E

* debug

* genesis

* genesis 2

* add genesis and solve params bug

* uncomment setup rate limiting

* added configurable gov module

* clean up

* clean up and genesis test

* upgrade handler

* genesis clean up

* clean up e2e

* wire querier

* route fix

* fix router

* debug things

* fix init genesis bug

* push fix

* lint

* fix

* clean up upgrades

* testing propper params after upgrade in E2E

* "properly" unmarshaling the parm

* goimports

* allowing for types other than objects in cosmwasm queries

* fix check for param reset

* remove unnecessary prints

* compiled with the proper vesion of workspace optimizer

* Revert "compiled with the proper vesion of workspace optimizer"

This reverts commit ab5cf6c.

* added length check

* params experiment for test with state-export

---------

Co-authored-by: Roman <roman@osmosis.team>
Co-authored-by: Roman <ackhtariev@gmail.com>
(cherry picked from commit a1e2b3d)

# Conflicts:
#	app/keepers/modules.go
#	app/modules.go
#	app/upgrades/v15/export_test.go
#	app/upgrades/v15/upgrade_test.go
#	app/upgrades/v15/upgrades.go
#	tests/e2e/configurer/chain/commands.go
#	tests/e2e/e2e_test.go
@nicolaslara nicolaslara changed the title Nicolas/backport rl genesis backport RL genesis Feb 23, 2023
@nicolaslara nicolaslara changed the title backport RL genesis Backport RL genesis Feb 23, 2023
@github-actions github-actions bot added C:app-wiring Changes to the app folder C:CLI C:docs Improvements or additions to documentation C:simulator Edits simulator or simulations C:x/epochs C:x/gamm Changes, features and bugs related to the gamm module. C:x/incentives C:x/lockup C:x/mint C:x/pool-incentives C:x/superfluid C:x/tokenfactory C:x/twap Changes to the twap module C:x/txfees T:build T:CI labels Feb 23, 2023
@nicolaslara nicolaslara changed the base branch from main to v14.x February 23, 2023 14:36
@mergify
Copy link
Contributor

mergify bot commented Feb 23, 2023

⚠️ The sha of the head commit of this PR conflicts with #4404. Mergify cannot evaluate rules on this PR. ⚠️

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 C:CLI C:docs Improvements or additions to documentation C:simulator Edits simulator or simulations C:x/epochs C:x/gamm Changes, features and bugs related to the gamm module. C:x/incentives C:x/lockup C:x/mint C:x/pool-incentives C:x/superfluid C:x/tokenfactory C:x/twap Changes to the twap module C:x/txfees T:build T:CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant