Opened 2 years ago

Closed 2 years ago

#27169 closed task (implemented)

monitor bootstrap directory info progress separately

Reported by: catalyst Owned by:
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: usability, ux, ux-team, bootstrap, 035-roadmap-subtask, 035-triaged-in-20180711, s8-bootstrap
Cc: brade, mcs, linda, isabela, iry Actual Points:
Parent ID: #27103 Points:
Reviewer: nickm Sponsor: Sponsor8


Abstract out the current monitoring of bootstrap directory information progress, so we can track it state more independently. This allows us to defer reporting that we have sufficient directory information until we know that we can actually connect to a relay or bridge at all.

This also allows us to eliminate or simplify special case logic in control_event_bootstrap() that handles incremental progress during descriptor downloads.

Child Tickets

Change History (5)

comment:1 Changed 2 years ago by catalyst

Status: newneeds_review

comment:2 Changed 2 years ago by nickm

Reviewer: nickm
Sponsor: Sponsor8

comment:3 Changed 2 years ago by dgoulet

Status: needs_reviewneeds_information

Review on the PR. Nothing crazy, for now questions for things I'm unclear about.

comment:4 Changed 2 years ago by dgoulet

Status: needs_informationmerge_ready

Ok I'm good with the changes.

comment:5 Changed 2 years ago by nickm

Resolution: implemented
Status: merge_readyclosed

nice. lgtm; merged to master.

Note: See TracTickets for help on using tickets.