Cosmos-sdk: gaiacli stake edit-validator gas requirements increase with time

[iammelea]

gaiacli stake edit-validator --address-validator=cosmosaccaddr1yfseqtj5sjhzz2q2ym09jym4h4nc4yevae0jp2 --moniker="melea-trust_www.cosmos-trust.com" --website="http://cosmos-trust.com/" --keybase-sig="4F" --details="Hola from Catalonia, Spain = to the Cosmos! Telegram user= (t)" --chain-id=gaia-7001 --from=meleatrust --gas=400000

Defaulting to account number: 100
Defaulting to next sequence number: 1
Password to sign with 'meleatrust':
ERROR: checkTx failed: (65548) === ABCI Log ===
Codespace: 1
Code: 12
ABCICode: 65548
Error: --= Error =--
Data: common.FmtError{format:"out of gas in location: SetPerByte", args:[]interface {}(nil)}
Msg Traces:
--= /Error =--

=== /ABCI Log ===

Solution :
gaiacli stake edit-validator --address-validator=cosmosaccaddr1yfseqtj5sjhzz2q2ym09jym4h4nc4yevae0jp2 --moniker="melea-trust_www.cosmos-trust.com" --website="http://cosmos-trust.com/" --keybase-sig="4BE49EABAA41B8BF" --details="Hola from Catalonia, Spain = to the Cosmos! Telegram user= (@Meleatrust)" --chain-id=gaia-7001 --from=meleatrust --gas=100000000
Defaulting to account number: 100
Defaulting to next sequence number: 1
Password to sign with 'meleatrust':
Committed at block 5873. Hash: F13902B1A696CDC7E0252477F6B4CB3FB82FEFD4

[rigelrozanski]

What do you mean after "setup"? Looks like there was just not enough gas sent in the first transaction? - But yeah that error message could be improved

[iammelea]

i want to said (setup =fixing)
and i don't explain well,

one diferent validator can edit validator with (--gas 400000) ok.

and some time later the same day i try the same but i cant. and have to pin up (--gas=100000000)

for can work.

i don't understand why i have to add to more gas in the same day for make the same.

[rigelrozanski]

hmm not sure this is a staking bug or something with the gas-counting... @cwgoes want to take a look into this one

[cwgoes]

I don't think it's a bug, but we still need to come up with a well-thought-out gas pricing model. handleMsgEditValidator reads and writes the whole validator struct, so gas cost will be proportional to the current size of the struct plus the size of the updated struct. If you add a long description you'll need to pay more gas.

[rigelrozanski]

Okay cool, what you're saying makes sense I'm going to close this issue for now unless a more specific problem comes out of this.