m-lab / etl

M-Lab ingestion pipeline
Apache License 2.0
22 stars 7 forks source link

Add siteinfo reloading #996

Closed stephen-soltesz closed 3 years ago

stephen-soltesz commented 3 years ago

This change removes the local etl/site package in favor of using the annotation-service/site package.

This change adds site.MustReload() to the worker main so that the server siteinfo data is available for server annotations.

Part of milestone: https://docs.google.com/document/d/1seI56IGAZzfIhmkZH_Pp67fU11kynyO6mwf7gU3HeiM/edit


This change is Reviewable

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 6482


Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/etl_worker/etl_worker.go 2 3 66.67%
<!-- Total: 2 3 66.67% -->
Totals Coverage Status
Change from base Build 6478: -0.5%
Covered Lines: 3488
Relevant Lines: 5550

💛 - Coveralls
stephen-soltesz commented 3 years ago

New bq rows include server location and network fields from siteinfo.