Opened 2 years ago

Closed 2 years ago

Last modified 16 months ago

#22134 closed defect (fixed)

Several bugs encountered running "make test-network" on a fast desktop

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

Description

Please see branch misc_bugs for fixes. Sorry for brevity of this ticket.

These fixes take my "make test-network" success rate from 0% to about 60%.

Child Tickets

Attachments (1)

better-output (45.0 KB) - added by nickm 2 years ago.
debug output from failing make test-network

Download all attachments as: .zip

Change History (7)

comment:1 Changed 2 years ago by nickm

Status: newneeds_review

comment:2 Changed 2 years ago by nickm

Cc: teor added

Changed 2 years ago by nickm

Attachment: better-output added

debug output from failing make test-network

comment:3 Changed 2 years ago by nickm

I've also attached a debug log from a failing run with this branch pplied.

The branch fixes 2 bugs and adds a bunch of debugging calls

comment:4 Changed 2 years ago by teor

I tested this yesterday, and it seems to work ok on Linux: no worse than the existing code.

But I agree there is some issue with chutney and the hidden service tests in test-network-all. I wonder what changed? (I opened #22152 to track this down.)

I would like to test this on macOS when I get a chance, and then merge it.

comment:5 in reply to:  4 Changed 2 years ago by teor

Resolution: fixed
Status: needs_reviewclosed

Replying to teor:

I tested this yesterday, and it seems to work ok on Linux: no worse than the existing code.

But I agree there is some issue with chutney and the hidden service tests in test-network-all. I wonder what changed? (I opened #22152 to track this down.)

I would like to test this on macOS when I get a chance, and then merge it.

All networks work on my macOS when rebased onto 8822169 (#20409 and #21901).
(But single-onion-ipv6 needed a few tries, more detail in #22152.)

Merged as 3a3c1d5.

comment:6 Changed 16 months ago by teor

Cc: teor removed

Remove useless CC

Note: See TracTickets for help on using tickets.