Skip to content

feat: calibrate fetch_canister_logs fee and add benchmarks#9911

Merged
maksymar merged 3 commits intomasterfrom
maksym/fetch-logs-fee
Apr 17, 2026
Merged

feat: calibrate fetch_canister_logs fee and add benchmarks#9911
maksymar merged 3 commits intomasterfrom
maksym/fetch-logs-fee

Conversation

@maksymar
Copy link
Copy Markdown
Contributor

  • Update fetch_canister_logs fee from placeholder values to calibrated.
  • Add fetch_canister_logs benchmarks using inter-canister update calls with a baseline (canister_status) for isolating fetch cost.
  • Add tests verifying fee rejection and cycle deduction.

Update fetch_canister_logs fee from placeholder values to calibrated:
- base_fee: 1_000_000 -> 5_000_000
- per_byte_fee: 800 -> 80

Add fetch_canister_logs benchmarks using inter-canister update calls
with a baseline (canister_status) for isolating fetch cost.

Add tests verifying fee rejection and cycle deduction.
@github-actions github-actions bot added the feat label Apr 16, 2026
@maksymar maksymar marked this pull request as ready for review April 16, 2026 16:54
@maksymar maksymar requested a review from a team as a code owner April 16, 2026 16:54
Comment thread rs/execution_environment/tests/subnet_size_test.rs
@maksymar maksymar enabled auto-merge April 17, 2026 08:29
@maksymar maksymar added this pull request to the merge queue Apr 17, 2026
Merged via the queue into master with commit 987cf44 Apr 17, 2026
37 checks passed
@maksymar maksymar deleted the maksym/fetch-logs-fee branch April 17, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants