Merge branch 'develop' into jack/query-tx-reorg

.circleci/config.yml

@@ -137,6 +137,24 @@ jobs:
  export PATH="$GOBIN:$PATH"
  make test_sim_gaia_fast
 
+ test_sim_gaia_import_export:
+ <<: *defaults
+ parallelism: 1
+ steps:
+ - attach_workspace:
+ at: /tmp/workspace
+ - checkout
+ - run:
+ name: dependencies
+ command: |
+ export PATH="$GOBIN:$PATH"
+ make get_vendor_deps
+ - run:
+ name: Test Gaia import/export simulation
+ command: |
+ export PATH="$GOBIN:$PATH"
+ make test_sim_gaia_import_export
+
  test_sim_gaia_multi_seed:
  <<: *defaults
  parallelism: 1
@@ -259,6 +277,9 @@ workflows:
  - test_sim_gaia_fast:
  requires:
  - setup_dependencies
+ - test_sim_gaia_import_export:
+ requires:
+ - setup_dependencies
  - test_sim_gaia_multi_seed:
  requires:
  - setup_dependencies

CHANGELOG.md

@@ -1,5 +1,62 @@
 # Changelog
 
+## 0.26.0
+
+BREAKING CHANGES
+
+* Gaia
+ * [gaiad init] [\#2602](https://github.com/cosmos/cosmos-sdk/issues/2602) New genesis workflow
+
+* SDK
+ * [simulation] [\#2665](https://github.com/cosmos/cosmos-sdk/issues/2665) only argument to simulation.Invariant is now app
+
+* Tendermint
+ * Upgrade to version 0.26.0
+
+FEATURES
+
+* Gaia CLI (`gaiacli`)
+ * [cli] [\#2569](https://github.com/cosmos/cosmos-sdk/pull/2569) Add commands to query validator unbondings and redelegations
+ * [cli] [\#2569](https://github.com/cosmos/cosmos-sdk/pull/2569) Add commands to query validator unbondings and redelegations
+ * [cli] [\#2524](https://github.com/cosmos/cosmos-sdk/issues/2524) Add support offline mode to `gaiacli tx sign`. Lookups are not performed if the flag `--offline` is on.
+ * [cli] [\#2558](https://github.com/cosmos/cosmos-sdk/issues/2558) Rename --print-sigs to --validate-signatures. It now performs a complete set of sanity checks and reports to the user. Also added --print-signature-only to print the signature only, not the whole transaction.
+ * [cli] [\#2704](https://github.com/cosmos/cosmos-sdk/pull/2704) New add-genesis-account convenience command to populate genesis.json with genesis accounts.
+
+* SDK
+ * [\#1336](https://github.com/cosmos/cosmos-sdk/issues/1336) Mechanism for SDK Users to configure their own Bech32 prefixes instead of using the default cosmos prefixes.
+
+IMPROVEMENTS
+
+* Gaia
+ * [\#2637](https://github.com/cosmos/cosmos-sdk/issues/2637) [x/gov] Switched inactive and active proposal queues to an iterator based queue
+
+* SDK
+ * [\#2573](https://github.com/cosmos/cosmos-sdk/issues/2573) [x/distribution] add accum invariance
+ * [\#2556](https://github.com/cosmos/cosmos-sdk/issues/2556) [x/mock/simulation] Fix debugging output
+ * [\#2396](https://github.com/cosmos/cosmos-sdk/issues/2396) [x/mock/simulation] Change parameters to get more slashes
+ * [\#2617](https://github.com/cosmos/cosmos-sdk/issues/2617) [x/mock/simulation] Randomize all genesis parameters
+ * [\#2669](https://github.com/cosmos/cosmos-sdk/issues/2669) [x/stake] Added invarant check to make sure validator's power aligns with its spot in the power store.
+ * [\#1924](https://github.com/cosmos/cosmos-sdk/issues/1924) [x/mock/simulation] Use a transition matrix for block size
+ * [\#2660](https://github.com/cosmos/cosmos-sdk/issues/2660) [x/mock/simulation] Staking transactions get tested far more frequently
+ * [\#2610](https://github.com/cosmos/cosmos-sdk/issues/2610) [x/stake] Block redelegation to and from the same validator
+ * [\#2652](https://github.com/cosmos/cosmos-sdk/issues/2652) [x/auth] Add benchmark for get and set account
+ * [\#2685](https://github.com/cosmos/cosmos-sdk/issues/2685) [store] Add general merkle absence proof (also for empty substores)
+ * [\#2708](https://github.com/cosmos/cosmos-sdk/issues/2708) [store] Disallow setting nil values
+
+BUG FIXES
+
+* Gaia
+ * [\#2670](https://github.com/cosmos/cosmos-sdk/issues/2670) [x/stake] fixed incorrect `IterateBondedValidators` and split into two functions: `IterateBondedValidators` and `IterateLastBlockConsValidators`
+ * [\#2691](https://github.com/cosmos/cosmos-sdk/issues/2691) Fix local testnet creation by using a single canonical genesis time
+ * [\#2648](https://github.com/cosmos/cosmos-sdk/issues/2648) [gaiad] Fix `gaiad export` / `gaiad import` consistency, test in CI
+
+* SDK
+ * [\#2625](https://github.com/cosmos/cosmos-sdk/issues/2625) [x/gov] fix AppendTag function usage error
+ * [\#2677](https://github.com/cosmos/cosmos-sdk/issues/2677) [x/stake, x/distribution] various staking/distribution fixes as found by the simulator
+ * [\#2674](https://github.com/cosmos/cosmos-sdk/issues/2674) [types] Fix coin.IsLT() impl, coins.IsLT() impl, and renamed coins.Is\* to coins.IsAll\* (see [\#2686](https://github.com/cosmos/cosmos-sdk/issues/2686))
+ * [\#2711](https://github.com/cosmos/cosmos-sdk/issues/2711) [x/stake] Add commission data to `MsgCreateValidator` signature bytes.
+ * Temporarily disable insecure mode for Gaia Lite
+
 ## 0.25.0
 
 *October 24th, 2018*

Gopkg.toml

@@ -36,7 +36,7 @@
 
 [[override]]
  name = "github.com/tendermint/tendermint"
- version = "v0.26.1-rc0" # TODO replace w/ 0.26.1
+ version = "v0.26.1-rc3" # TODO replace w/ 0.26.1
 
 ## deps without releases:
 

Makefile

@@ -171,6 +171,15 @@ test_sim_gaia_fast:
  @echo "Running quick Gaia simulation. This may take several minutes..."
  @go test ./cmd/gaia/app -run TestFullGaiaSimulation -SimulationEnabled=true -SimulationNumBlocks=500 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=10 -v -timeout 24h
 
+test_sim_gaia_import_export:
+ @echo "Running Gaia import/export simulation. This may take several minutes..."
+ @go test ./cmd/gaia/app -run TestGaiaImportExport -SimulationEnabled=true -SimulationNumBlocks=50 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=4 -v -timeout 24h
+ @go test ./cmd/gaia/app -run TestGaiaImportExport -SimulationEnabled=true -SimulationNumBlocks=50 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=11 -v -timeout 24h
+ @go test ./cmd/gaia/app -run TestGaiaImportExport -SimulationEnabled=true -SimulationNumBlocks=50 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=12 -v -timeout 24h
+ @go test ./cmd/gaia/app -run TestGaiaImportExport -SimulationEnabled=true -SimulationNumBlocks=50 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=13 -v -timeout 24h
+ @go test ./cmd/gaia/app -run TestGaiaImportExport -SimulationEnabled=true -SimulationNumBlocks=50 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=414 -v -timeout 24h
+ @go test ./cmd/gaia/app -run TestGaiaImportExport -SimulationEnabled=true -SimulationNumBlocks=50 -SimulationBlockSize=200 -SimulationCommit=true -SimulationSeed=4142 -v -timeout 24h
+
 test_sim_gaia_multi_seed:
  @echo "Running multi-seed Gaia simulation. This may take awhile!"
  @bash scripts/multisim.sh 25
@@ -250,4 +259,5 @@ localnet-stop:
 check_tools check_dev_tools get_tools get_dev_tools get_vendor_deps draw_deps test test_cli test_unit \
 test_cover test_lint benchmark devdoc_init devdoc devdoc_save devdoc_update \
 build-linux build-docker-gaiadnode localnet-start localnet-stop \
-format check-ledger test_sim_gaia_nondeterminism test_sim_modules test_sim_gaia_fast test_sim_gaia_multi_seed update_tools update_dev_tools
+format check-ledger test_sim_gaia_nondeterminism test_sim_modules test_sim_gaia_fast \
+test_sim_gaia_multi_seed test_sim_gaia_import_export update_tools update_dev_tools

PENDING.md

@@ -6,30 +6,25 @@ BREAKING CHANGES
 
 * Gaia CLI (`gaiacli`)
  * [cli] [\#2728](https://github.com/cosmos/cosmos-sdk/pull/2728) Seperate `tx` and `query` subcommands by module
+ * [cli] [\#2727](https://github.com/cosmos/cosmos-sdk/pull/2727) Fix unbonding command flow
 
 * Gaia
- * [gaiad init] \#2602 New genesis workflow
 
 * SDK
- * [simulation] \#2665 only argument to simulation.Invariant is now app
+ * [\#2752](https://github.com/cosmos/cosmos-sdk/pull/2752) Don't hardcode bondable denom.
 
 * Tendermint
- * Upgrade to version 0.26.0
+
 
 FEATURES
 
 * Gaia REST API (`gaiacli advanced rest-server`)
 
 * Gaia CLI (`gaiacli`)
- * [cli] [\#2569](https://github.com/cosmos/cosmos-sdk/pull/2569) Add commands to query validator unbondings and redelegations
- * [cli] [\#2569](https://github.com/cosmos/cosmos-sdk/pull/2569) Add commands to query validator unbondings and redelegations
- * [cli] [\#2524](https://github.com/cosmos/cosmos-sdk/issues/2524) Add support offline mode to `gaiacli tx sign`. Lookups are not performed if the flag `--offline` is on.
- * [cli] [\#2558](https://github.com/cosmos/cosmos-sdk/issues/2558) Rename --print-sigs to --validate-signatures. It now performs a complete set of sanity checks and reports to the user. Also added --print-signature-only to print the signature only, not the whole transaction.
 
 * Gaia
 
 * SDK
- * (#1336) Mechanism for SDK Users to configure their own Bech32 prefixes instead of using the default cosmos prefixes.
 
 * Tendermint
 
@@ -39,22 +34,12 @@ IMPROVEMENTS
 * Gaia REST API (`gaiacli advanced rest-server`)
 
 * Gaia CLI (`gaiacli`)
+ * [\#2749](https://github.com/cosmos/cosmos-sdk/pull/2749) Add --chain-id flag to gaiad testnet
 
 * Gaia
- - #2637 [x/gov] Switched inactive and active proposal queues to an iterator based queue
 
 * SDK
- - \#2573 [x/distribution] add accum invariance
- - \#2556 [x/mock/simulation] Fix debugging output
- - \#2396 [x/mock/simulation] Change parameters to get more slashes
- - \#2617 [x/mock/simulation] Randomize all genesis parameters
- - \#2669 [x/stake] Added invarant check to make sure validator's power aligns with its spot in the power store.
- - \#1924 [x/mock/simulation] Use a transition matrix for block size
- - \#2660 [x/mock/simulation] Staking transactions get tested far more frequently
- - \#2610 [x/stake] Block redelegation to and from the same validator
- - \#2652 [x/auth] Add benchmark for get and set account
- - \#2685 [store] Add general merkle absence proof (also for empty substores)
- - \#2708 [store] Disallow setting nil values
+ - [x/mock/simulation] [\#2720] major cleanup, introduction of helper objects, reorganization
 
 * Tendermint
 
@@ -66,13 +51,10 @@ BUG FIXES
 * Gaia CLI (`gaiacli`)
 
 * Gaia
- - \#2670 [x/stake] fixed incorrect `IterateBondedValidators` and split into two functions: `IterateBondedValidators` and `IterateLastBlockConsValidators`
- - \#2691 Fix local testnet creation by using a single canonical genesis time
-
+  * [\#2723] Use `cosmosvalcons` Bech32 prefix in `tendermint show-address`
+  * [\#2742](https://github.com/cosmos/cosmos-sdk/issues/2742) Fix time format of TimeoutCommit override 
+ 
 * SDK
- - \#2625 [x/gov] fix AppendTag function usage error
- - \#2677 [x/stake, x/distribution] various staking/distribution fixes as found by the simulator
- - \#2674 [types] Fix coin.IsLT() impl, coins.IsLT() impl, and renamed coins.Is\* to coins.IsAll\* (see \#2686)
- - \#2711 [x/stake] Add commission data to `MsgCreateValidator` signature bytes.
 
 * Tendermint
+ * [\#2797](https://github.com/tendermint/tendermint/pull/2797) AddressBook requires addresses to have IDs; Do not crap out immediately after sending pex addrs in seed mode

client/lcd/lcd_test.go

@@ -31,6 +31,7 @@ import (
  "github.com/cosmos/cosmos-sdk/x/gov"
  "github.com/cosmos/cosmos-sdk/x/slashing"
  "github.com/cosmos/cosmos-sdk/x/stake"
+ stakeTypes "github.com/cosmos/cosmos-sdk/x/stake/types"
 )
 
 func init() {
@@ -265,15 +266,15 @@ func TestCoinSend(t *testing.T) {
  coins := acc.GetCoins()
  mycoins := coins[0]
 
- require.Equal(t, "steak", mycoins.Denom)
+ require.Equal(t, stakeTypes.DefaultBondDenom, mycoins.Denom)
  require.Equal(t, initialBalance[0].Amount.SubRaw(1), mycoins.Amount)
 
  // query receiver
  acc = getAccount(t, port, receiveAddr)
  coins = acc.GetCoins()
  mycoins = coins[0]
 
- require.Equal(t, "steak", mycoins.Denom)
+ require.Equal(t, stakeTypes.DefaultBondDenom, mycoins.Denom)
  require.Equal(t, int64(1), mycoins.Amount.Int64())
 
  // test failure with too little gas
@@ -326,7 +327,7 @@ func DisabledTestIBCTransfer(t *testing.T) {
  coins := acc.GetCoins()
  mycoins := coins[0]
 
- require.Equal(t, "steak", mycoins.Denom)
+ require.Equal(t, stakeTypes.DefaultBondDenom, mycoins.Denom)
  require.Equal(t, initialBalance[0].Amount.SubRaw(1), mycoins.Amount)
 
  // TODO: query ibc egress packet state
@@ -514,7 +515,7 @@ func TestValidatorQuery(t *testing.T) {
 }
 
 func TestBonding(t *testing.T) {
- name, password, denom := "test", "1234567890", "steak"
+ name, password, denom := "test", "1234567890", stakeTypes.DefaultBondDenom
  addr, seed := CreateAddr(t, name, password, GetKeyBase(t))
 
  cleanup, valPubKeys, operAddrs, port := InitializeTestLCD(t, 2, []sdk.AccAddress{addr})
@@ -564,7 +565,7 @@ func TestBonding(t *testing.T) {
  // sender should have not received any coins as the unbonding has only just begun
  acc = getAccount(t, port, addr)
  coins = acc.GetCoins()
- require.Equal(t, int64(40), coins.AmountOf("steak").Int64())
+ require.Equal(t, int64(40), coins.AmountOf(stakeTypes.DefaultBondDenom).Int64())
 
  unbonding := getUndelegation(t, port, addr, operAddrs[0])
  require.Equal(t, "30", unbonding.Balance.Amount.String())
@@ -663,11 +664,11 @@ func TestDeposit(t *testing.T) {
 
  // query proposal
  proposal = getProposal(t, port, proposalID)
- require.True(t, proposal.GetTotalDeposit().IsEqual(sdk.Coins{sdk.NewInt64Coin("steak", 10)}))
+ require.True(t, proposal.GetTotalDeposit().IsEqual(sdk.Coins{sdk.NewInt64Coin(stakeTypes.DefaultBondDenom, 10)}))
 
  // query deposit
  deposit := getDeposit(t, port, proposalID, addr)
- require.True(t, deposit.Amount.IsEqual(sdk.Coins{sdk.NewInt64Coin("steak", 10)}))
+ require.True(t, deposit.Amount.IsEqual(sdk.Coins{sdk.NewInt64Coin(stakeTypes.DefaultBondDenom, 10)}))
 }
 
 func TestVote(t *testing.T) {
@@ -861,7 +862,7 @@ func doSendWithGas(t *testing.T, port, seed, name, password string, addr sdk.Acc
  sequence := acc.GetSequence()
  chainID := viper.GetString(client.FlagChainID)
  // send
- coinbz, err := cdc.MarshalJSON(sdk.NewInt64Coin("steak", 1))
+ coinbz, err := cdc.MarshalJSON(sdk.NewInt64Coin(stakeTypes.DefaultBondDenom, 1))
  if err != nil {
  panic(err)
  }
@@ -947,7 +948,7 @@ func doIBCTransfer(t *testing.T, port, seed, name, password string, addr sdk.Acc
  "account_number":"%d",
  "sequence":"%d"
  }
- }`, "steak", name, password, chainID, accnum, sequence))
+ }`, stakeTypes.DefaultBondDenom, name, password, chainID, accnum, sequence))
 
  res, body := Request(t, port, "POST", fmt.Sprintf("/ibc/testchain/%s/send", receiveAddr), jsonStr)
  require.Equal(t, http.StatusOK, res.StatusCode, body)
@@ -1096,7 +1097,7 @@ func doDelegate(t *testing.T, port, seed, name, password string,
  "account_number":"%d",
  "sequence":"%d"
  }
- }`, delAddr, valAddr, "steak", amount, name, password, chainID, accnum, sequence))
+ }`, delAddr, valAddr, stakeTypes.DefaultBondDenom, amount, name, password, chainID, accnum, sequence))
 
  res, body := Request(t, port, "POST", fmt.Sprintf("/stake/delegators/%s/delegations", delAddr), jsonStr)
  require.Equal(t, http.StatusOK, res.StatusCode, body)
@@ -1339,15 +1340,15 @@ func doSubmitProposal(t *testing.T, port, seed, name, password string, proposerA
  "description": "test",
  "proposal_type": "Text",
  "proposer": "%s",
- "initial_deposit": [{ "denom": "steak", "amount": "%d" }],
+ "initial_deposit": [{ "denom": "%s", "amount": "%d" }],
  "base_req": {
  "name": "%s",
  "password": "%s",
  "chain_id": "%s",
  "account_number":"%d",
  "sequence":"%d"
  }
- }`, proposerAddr, amount, name, password, chainID, accnum, sequence))
+ }`, proposerAddr, stakeTypes.DefaultBondDenom, amount, name, password, chainID, accnum, sequence))
  res, body := Request(t, port, "POST", "/gov/proposals", jsonStr)
  require.Equal(t, http.StatusOK, res.StatusCode, body)
 
@@ -1369,15 +1370,15 @@ func doDeposit(t *testing.T, port, seed, name, password string, proposerAddr sdk
  // deposit on proposal
  jsonStr := []byte(fmt.Sprintf(`{
  "depositer": "%s",
- "amount": [{ "denom": "steak", "amount": "%d" }],
+ "amount": [{ "denom": "%s", "amount": "%d" }],
  "base_req": {
  "name": "%s",
  "password": "%s",
  "chain_id": "%s",
  "account_number":"%d",
  "sequence": "%d"
  }
- }`, proposerAddr, amount, name, password, chainID, accnum, sequence))
+ }`, proposerAddr, stakeTypes.DefaultBondDenom, amount, name, password, chainID, accnum, sequence))
  res, body := Request(t, port, "POST", fmt.Sprintf("/gov/proposals/%d/deposits", proposalID), jsonStr)
  require.Equal(t, http.StatusOK, res.StatusCode, body)