Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#8530 closed enhancement (implemented)

Add a make target which bootstraps a local testing network and results in PASS or FAIL

Reported by: ln5 Owned by:
Priority: Medium Milestone: Tor: 0.2.5.x-final
Component: Core Tor/Tor Version:
Severity: Keywords: SponsorF20131031 tor-relay testing
Cc: gsathya Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

This is part of SponsorF Y3 #18. One of the goals is to make sure that
we run Tor with TestingTorNetwork more often to find bugs in that.

Child Tickets

Change History (12)

comment:1 Changed 6 years ago by gsathya

Cc: gsathya added

comment:2 Changed 6 years ago by nickm

Keywords: tor-relay testing added

comment:3 Changed 6 years ago by ln5

Status: newneeds_review

Branch bug8530 in my public Tor repo implements a first version of this.

Note that it's based on Tor branch bug8532 and requires Chutney branch
bug8532.

One of the shortcomings is that it expects a copy of Chutney in
./chutney. Do we want a configure option to Tor for pointing out a
Chutney directory?

On a side note, we should probably have an official Chutney repo soon.

comment:4 Changed 6 years ago by nickm

On a side note, we should probably have an official Chutney repo soon.

There is now an official chutney repo. :)

I think that instead of assuming chutney, it can (for now) require that you have set a $CHUTNEY_PATH environment variable. That would probably be cleaner.

comment:5 Changed 6 years ago by ln5

Commit b567efcf in branch bug8530 of my Tor repo fixes this.

I forgot to mention that we also need branch bug8531 in my Chutney
repo in order for this to work.

comment:6 Changed 6 years ago by nickm

This looks fine. (Does it work on non-bash?)

I'll merge it once #8531 is in.

comment:7 Changed 6 years ago by ln5

It does work on non-bash.

And with a recent update to branch bug83530 (Tor) it also avoids
absolute paths (great for other unices).

comment:8 Changed 6 years ago by ln5

#8531 is in.

comment:9 Changed 6 years ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Merged!

comment:10 Changed 6 years ago by arma

In the future, please mention the ticket/bug number in the changes file.

comment:11 Changed 6 years ago by arma

Should we document the existence of this feature in doc/HACKING or somewhere else?

comment:12 Changed 6 years ago by nickm

Should we document the existence of this feature in doc/HACKING or somewhere else?

Done in 5baacf5a9ce9721dfc

Note: See TracTickets for help on using tickets.