[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #18910 [Metrics/CollecTor]: distributing descriptors accross CollecTor instances
#18910: distributing descriptors accross CollecTor instances
-------------------------------+---------------------------------
Reporter: iwakeh | Owner: iwakeh
Type: enhancement | Status: needs_review
Priority: High | Milestone: CollecTor 1.1.0
Component: Metrics/CollecTor | Version:
Severity: Normal | Resolution:
Keywords: ctip | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------------+---------------------------------
Comment (by karsten):
So, I reviewed about half of your -3 branch now but still more time
tomorrow to finish the review. Here's a quick suggestion though:
Should we put out a new metrics-lib release tomorrow that this branch will
be "based on"? I could imagine having the following changes in that
metrics-lib release:
- `DescriptorIndexCollector` is the new default.
- That same class uses `/index/index.json[.gz]` as path if it only
receives a base URL. I believe I suggested such a thing on #20039.
- metrics-lib master already contains a fix that would be useful to have
for new CollecTor operators: "Avoid running into an IOException and
logging a warning for it." (#20320)
Note that I'm not suggesting to fix the 2G file size thing in that
release. Let's stay realistic. ;)
If we do this, are there any other metrics-lib changes that we should
include in the release that would help us with this CollecTor change?
More feedback on the -3 branch tomorrow. Please don't amend the commits
in that branch until then.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/18910#comment:53>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs