Skip to content

feat: Set subnet size and cost schedule of HTTP outcalls context#10761

Open
eichhorl wants to merge 7 commits into
masterfrom
eichhorl/set-subnet-size-and-schedule
Open

feat: Set subnet size and cost schedule of HTTP outcalls context#10761
eichhorl wants to merge 7 commits into
masterfrom
eichhorl/set-subnet-size-and-schedule

Conversation

@eichhorl

@eichhorl eichhorl commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

These fields were introduced in #10712. They are required to correctly calculate part of the new pricing inside of the execution environment, where no registry client is available.

With this PR we start setting the fields to their correct values. In a subsequent release, we can then make the cost schedule mandatory.

@github-actions github-actions Bot added the feat label Jul 13, 2026
@eichhorl
eichhorl force-pushed the eichhorl/set-subnet-size-and-schedule branch from 4f33e5c to 901d317 Compare July 13, 2026 16:54
@eichhorl
eichhorl marked this pull request as ready for review July 20, 2026 11:36
@eichhorl
eichhorl requested a review from a team as a code owner July 20, 2026 11:36
@zeropath-ai

zeropath-ai Bot commented Jul 20, 2026

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to b8928ba.

Security Overview
Detected Code Changes
Change Type Relevant files
Enhancement ► rs/execution_environment/src/execution_environment.rs
    Add cost_schedule handling when generating CanisterHttpRequestContext from flexible args and from args
Enhancement ► rs/types/types/src/canister_http.rs
    Add cost_schedule parameter to CanisterHttpRequestContext constructors
    Populate subnet_size and cost_schedule fields accordingly
    Update tests to pass cost schedule in context generation

@eichhorl
eichhorl force-pushed the eichhorl/set-subnet-size-and-schedule branch from 594ffcc to b8928ba Compare July 21, 2026 08:58
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.

1 participant