coveralls
commented
4 years ago Pull Request Test Coverage Report for Build 69
- 14 of 14 (100.0%) changed or added relevant lines in 2 files are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage remained the same at 100.0%
| Totals | |
|---|---|
| Change from base Build 64: | 0.0% |
| Covered Lines: | 222 |
| Relevant Lines: | 222 |
stephen-soltesz
Previously, the uuid-annotator was failing to lookup 100% IPs. This turned out to be caused by a failure to determine direction of flows in: https://github.com/m-lab/uuid-annotator/blob/1fc8c0aa523ff58f11036503f98a9b456f40f8c1/ipannotator/ipannotator.go#L49-L68
Because it was comparing flow IPs ("4.14.5.71") to local interfaces with "" because the collection of the localIPs in main.go failed to parse the netblock format for local interface addrs.
This change casts the local interface addrs to *net.IPNet` and extracts the net.IP from within. This change also includes additional logging. And, the annotations result struct is annotated for bigquery with standard column names.
This change is