Opened 11 days ago
Closed 30 hours ago
#26702 closed enhancement (implemented)
Remind authority operators that bandwidth files should be written atomcally
| Reported by: | teor | Owned by: | teor |
|---|---|---|---|
| Priority: | Medium | Milestone: | Tor: 0.3.5.x-final |
| Component: | Core Tor/Tor | Version: | |
| Severity: | Normal | Keywords: | torspec, 035-removed-20180711, fast-fix, doc |
| Cc: | Actual Points: | ||
| Parent ID: | #26851 | Points: | |
| Reviewer: | Sponsor: |
Description (last modified by )
Tor reads bandwidth files when it's voting, around 50 minutes past every hour, and 20 minutes past hours when the consensus has failed.
We should recommend that authority operators generate and transfer bandwidth files between 5-15 or 35-45 minutes outside of 15-25 and 45-55 minutes past the hour. The best place for this might be in dir-spec or the bandwidth file spec.
Child Tickets
Change History (18)
comment:1 Changed 11 days ago by
| Description: | modified (diff) |
|---|
comment:2 Changed 9 days ago by
I thought to create a crontab file in the Debian package, But because would be needed for other distributions and it's essential to run sbws, i'm thinking now to add it to sbws repo. Created #26740 for it.
comment:3 Changed 9 days ago by
| Keywords: | 035-removed-20180711 added |
|---|---|
| Milestone: | Tor: 0.3.5.x-final → Tor: unspecified |
These tickets are being triaged out of 0.3.5. The ones marked "035-roadmap-proposed" may return.
comment:4 Changed 4 days ago by
| Keywords: | fast-fix doc added |
|---|---|
| Milestone: | Tor: unspecified → Tor: 0.3.5.x-final |
| Owner: | set to teor |
| Status: | new → assigned |
comment:5 Changed 4 days ago by
| Description: | modified (diff) |
|---|---|
| Status: | assigned → needs_review |
The appropriate place for this documentation is in the man page, under the V3BandwidthsFile option. It's the place that authority operators are most likely to look when configuring a bandwidth authority.
Please see my branch ticket26702 on https://github.com/teor2345/tor.git
(We could also document this requirement in sbws, but sbws does atomic symlinks, so it's only important when using scp from a different host.)
comment:6 Changed 4 days ago by
(We could also document this requirement in sbws, but sbws does atomic symlinks, so it's only important when using scp from a different host.)
I left a comment in #26740.
comment:7 Changed 4 days ago by
The distcheck travis CI on this doc-only branch failed, probably because of #26787.
comment:9 follow-up: 11 Changed 3 days ago by
I'm sorry; I can't find this branch. Are you sure it's where you say?
comment:11 Changed 2 days ago by
Replying to nickm:
I'm sorry; I can't find this branch. Are you sure it's where you say?
I forked torproject/tor yesterday so I could do pull requests. (GitHub doesn't support pull requests between repositories, unless they are in the same GitHub fork tree.) My old tor repository is at tor-old.
I have re-pushed the branch ticket26702 to https://github.com/teor2345/tor.git
comment:12 Changed 2 days ago by
I don't think this is what we should recommend. I think that instead we should recommend that these files be replaced atomically, via rename. How would that be?
comment:13 Changed 2 days ago by
| Parent ID: | #25925 → #26851 |
|---|---|
| Status: | merge_ready → needs_revision |
Ok, I think that's better, I'll update all the docs at once.
comment:14 Changed 2 days ago by
| Summary: | Remind authority operators that bandwidth files should be transferred outside the voting period → Remind authority operators that bandwidth files should be written atomcally |
|---|
comment:15 Changed 2 days ago by
| Status: | needs_revision → merge_ready |
|---|
Please see my branch ticket26702-atomically on https://github.com/teor2345/tor.git
comment:17 Changed 2 days ago by
The appveyor CI failed for this doc-only branch, I opened #26853 to fix it.
It doesn't matter if the generator is remote or local, it still needs to be scheduled correctly.