m-lab / etl

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

Strip "unsafe" from ndt5 and ndt7 UUIDs #1107

Closed stephen-soltesz closed 1 year ago

stephen-soltesz commented 1 year ago

This change is an update in response to the "unsafe UUIDs" incident 2022-11-30. Between 2022-11-08 and 2022-11-30 the NDT server was started without the -uuid-prefix-file flag which resulted in UUIDs with the "unsafe" string while other sidecar data (e.g. annotation) used the correct UUID. This mismatch results in the two datatypes failing to join.

This change is sufficient to correct the NDT UUIDs and allow the two datatypes to join correctly.


This change is Reviewable

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 7428


Totals Coverage Status
Change from base Build 7421: 0.03%
Covered Lines: 3329
Relevant Lines: 4946

💛 - Coveralls