Skip to content

Commit

Permalink
Fix a bug in the known images test from the locator changes
Browse files Browse the repository at this point in the history
This was previously looking for namespaces starting with e2e-, my change
flipped it to a contains, which then picked up openshift-e2e-loki, which
does use a external image that we do not want this test failing on.

Restored to a prefix check.
  • Loading branch information
dgoodwin committed Apr 17, 2024
1 parent f075c64 commit 4037ee7
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ func (w *clusterImageValidator) EvaluateTestsFromConstructedIntervals(ctx contex
continue
}
// only look at pull events from an e2e-* namespace
if !strings.Contains(event.StructuredLocator.Keys[monitorapi.LocatorNamespaceKey], "e2e-") {
if !strings.HasPrefix(event.StructuredLocator.Keys[monitorapi.LocatorNamespaceKey], "e2e-") {
continue
}

Expand Down

0 comments on commit 4037ee7

Please sign in to comment.