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

[LocalNet] Add gas fees #794

Open
6 tasks
Olshansk opened this issue Sep 4, 2024 · 8 comments
Open
6 tasks

[LocalNet] Add gas fees #794

Olshansk opened this issue Sep 4, 2024 · 8 comments
Assignees
Labels
on-chain On-chain business logic

Comments

@Olshansk
Copy link
Member

Olshansk commented Sep 4, 2024

Objective

Add transaction/gas fees to poktroll.
Now that Alpha and Beta TestNets are already running with fees turned on, this ticket is solely focused on ensuring gas fees on LocalNet.

Origin Document

Screenshot 2024-09-04 at 3 03 48 PM
Screenshot 2024-09-04 at 3 01 53 PM

Goals

  • Parity with Morse w.r.t Tx Fees (already achieved on Alpha/Beta TestNets)
  • Unblock poktscan from deserializing tx fees (already unblocked on Alpha/Beta TestNets)
  • Setting up the developer environment to always account for gas fees.

Deliverables

  • Set a tx/gas fee for all transactions (LocalNet only).
  • Ensure this is part of LocalNet and future TestNets (Alpha/Beta already handled; only LocalNet remains)

General deliverables

  • Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
  • Testing: Add new tests (unit and/or E2E) to the test suite.
  • Makefile: Add new targets to the Makefile to make the new functionality easier to use.
  • Documentation: Update architectural or development READMEs; use [mermaid](https://mermaid-js.github.io/mermaid/) diagrams where appropriate.

Creator: @Olshansk
Co-Owners: @okdas

@Olshansk Olshansk added the on-chain On-chain business logic label Sep 4, 2024
@Olshansk Olshansk added this to the Shannon Beta TestNet Launch milestone Sep 4, 2024
@Olshansk Olshansk added this to Shannon Sep 4, 2024
@okdas
Copy link
Member

okdas commented Sep 4, 2024

@Olshansk I've added the gas fees in #768. They're not constant, though. I can look into that to see if constant fees can be easily configured.

@okdas
Copy link
Member

okdas commented Sep 5, 2024

I flipped the config temporarily in TestNet, and

  • faucet broke as it required a fee configuration. Error:
    err: Error: Broadcasting transaction failed with code 13 (codespace: sdk). Log: insufficient fees; got:  required: 1upokt: insufficient fee
    
  • All CLI commands that we have in documentation also stopped working due to a similar issue. So we either need to change the documentation or CLI defaults.

Will take a look at both when I start working on the ticket.

@Olshansk Olshansk moved this to 🏗 In progress in Shannon Sep 6, 2024
@Olshansk
Copy link
Member Author

Olshansk commented Sep 6, 2024

@okdas Once you take a look at it, let's go with CLI defaults (if it's possible) and just document the flag in the user guide in one spot.

okdas added a commit that referenced this issue Sep 9, 2024
## Summary

`minimum-gas-prices` was increased to `0.000000001upokt` on LocalNet. We
are not ready for that yet. #794 will address the default fees values,
so all existing documentation and processes will continue working with
non-zero gas prices.

---------

Co-authored-by: Daniel Olshansky <[email protected]>
@okdas
Copy link
Member

okdas commented Sep 17, 2024

Related doc: https://www.notion.so/buildwithgrove/Proof-submission-cost-Tx-fee-vs-Spend-coins-044bcab72c1d4aef8dba523bd9bdf42d?pvs=4

okdas added a commit that referenced this issue Nov 14, 2024
## Summary

`minimum-gas-prices` was increased to `0.000000001upokt` on LocalNet. We
are not ready for that yet. #794 will address the default fees values,
so all existing documentation and processes will continue working with
non-zero gas prices.

---------

Co-authored-by: Daniel Olshansky <[email protected]>
@okdas okdas changed the title [Foundation] Add constant gas fees [Foundation] Add gas fees Nov 18, 2024
@okdas
Copy link
Member

okdas commented Nov 18, 2024

We've decided we are not going to add a constant gas fee. However, we still need to adjust minimum-gas-prices on our networks to make sure we don't miss anything gas related. Will prioritize this in preparation for beta testnet and #925.

@okdas okdas mentioned this issue Nov 21, 2024
15 tasks
@okdas
Copy link
Member

okdas commented Dec 2, 2024

Beta TestNet has been running with the fees turned on, and we're working to add them on LocalNet in #940.

@okdas
Copy link
Member

okdas commented Feb 3, 2025

No changes. Introducing gas fees to LocalNet is currently not the priority due to a larg~er work on the tests side. We're going to revisit on v0.52+ of cosmos-sdk, as there are some enhancements in the configs we can potentially utilize for fees/default values.

See #940 for more details.

@okdas okdas moved this from 🏗 In progress to 📋 Backlog in Shannon Feb 3, 2025
@Olshansk Olshansk moved this from 📋 Backlog to 🔖 Ready in Shannon Mar 4, 2025
@okdas okdas changed the title [Foundation] Add gas fees [LocalNet] Add gas fees Mar 4, 2025
@okdas
Copy link
Member

okdas commented Mar 4, 2025

Repurposing this ticket to add gas fees on LocalNet. How the fees work is currently understood and implemented on TestNets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on-chain On-chain business logic
Projects
Status: 🔖 Ready
Development

No branches or pull requests

2 participants