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

feat(sequencer): allow querying fee components #1748

Merged
merged 3 commits into from
Dec 4, 2024

Conversation

SuperFluffy
Copy link
Member

@SuperFluffy SuperFluffy commented Oct 25, 2024

Summary

Returns a all fee components at the latest height when sending an ABCI info request containing the path "fees/components".

Background

Right now information on the sequencer state is very limited. This patch allows getting a bit more info out of it.

Changes

  • Adds a handler for the path "fees/components" into the ABCI info service running inside sequencer.
  • The handler returns a JSON object containing the values for each component. Each component can return one of the 3 states shown below:
{
    "transaction": {
        base: 1,
        multiplier: 2,
    },
    "rollup_data_submission": "not set",
    "ibc_relay": "<some error message",
}

Testing

Provided an error for sending a high level ABCI info request to the Info service.

@SuperFluffy SuperFluffy requested a review from a team as a code owner October 25, 2024 16:04
@SuperFluffy SuperFluffy requested a review from noot October 25, 2024 16:04
@github-actions github-actions bot added the sequencer pertaining to the astria-sequencer crate label Oct 25, 2024
@SuperFluffy SuperFluffy requested a review from joroshiba October 25, 2024 16:06
@noot noot added this pull request to the merge queue Dec 4, 2024
Merged via the queue into main with commit e1a4f02 Dec 4, 2024
49 checks passed
@noot noot deleted the superfluffy/all-the-queries branch December 4, 2024 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sequencer pertaining to the astria-sequencer crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants