Skip to content

Commit

Permalink
harness: don't require cosa build for --oscontainer
Browse files Browse the repository at this point in the history
The `--oscontainer` switch takes either a filename to an OCI archive or
a pullspec. In neither case is a cosa directory required.

Fixes b8f70e8 ("kola: handle `--oscontainer` imperatively") which
mistakenly cargo culted that bit from surrounding code.
  • Loading branch information
jlebon committed Mar 15, 2024
1 parent fdbab67 commit 836f8ac
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions mantle/kola/harness.go
Original file line number Diff line number Diff line change
Expand Up @@ -1840,9 +1840,6 @@ func runTest(h *harness.H, t *register.Test, pltfrm string, flight platform.Flig
}

if Options.OSContainer != "" {
if CosaBuild == nil {
h.Fatalf("Requested oscontainer pivot, but no cosa build found")
}
rebase_arg := Options.OSContainer
// if it looks like a path to an OCI archive, then copy it into the system
if strings.HasSuffix(Options.OSContainer, ".ociarchive") {
Expand Down

0 comments on commit 836f8ac

Please sign in to comment.