Skip to content

Commit

Permalink
Do not expect info field in received report
Browse files Browse the repository at this point in the history
  • Loading branch information
epapbak committed Aug 2, 2024
1 parent 869a74b commit 22393f7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion consumer.go
Original file line number Diff line number Diff line change
Expand Up @@ -538,13 +538,14 @@ func checkReportStructure(r Report) error {
// the structure is not well defined yet, so all we should do is to check if all keys are there
expectedKeys := []string{
fingerprintsAttribute,
infoAttribute,
reportsAttribute,
systemAttribute,
}

// 'skips' key is now optional, we should not expect it anymore:
// https://github.com/RedHatInsights/insights-results-aggregator/issues/1206
// Simialrly, 'info' key is now optional too.
// https://github.com/RedHatInsights/insights-results-aggregator/pull/1996
// expectedKeys := []string{"fingerprints", "info", "reports", "skips", "system"}

// check if the structure contains all expected keys
Expand Down

0 comments on commit 22393f7

Please sign in to comment.