Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ticket29806 035 squashed merged #821

Merged
merged 3 commits into from Mar 27, 2019
Merged

Ticket29806 035 squashed merged #821

merged 3 commits into from Mar 27, 2019

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@teor2345 @coveralls @torproject-pusher @juga0
@teor2345
Copy link
Contributor

@teor2345 teor2345 commented Mar 21, 2019

No description provided.

juga0 and others added 2 commits Mar 21, 2019
@juga0 @teor2345
bwauth: Ignore bandwidth file lines with "vote=0"
4ab2e9a 
so that the relays that would be "excluded" from the bandwidth
file because of something failed can be included to diagnose what
failed, without still including these relays in the bandwidth
authorities vote.

Closes #29806.
@teor2345
Merge branch 'ticket29806_034_squashed' into ticket29806_035_squashed…
3adb689 
…_merged

Copy and paste the vote=0 code from the old src/or/dirserv.c
to the new src/feature/dirauth/bwauth.c.
@teor2345 teor2345 mentioned this pull request Mar 21, 2019
Closed
@teor2345
test/dir: add an extra argument to dirserv_read_measured_bandwidths()
091f868 
Part of 29806.
@coveralls
Copy link

@coveralls coveralls commented Mar 21, 2019

Pull Request Test Coverage Report for Build 4322

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 60.321%
Totals Coverage Status
Change from base Build 4293: 0.002%
Covered Lines: 42870
Relevant Lines: 71070
💛 - Coveralls

@torproject-pusher torproject-pusher merged commit 091f868 into torproject:maint-0.3.5 Mar 27, 2019
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment