Skip to content

Commit

Permalink
chore: don't compress brokerpaks in integration tests (#1119)
Browse files Browse the repository at this point in the history
Compression is computationally expensive, and there's no value in doing it during the integration tests
as the brokerpak will be immediately expanded after being compressed. So we now specify the --compress=false
flag which will mean that the integration tests consume less compute, and might run faster.

Co-authored-by: Felisia Martini <[email protected]>
  • Loading branch information
blgm and FelisiaM authored Oct 11, 2024
1 parent 733bb32 commit 212eb36
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion integrationtest/packer/brokerpak.go
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ func BuildBrokerpak(csbPath, sourcePath string, opts ...BuildBrokerpakOption) (s
ctx, cancel := context.WithTimeout(context.Background(), 10*time.Minute)
defer cancel()

cmd := exec.CommandContext(ctx, csbPath, "pak", "build", "--target", "current", sourcePath)
cmd := exec.CommandContext(ctx, csbPath, "pak", "build", "--target=current", "--compress=false", sourcePath)
cmd.Dir = cfg.dir

output, err := cmd.CombinedOutput()
Expand Down

0 comments on commit 212eb36

Please sign in to comment.