Opened 2 months ago

Closed 6 weeks ago

#30459 closed defect (fixed)

Let chutney tell Tor whether a network is supported

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.2.x-final
Component: Core Tor/Chutney Version:
Severity: Normal Keywords: tor-pt, 041-accepted-20190115, network-team-roadmap-2019-Q1Q2, tor-ci, 041-deferred-20190530
Cc: teor, cohosh Actual Points: 1
Parent ID: #29024 Points:
Reviewer: Sponsor: Sponsor19

Description

We should do this anyway to simplify Tor's test-networks-all logic, but it will be helpful in order to have PT tests that can check whether the right PTs are installed.

Child Tickets

Change History (9)

comment:1 Changed 2 months ago by nickm

Status: assignedneeds_review

See my branch net-requirements with PR at https://github.com/torproject/chutney/pull/27 .

It gives chutney the ability to detect missing ipv6, missing tor-stable, missing tor-gencert. It also refactors some code a bit. If #30452 is merged, it also detects directory authorities built with a missing dirauth module.

comment:2 Changed 2 months ago by nickm

Keywords: tor-ci added

comment:3 Changed 7 weeks ago by nickm

Keywords: 041-deferred-20190530 added

Marking these tickets as deferred from 041.

comment:4 Changed 7 weeks ago by nickm

Milestone: Tor: 0.4.1.x-finalTor: 0.4.2.x-final

comment:5 Changed 6 weeks ago by teor

Status: needs_reviewneeds_revision

Looks good to me, but I have a few questions about the details.

comment:6 Changed 6 weeks ago by nickm

Status: needs_revisionneeds_review

I've made a change and responded to your questions. Please let me know if you don't still don't agree about anything that I didn't change, and I can try to make a change.

comment:7 Changed 6 weeks ago by teor

Status: needs_reviewmerge_ready

Looks good to me, I will squash and merge.

comment:8 Changed 6 weeks ago by teor

The squashed PR is:
https://github.com/torproject/chutney/pull/32

Waiting for CI before merge.

comment:9 Changed 6 weeks ago by teor

Resolution: fixed
Status: merge_readyclosed

Merged to master as ebaac29.

Note: See TracTickets for help on using tickets.