Skip to content

Commit

Permalink
Merge #139092
Browse files Browse the repository at this point in the history
139092: build: disable the use of PGO in release, `roachtest`, `--cross` builds r=rail a=rickystewart

Epic: CRDB-41952
Release note (build change): Roll back the use of PGO for releases. We plan to implement this instead for 25.2.

Co-authored-by: Ricky Stewart <[email protected]>
  • Loading branch information
craig[bot] and rickystewart committed Jan 15, 2025
2 parents f1a746c + 65519b9 commit b103f98
Show file tree
Hide file tree
Showing 6 changed files with 127 additions and 127 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,12 +66,12 @@ artifacts=()
case "$component" in
cockroach)
# Cockroach binary.
bazel_args=(--config force_build_cdeps --config pgo --norun_validations //pkg/cmd/cockroach $crdb_extra_flags)
bazel_args=(--config force_build_cdeps --norun_validations //pkg/cmd/cockroach $crdb_extra_flags)
artifacts=("pkg/cmd/cockroach/cockroach_/cockroach:bin/cockroach.$os-$arch")
;;
cockroach-ea)
# Cockroach binary with enabled assertions (EA).
bazel_args=(--config force_build_cdeps --config pgo --norun_validations //pkg/cmd/cockroach --crdb_test $crdb_extra_flags)
bazel_args=(--config force_build_cdeps --norun_validations //pkg/cmd/cockroach --crdb_test $crdb_extra_flags)
artifacts=("pkg/cmd/cockroach/cockroach_/cockroach:bin/cockroach-ea.$os-$arch")
;;
workload)
Expand Down
2 changes: 1 addition & 1 deletion dev
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ fi
set -euo pipefail

# Bump this counter to force rebuilding `dev` on all machines.
DEV_VERSION=106
DEV_VERSION=107

THIS_DIR=$(cd "$(dirname "$0")" && pwd)
BINARY_DIR=$THIS_DIR/bin/dev-versions
Expand Down
2 changes: 1 addition & 1 deletion pkg/cmd/dev/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ func (d *dev) crossBuild(
volume string,
dockerArgs []string,
) error {
bazelArgs = append(bazelArgs, fmt.Sprintf("--config=%s", crossConfig), "--config=nolintonbuild", "-c", "opt", "--config=pgo")
bazelArgs = append(bazelArgs, fmt.Sprintf("--config=%s", crossConfig), "--config=nolintonbuild", "-c", "opt")
configArgs := getConfigArgs(bazelArgs)
dockerArgs, err := d.getDockerRunArgs(ctx, volume, false, dockerArgs)
if err != nil {
Expand Down
Loading

0 comments on commit b103f98

Please sign in to comment.