Opened 4 months ago

Last modified 3 days ago

#26694 reopened enhancement

dir-spec: DirAuths should expose bwauth bandwidth files

Reported by: juga Owned by: juga
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-dirauth, metrics, tor-bwauth, 035-triaged-in-20180711
Cc: Actual Points:
Parent ID: #21377 Points:
Reviewer: juga, teor Sponsor:

Description

This ticket is for changing dir-spec to implement #21377

Child Tickets

Change History (11)

comment:1 Changed 4 months ago by teor

Status: assignedneeds_revision

The pull request https://github.com/torproject/torspec/pull/21 is in needs_revision.

Let's deal with it in this ticket.

comment:2 Changed 4 months ago by juga

Status: needs_revisionneeds_review

comment:3 Changed 4 months ago by nickm

Keywords: 035-triaged-in-20180711 added

comment:4 Changed 4 months ago by teor

Cc: teor removed
Reviewer: juga, teor

Hi juga,

I made some edits to the spec.
Please see my branch ticket21377 at https://github.com/teor2345/torspec.git
My pull request is available at ​https://github.com/torproject/torspec/pull/26

Or you can push my commits to your ​https://github.com/torproject/torspec/pull/21
Please see my comments on ​https://github.com/torproject/torspec/pull/21

If you're happy with it, let's merge my branch ticket21377-squashed.

comment:5 Changed 4 months ago by juga

PR 26 lgtm, yes, let's merge squashed branch.

comment:6 Changed 4 months ago by teor

Status: needs_reviewmerge_ready

Please assign a proposal number to proposals/xxx-expose-bandwidth-files.txt, then merge my torspec branch ticket21377-squashed from https://github.com/teor2345/torspec.git

I emailed the proposal to tor-dev:
https://lists.torproject.org/pipermail/tor-dev/2018-July/013322.html

comment:7 Changed 4 months ago by nickm

It's now prop296.

comment:8 Changed 4 months ago by teor

Resolution: fixed
Status: merge_readyclosed

The spec has been merged to torspec and dir-spec, we'll continue the implementation in #21377.

comment:9 Changed 8 days ago by teor

Resolution: fixed
Status: closedreopened

This spec needs an update, because the feature did not make it into 0.3.5:
https://github.com/torproject/torspec/blob/master/dir-spec.txt#L2640

comment:10 Changed 4 days ago by juga

It first appeared in tor-0.3.5.1-alpha, should it be changed to that?, or should we wait until it make it into a release and put that instead?

comment:11 in reply to:  10 Changed 3 days ago by teor

Replying to juga:

It first appeared in tor-0.3.5.1-alpha, should it be changed to that?

Which feature first appeared in tor-0.3.5.1-alpha?

Tor 0.3.5.1-alpha has been released. But the code for "DirAuths should expose bwauth bandwidth files" is in #21377. And it has not been merged yet.

or should we wait until it make it into a release and put that instead?

We should wait until #21377 is merged, then update the spec.

Note: See TracTickets for help on using tickets.