Opened 12 months ago

Last modified 8 months ago

#25265 new defect

Set consensus parameter CircuitPriorityHalflife to enable the EWMA circuit policy

Reported by: dgoulet Owned by:
Priority: Medium Milestone:
Component: Core Tor/Chutney Version:
Severity: Normal Keywords:
Cc: teor Actual Points:
Parent ID: #25157 Points:
Reviewer: Sponsor:

Description

So turns out that Chutney is not using EWMA policy at all for circuit priority which means that its been using the default policy which is round robin.

I found this by working on a patch that for testing was assert()ing on the cmux->policy and it exploded.

See cell_ewma_set_scale_factor() on how the EWMA policy is enabled/disabled.

The real network sets: CircuitPriorityHalflifeMsec 30000

Child Tickets

Change History (2)

comment:1 Changed 12 months ago by teor

Parent ID: #25157

Chutney should use the consensus parameters from the public network, see #25157.
For bonus points, we'll work out some way of keeping them in sync.

comment:2 Changed 8 months ago by teor

Cc: teor added; teor@… removed

Shorten useful CCs

Note: See TracTickets for help on using tickets.