Skip to content

Commit

Permalink
chore: don't compress brokerpaks in integration tests
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.
  • Loading branch information
blgm committed Oct 9, 2024
1 parent 1144fb3 commit 2341dac
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 2341dac

Please sign in to comment.