This repository has been archived by the owner. It is now read-only.
torproject / sbws Public archive
Conversation
The time it took to run prioritize function is not relevant for the operator. After several days, prioritiy would be adjusted for many relays, so don't log it. We can have different debug verbose levels in the future.
that would be shown at info level.
If rtt is configured to 0, the rtt measurements are not run, but it was being logged that they're run
obtained when trying to build the circuit, or None if it was successful. That way it can be logged and stored in the results.
Warnings can make think the operator that sbws is not working, while it's normal that some measurements fail. Resultdump already logs at info level what succcess and what fails. Keep these logs at debug level.
When a new loop starts and when it ends, with the number of relays measured and the time it took.
since it takes some seconds and the operator would not see any progress at the beginning, until relays start to be measured.
since it takes some seconds.
bandwidth or rtt, so that it can be used to be stored and logged.
The error that is returned by an exception, not a custom error. So that it's easier to debug what fails.
Closes bug #28736. Bugfix v0.3.0.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
No description provided.
The text was updated successfully, but these errors were encountered: