Opened 4 months ago

Closed 8 weeks ago

#28928 closed task (implemented)

update control-spec.txt for new bootstrap phases

Reported by: catalyst Owned by: catalyst
Priority: High Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: sponsor19-also, s8-bootstrap, tor-spec, postfreeze-ok, 040-must, spec, nickm-merge
Cc: brade, mcs, arma Actual Points:
Parent ID: #28018 Points:
Reviewer: teor Sponsor: Sponsor19

Description

Update control-spec.txt to reflect the new phases implemented by #27167.

Child Tickets

Change History (13)

comment:2 Changed 4 months ago by catalyst

Owner: set to catalyst
Status: newassigned
Type: defecttask

comment:3 Changed 4 months ago by nickm

I've merged the above edits for now, on the theory that documentation is better than none. Thanks!

comment:4 Changed 3 months ago by catalyst

Sponsor: Sponsor8Sponsor19

comment:5 Changed 3 months ago by catalyst

Status: assignedneeds_review

More WIP edits in https://github.com/torproject/torspec/pull/50 with additional details, along with grouping phases into three stages. Also added an overview in Section 5.5.1, but that might needs some more refinement?

comment:6 Changed 3 months ago by nickm

This all looks like good progress to me. I'm happy to take it whenever you say it's ready to go. It would be good to have feedback from the downstream users of this information, this patch seems to be all addition of information, so IMO it can't do any harm. From my POV it explains things well.

comment:7 Changed 3 months ago by asn

Reviewer: teor

comment:8 Changed 3 months ago by nickm

Keywords: postfreeze-ok added

Mark some tickets as postfreeze-ok, to indicate that I think they are okay to accept in 0.4.0 post-freeze. Does not indicate that they are all necessary to do postfreeze.

comment:9 Changed 2 months ago by nickm

Keywords: 040-must added

Marking tickets as 040-must based on triage with dgoulet.

comment:10 Changed 2 months ago by nickm

Keywords: spec added
Priority: MediumHigh

comment:11 Changed 8 weeks ago by teor

Status: needs_reviewmerge_ready

This seems fine to me.

Thanks!

comment:12 Changed 8 weeks ago by nickm

Keywords: nickm-merge added

comment:13 Changed 8 weeks ago by nickm

Resolution: implemented
Status: merge_readyclosed

Merged!

Note: See TracTickets for help on using tickets.