Cosmos gas multiplier params

proto/cosmos/params/types/types.proto

@@ -13,6 +13,12 @@ message FeesParams {
       [(gogoproto.nullable) = false, (gogoproto.castrepeated) = "github.com/cosmos/cosmos-sdk/types.DecCoins"];
 }
 
+message CosmosGasParams {
+  uint64 cosmos_gas_multiplier_numerator   = 1;
+  uint64 cosmos_gas_multiplier_denominator = 2;
+}
+
 message GenesisState {
   FeesParams fees_params = 1 [(gogoproto.nullable) = false];
+  CosmosGasParams cosmos_gas_params = 2 [(gogoproto.nullable) = false];
 }

x/auth/ante/expected_keepers.go

@@ -24,4 +24,6 @@ type FeegrantKeeper interface {
 type ParamsKeeper interface {
 	SetFeesParams(ctx sdk.Context, feesParams paramtypes.FeesParams)
 	GetFeesParams(ctx sdk.Context) paramtypes.FeesParams
+	SetCosmosGasParams(ctx sdk.Context, cosmosGasParams paramtypes.CosmosGasParams)
+	GetCosmosGasParams(ctx sdk.Context) paramtypes.CosmosGasParams
 }

x/params/keeper/keeper.go

@@ -59,6 +59,30 @@ func (k Keeper) GetFeesParams(ctx sdk.Context) types.FeesParams {
 	return feesParams
 }
 
+func (k Keeper) SetCosmosGasParams(ctx sdk.Context, cosmosGasParams types.CosmosGasParams) {
+	cosmosGasParams.Validate()
+	subspace, exist := k.GetSubspace(types.ModuleName)
+	if !exist {
+		panic("subspace params should exist")
+	}
+	subspace.Set(ctx, types.ParamStoreKeyCosmosGasParams, cosmosGasParams)
+}
+
+func (k Keeper) GetCosmosGasParams(ctx sdk.Context) types.CosmosGasParams {
+	subspace, _ := k.GetSubspace(types.ModuleName)
+
+	if !subspace.Has(ctx, types.ParamStoreKeyCosmosGasParams) {
+		defaultParams := *types.DefaultCosmosGasParams()
+		k.SetCosmosGasParams(ctx, defaultParams)
+		return defaultParams
+	}
+
+	bz := subspace.GetRaw(ctx, types.ParamStoreKeyCosmosGasParams)
+	var cosmosGasParams types.CosmosGasParams
+	json.Unmarshal(bz, &cosmosGasParams)
+	return cosmosGasParams
+}
+
 // Logger returns a module-specific logger.
 func (k Keeper) Logger(ctx sdk.Context) log.Logger {
 	return ctx.Logger().With("module", "x/"+proposal.ModuleName)

x/params/keeper/migrations.go

@@ -0,0 +1,23 @@
+package keeper
+
+import (
+	sdk "github.com/cosmos/cosmos-sdk/types"
+	"github.com/cosmos/cosmos-sdk/x/params/types"
+)
+
+// Migrator is a struct for handling in-place store migrations.
+type Migrator struct {
+	keeper Keeper
+}
+
+// NewMigrator returns a new Migrator.
+func NewMigrator(keeper Keeper) Migrator {
+	return Migrator{keeper: keeper}
+}
+
+// Migrate1to2 migrates from version 1 to 2.
+func (m Migrator) Migrate1to2(ctx sdk.Context) error {
+	defaultGenesis := types.DefaultGenesis()
+	m.keeper.SetCosmosGasParams(ctx, defaultGenesis.CosmosGasParams)
+	return nil
+}

x/params/keeper/migrations_test.go

@@ -0,0 +1,22 @@
+package keeper_test
+
+import (
+	"testing"
+
+	pk "github.com/cosmos/cosmos-sdk/x/params/keeper"
+	"github.com/cosmos/cosmos-sdk/x/params/types"
+
+	"github.com/stretchr/testify/require"
+)
+
+func TestMigrate1to2(t *testing.T) {
+	_, ctx, _, _, keeper := testComponents()
+	m := pk.NewMigrator(keeper)
+	err := m.Migrate1to2(ctx)
+	require.Nil(t, err)
+	cosmosGasParams := keeper.GetCosmosGasParams(ctx)
+	// ensure set to defaults
+	defaultParams := types.DefaultGenesis()
+	require.Equal(t, defaultParams.CosmosGasParams.CosmosGasMultiplierNumerator, cosmosGasParams.CosmosGasMultiplierNumerator)
+	require.Equal(t, defaultParams.CosmosGasParams.CosmosGasMultiplierDenominator, cosmosGasParams.CosmosGasMultiplierDenominator)
+}

x/params/module.go

@@ -120,6 +120,8 @@ func (am AppModule) LegacyQuerierHandler(legacyQuerierCdc *codec.LegacyAmino) sd
 // module-specific gRPC queries.
 func (am AppModule) RegisterServices(cfg module.Configurator) {
 	proposal.RegisterQueryServer(cfg.QueryServer(), am.keeper)
+	m := keeper.NewMigrator(am.keeper)
+	_ = cfg.RegisterMigration(types.ModuleName, 1, m.Migrate1to2)
 }
 
 // ProposalContents returns all the params content functions used to
@@ -147,4 +149,4 @@ func (am AppModule) ExportGenesis(_ sdk.Context, _ codec.JSONCodec) json.RawMess
 }
 
 // ConsensusVersion implements AppModule/ConsensusVersion.
-func (AppModule) ConsensusVersion() uint64 { return 1 }
+func (AppModule) ConsensusVersion() uint64 { return 2 }

x/params/types/genesis.go

@@ -12,13 +12,24 @@ func DefaultFeesParams() *FeesParams {
 	}
 }
 
+func DefaultCosmosGasParams() *CosmosGasParams {
+	return &CosmosGasParams{
+		CosmosGasMultiplierNumerator:   1,
+		CosmosGasMultiplierDenominator: 1,
+	}
+}
+
 // DefaultGenesis returns the default Capability genesis state
 func DefaultGenesis() *GenesisState {
 	return &GenesisState{
-		FeesParams: *DefaultFeesParams(),
+		FeesParams:      *DefaultFeesParams(),
+		CosmosGasParams: *DefaultCosmosGasParams(),
 	}
 }
 
 func (gs GenesisState) Validate() error {
+	if err := gs.CosmosGasParams.Validate(); err != nil {
+		return err
+	}
 	return gs.FeesParams.Validate()
 }

x/params/types/params.go

@@ -1,12 +1,14 @@
 package types
 
 import (
+	"errors"
 	"fmt"
 
 	sdk "github.com/cosmos/cosmos-sdk/types"
 )
 
 var ParamStoreKeyFeesParams = []byte("FeesParams")
+var ParamStoreKeyCosmosGasParams = []byte("CosmosGasParams")
 
 func NewFeesParams(minGasPrices sdk.DecCoins) FeesParams {
 	return FeesParams{
@@ -18,6 +20,7 @@ func NewFeesParams(minGasPrices sdk.DecCoins) FeesParams {
 func ParamKeyTable() KeyTable {
 	return NewKeyTable(
 		NewParamSetPair(ParamStoreKeyFeesParams, &FeesParams{}, validateFeesParams),
+		NewParamSetPair(ParamStoreKeyCosmosGasParams, &CosmosGasParams{}, validateCosmosGasParams),
 	)
 }
 
@@ -41,3 +44,30 @@ func validateFeesParams(i interface{}) error {
 	}
 	return nil
 }
+
+func NewCosmosGasParams(multiplierNumerator uint64, multiplierDenominator uint64) CosmosGasParams {
+	return CosmosGasParams{
+		CosmosGasMultiplierNumerator:   multiplierNumerator,
+		CosmosGasMultiplierDenominator: multiplierDenominator,
+	}
+}
+
+func (cg *CosmosGasParams) Validate() error {
+	if cg.CosmosGasMultiplierDenominator == 0 {
+		return errors.New("Cosmos Gas Multiplier Denominator can not be 0")
+	}
+
+	return nil
+}
+
+func validateCosmosGasParams(i interface{}) error {
+	v, ok := i.(CosmosGasParams)
+	if !ok {
+		return fmt.Errorf("invalid parameter type: %T", i)
+	}
+
+	if err := v.Validate(); err != nil {
+		return err
+	}
+	return nil
+}