Opened 2 years ago

Last modified 19 months ago

#20519 assigned enhancement

Untangle and modernize relaydescs module

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

Description

  • unify descriptor parsing throughout all classes
  • separate statistics
  • untangle the following picture
											       
     +--------------------------------------------------------------------+ 
     |       	     	      	       	    	     	     	       	  | 
     | 	X o----------+ Y : X holds a Y reference somewhere                | 
     |               	      	       	    	     	     	       	  | 
     +--------------------------------------------------------------------+ 
     |		  		       	    	     	     	       	  | 
     |                                 	    	     	     	       	  | 
     |               +------------------------+	     	     	       	  | 
     |               |  RelayDescriptorParser +\     	     	       	  | 
     |		     +-----+-----o-+o---o-----+	\    	     	       	  | 
     |		  	  /     /  | \ 	 --	 -\  	     	       	  | 
     |                   /     /   |  \    \-      -\	     	       	  | 
     |                  /     /     \ |      \-      -\	     	       	  | 
     |                 /     /      |  \       \-      -\    	       	  | 
     |		      /     /	    |  |         \-	 -\  	       	  | 
     |               /     /        |   \          \-      -\	       	  | 
     |		    /	  / 	     \  |	     \-	     -\	       	  | 
     |		   / 	 /  	     |   \	       \-      \       	  | 
     |        +---o-----+-------+    |   |      +-------+-------o-+    	  | 
     |	      |  ArchiveReader 	|     \	  \	|  ArchiveWriter  |    	  | 
     |	      +-----------------+     |	  |	+---o-------------+    	  | 
     |			 	      |	   \	    |		       	  | 
     |			 	      |    |	    |		       	  | 
     |                                 \    \      /                   	  | 
     |				  +----o----+------+----------+	       	  | 
     |				  | RelayDescriptorDownloader |	       	  | 
     |                            +---------------------------+        	  | 
     |                           /                                     	  | 
     |				/					  | 
     |			       /					  | 
     |	       +--------------o--------------+				  | 
     |         | CachedRelayDescriptorParser |                            | 
     |	       +-----------------------------+				  | 
     |                                                                    | 
     |                                                                    |
     +--------------------------------------------------------------------+

Child Tickets

TicketTypeStatusOwnerSummary
#20228enhancementassignedmetrics-teamAppend all votes with same valid-after time to a single file in `recent/`
#20335defectclosedReferenceChecker causes OOM
#20345enhancementassignedmetrics-teamAdd support for synchronizing microdescriptors from another instance

Change History (4)

comment:1 Changed 19 months ago by karsten

Summary: untangle releaydesc and modernizeUntangle and modernize relaydesc module

Tweak summary a bit.

comment:2 Changed 19 months ago by karsten

Summary: Untangle and modernize relaydesc moduleUntangle and modernize relaydescs module

The module is called relaydescs, not relaydesc. OK.

comment:3 Changed 19 months ago by karsten

Keywords: metrics-2018 added

comment:4 Changed 19 months ago by karsten

Owner: set to metrics-team
Status: newassigned
Note: See TracTickets for help on using tickets.