Opened 5 months ago

Last modified 3 months ago

#33581 new defect

Restore bridge networkstatus checks in chutney

Reported by: teor Owned by:
Priority: Medium Milestone:
Component: Core Tor/Chutney Version:
Severity: Normal Keywords: ipv6, prop311
Cc: teor Actual Points:
Parent ID: #33582 Points: 0.1
Reviewer: Sponsor: Sponsor55-can

Description (last modified by teor)

This issue depends on the tor bridge descriptor upload fix in #33582, robust reachability self-tests in #33222, or bridge log checks in #34037.

In chutney networks, there's a race condition when bridges try to publish their descriptor to the bridge authority:

  • bridges try to publish their descriptors before bootstrapping
  • but bridges can't publish their descriptors, because they don't have enough directory info to build a circuit to the bridge authority

Also, bridges do not retry publishing their descriptor immediately after they bootstrap.

We can only do the networkstatus-bridges check on tor versions with the #33222 or #33582 fixes. So we'll need to check for:

  • the next tor 0.4.4-alpha version, or
  • an environmental variable that enables these tests.

We don't have to do these fixes, because it should be enough to test relay reachability. But we would risk breaking bridge reachability tests, and not knowing about it until after a release.

Also, the chutney workarounds seem to cause weird race conditions, which are time-consuming to diagnose and fix.

Child Tickets

TicketStatusOwnerSummaryComponent
#33407closedMake chutney bridge authorities publish bridges in their networkstatus-bridgesCore Tor/Tor

Change History (8)

comment:1 Changed 5 months ago by teor

Description: modified (diff)

comment:2 Changed 5 months ago by teor

Owner: teor deleted
Sponsor: Sponsor55-mustSponsor55-can

comment:3 Changed 5 months ago by teor

Status: assignednew

comment:4 Changed 5 months ago by teor

Parent ID: #33232#33582

comment:5 Changed 3 months ago by teor

Description: modified (diff)

comment:6 Changed 3 months ago by teor

Description: modified (diff)

comment:7 Changed 3 months ago by teor

Description: modified (diff)

AssumeReachable isn't required to trigger this issue.

comment:8 Changed 3 months ago by teor

Description: modified (diff)
Note: See TracTickets for help on using tickets.