Skip to content

Commit

Permalink
fix: add FeeTier support to galileo tx sending
Browse files Browse the repository at this point in the history
During Testnet 77 we enabled fees [0] and in the process broke some
things like Galileo. Adding FeeTier support is rather straightforward,
although this changeset requires a patch to the `penumbra-fee` crate in
order to compile [1].

[0] penumbra-zone/penumbra#4306
[1] penumbra-zone/penumbra#4539
  • Loading branch information
conorsch committed Jun 4, 2024
1 parent 6c71686 commit a55b46f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
1 change: 1 addition & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ parallel = ["penumbra-wallet/parallel"]
# Penumbra dependencies
penumbra-proto = { path = "../penumbra/crates/proto", features = ["rpc", "box-grpc"] }
penumbra-asset = { path = "../penumbra/crates/core/asset" }
penumbra-fee = { path = "../penumbra/crates/core/component/fee" }
penumbra-keys = { path = "../penumbra/crates/core/keys" }
penumbra-custody = { path = "../penumbra/crates/custody" }
penumbra-wallet = { path = "../penumbra/crates/wallet" }
Expand Down
7 changes: 4 additions & 3 deletions src/opt/serve.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ use futures_util::{stream::StreamExt, stream::TryStreamExt};
use num_traits::identities::Zero;
use penumbra_asset::Value;
use penumbra_custody::soft_kms::SoftKms;
use penumbra_fee::FeeTier;
use penumbra_proto::{
custody::v1::{
custody_service_client::CustodyServiceClient, custody_service_server::CustodyServiceServer,
Expand All @@ -27,9 +28,9 @@ use crate::{

#[derive(Debug, Clone, Parser)]
pub struct Serve {
/// The transaction fee for each response (paid in upenumbra).
#[structopt(long, default_value = "0")]
fee: u64,
/// The level of gas prices to pay on send transactions. Defaults to "low".
#[structopt(long, default_value_t)]
fee_tier: FeeTier,
/// Per-user rate limit (e.g. "10m" or "1day").
#[clap(short, long, default_value = "1day", parse(try_from_str = humantime::parse_duration))]
rate_limit: Duration,
Expand Down

0 comments on commit a55b46f

Please sign in to comment.