Never include non-Valid nodes in consensus.

Right now, we don't include non-Running nodes. Let's make it so we never include non-Valid nodes either.

changed milestone to %Tor: 0.2.9.x-final

added TorCoreTeam201608 actualpoints::.1 component::core tor/tor milestone::Tor: 0.2.9.x-final owner::nickm parent::20001 points::.1 priority::medium prop272 resolution::implemented review-group-8 severity::normal status::closed type::defect labels

This is part of proposal 272.

Trac:
Owner: N/A to nickm
Status: new to accepted

See branch "ticket20002" in my public repository.

Trac:
Actualpoints: N/A to .1
Status: accepted to needs_review
Keywords: needs-proposal deleted, prop272 added

Trac:
Keywords: prop272 deleted, prop272 TorCoreTeam201608 added

Trac:
Keywords: prop272 TorCoreTeam201608 deleted, prop272, TorCoreTeam201608, review-group-8 added

This looks just fine to me.

Trac:
Status: needs_review to merge_ready

Merged! Thanks for the review.

(Actually hang on. Not merged yet. Running some tests first)

Okay. Now merged for real. Pfew!

Trac:
Status: merge_ready to closed
Resolution: N/A to implemented

closed

changed time estimate to 48m

added 48m of time spent

moved to tpo/core/tor#20002 (closed)