#27973 closed defect (fixed)

Update DirectoryAuthoritiesAddresses in default properties and on both instances

Reported by: karsten Owned by: karsten
Priority: Medium Milestone:
Component: Metrics/CollecTor Version:
Severity: Normal Keywords:
Cc: metrics-team Actual Points:
Parent ID: Points:
Reviewer: irl Sponsor:

Description

While looking into possibly missing descriptors I found that we didn't update DirectoryAuthoritiesAddresses in a while. We're still using longclaw's old IP address (changed in October 2017) and missing bastet entirely (also added in October 2017). Let's fix collector.properties and update this configuration on both CollecTor instances. I'll provide a patch in a minute.

Ideally, we'd print out a log message when noticing a mismatch. (Note that moria1 uses a different address in the tor sources than in the document it produces.) This might be another ticket, though.

Child Tickets

Change History (4)

comment:1 Changed 11 months ago by karsten

Status: assignedneeds_review

Please review commit e82d3b5 in my task-27973 repository.

If it turns out that this is the reason for missing descriptors, I might go ahead and change this setting on colchicifolium. Not missing descriptors trumps sticking to the review process, IMO. Though for now I'll leave it unchanged in order to first find out what's going on. Will post here if I change something.

comment:2 Changed 11 months ago by irl

Status: needs_reviewmerge_ready

Looks good to me. Let's include this in the CollecTor 1.7.1 (#28001) release?

comment:3 Changed 11 months ago by irl

Reviewer: irl

comment:4 Changed 11 months ago by karsten

Resolution: fixed
Status: merge_readyclosed

Sounds good. Pushed to master. Closing. Thanks!

Note: See TracTickets for help on using tickets.