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

v2.3.1 #121

Merged
merged 93 commits into from
Apr 20, 2023
Merged

v2.3.1 #121

merged 93 commits into from
Apr 20, 2023

Conversation

emidev98
Copy link
Contributor

Change log

  1. Upgrade cosmos-sdk to use 0.46.x
  2. Updated wasm to use github.com/notional-labs/wasmd v0.30.0-sdk469.4
  3. Updated ibc and packetfowardingmiddleware from v3 to v6
  4. Added ibc-hooks as a custom module
  5. Added the token-factory module
  6. Enabled ica-controller module
  7. Added alliance module

Misc updates

  1. Added scripts to run a ICA host demo
  2. Added testing script for simulating a chain upgrade

Tests performed

  • Code compiles
  • Making sure all lint issues are fixed
  • Chain simulation runs
  • Started a chain with current version of terrad, putting in a software update proposal and switching to the new binary

Integration Tests

  • Build and spinup two chains,
  • Create a relayer between this chains using rly software,
  • Alliance module test bridge token, create an alliance, delegate and validate the custom bank module integration,
  • IbcHooks deploy a cosmwasm contract, instantiate the contract, use an IBC hook to increment the counter and validate the counter has been incremented and the funds have been received,
  • ICA register a controller, send funds to that controller, execute an MsgDelegate with IBC and validate the delegation went thru correctly,
  • TokenFactory create a new denom, mint tokens from that denom, query the balances, burn tokens, do an ibc transfer using the previously created denom and validate the IBC transfer on both chains,
  • Terra Vesting Accounts create and validate a vesting account created on genesis containing the correct balances, deploy a vesting account and validate that the tokens are vesting over 5 iterations with sleep of 3 seconds.

Michal Turcan and others added 11 commits March 22, 2023 14:05
…tory, cosmos v0.46) (#106)

* Cleanup (#98)

* golangci-lint run ./... --fix

* gofumpt

* cleaner linter config

* remove init.go

* Update/readme (#110)

* Create banner.png

* Update readme.md

* Delete banner.png

* Create CODE_OF_CONDUCT.md

* Update readme.md

* fix: swagger gen

* Revert "Cleanup (#98)" (#115)

This reverts commit a97277b.

* feat: update swagger gen

---------

Co-authored-by: Jacob Gadikian <jacobgadikian@gmail.com>
Co-authored-by: Evan <87997759+evanorti@users.noreply.github.com>
Co-authored-by: emidev98 <emilian@terra.money>
Co-authored-by: emidev98 <49301655+emidev98@users.noreply.github.com>
Co-authored-by: Greg Junge <greg@terra.money>
@emidev98
Copy link
Contributor Author

@gregnuj this are all the changes for version v2.3.0 that were never merged to main.
Should we merge them?

* Update readme.md

* Update CHANGELOG.md

* Update RELEASES.md

* Update SECURITY.md

* Update README.md

* Update README.md
@gregnuj
Copy link

gregnuj commented Apr 18, 2023

I prefer main track the current mainnet release and test track the current testnet release, so my intention is to update main when we update mainnet, but I am open to feedback

@emidev98
Copy link
Contributor Author

Cool, then I assume that after the gov proposal passes and the network is upgraded to the next version this PR can be merged

emidev98 and others added 2 commits April 19, 2023 08:14
* fix: security wasmvm

* add dokerfile updates

* update wasmd

* update ledger-go dep

* chore: increase code version

---------

Co-authored-by: gregnuj <greg@terra.money>
* fix: security wasmvm

* add dokerfile updates

* update wasmd

* update ledger-go dep

* Update markdown files (#123) (#124)

* Update readme.md

* Update CHANGELOG.md

* Update RELEASES.md

* Update SECURITY.md

* Update README.md

* Update README.md

Co-authored-by: Evan <87997759+evanorti@users.noreply.github.com>

* Test (#126)

* Update markdown files (#123)

* Update readme.md

* Update CHANGELOG.md

* Update RELEASES.md

* Update SECURITY.md

* Update README.md

* Update README.md

* fix: security wasmvm (#122)

* fix: security wasmvm

* add dokerfile updates

* update wasmd

* update ledger-go dep

* chore: increase code version

---------

Co-authored-by: gregnuj <greg@terra.money>
Co-authored-by: Evan <87997759+evanorti@users.noreply.github.com>
Co-authored-by: emidev98 <emilian@terra.money>
@sascha1337
Copy link

happening 🔥

@emidev98 emidev98 changed the title v2.3.0 v2.3.1 Apr 20, 2023
@terra-money terra-money deleted a comment from codecov bot Apr 20, 2023
@codecov
Copy link

codecov bot commented Apr 20, 2023

Codecov Report

Merging #121 (3f8d83d) into main (4e583cc) will decrease coverage by 3.89%.
The diff coverage is 72.99%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   73.64%   69.75%   -3.89%     
==========================================
  Files           8        4       -4     
  Lines         664      701      +37     
==========================================
  Hits          489      489              
- Misses        149      198      +49     
+ Partials       26       14      -12     
Impacted Files Coverage Δ
app/export.go 13.33% <0.00%> (-24.32%) ⬇️
app/genesis.go 1.69% <0.00%> (-0.13%) ⬇️
app/app.go 86.00% <88.71%> (-3.77%) ⬇️

... and 2 files with indirect coverage changes

@gregnuj gregnuj merged commit 9f557e6 into main Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants