Skip to content

Commit 5c8ce23

Browse files
Log ArchiveURL in scamper1 errors (#1072)
* Log ArchiveURL in scamper1 errors * Use consistent naming
1 parent c7d2fdf commit 5c8ce23

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

parser/scamper1.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,9 +107,10 @@ func (p *Scamper1Parser) ParseAndInsert(fileMetadata map[string]bigquery.Value,
107107
defer metrics.WorkerState.WithLabelValues(p.TableName(), scamper1).Dec()
108108

109109
scamperOutput, err := parser.ParseTraceroute(rawContent)
110+
archiveURL := fileMetadata["filename"].(string)
110111
if err != nil {
111112
metrics.TestTotal.WithLabelValues(p.TableName(), scamper1, err.Error()).Inc()
112-
return fmt.Errorf("failed to parse scamper1 file: %s, error: %w", testName, err)
113+
return fmt.Errorf("failed to parse scamper1 file: %s, archiveURL: %s, error: %w", testName, archiveURL, err)
113114
}
114115

115116
bqScamperOutput := schema.BQScamperOutput{
@@ -122,7 +123,7 @@ func (p *Scamper1Parser) ParseAndInsert(fileMetadata map[string]bigquery.Value,
122123
parseInfo := schema.ParseInfo{
123124
Version: Version(),
124125
Time: time.Now(),
125-
ArchiveURL: fileMetadata["filename"].(string),
126+
ArchiveURL: archiveURL,
126127
Filename: testName,
127128
GitCommit: GitCommit(),
128129
}

0 commit comments

Comments
 (0)