Skip to content

roachtest: fix conditional build for arm64 and fips#157759

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
golgeek:ludo/roachtest-fix-build
Nov 13, 2025
Merged

roachtest: fix conditional build for arm64 and fips#157759
craig[bot] merged 1 commit intocockroachdb:masterfrom
golgeek:ludo/roachtest-fix-build

Conversation

@golgeek
Copy link
Copy Markdown
Contributor

@golgeek golgeek commented Nov 13, 2025

PR #157158 introduced a conditional build for ARM64 and FIPS architectures based on the metamorphic probability flags.

The initial implementation used bc to compare floating point, but the command is not available on the TeamCity agents:

build/teamcity/cockroach/nightlies/roachtest_nightly_impl.sh: line 39: bc: command not found

This patch uses awk in place of bc.

Epic: none
Release note: None

PR cockroachdb#157158 introduced a conditional build for ARM64 and FIPS
architectures based on the metamorphic probability flags. The initial
implementation used `bc` to compare floating point, but the command is
not available on the TeamCity agents.

This patch uses `awk` in place of `bc`.

Epic: none
Release note: None
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@golgeek
Copy link
Copy Markdown
Contributor Author

golgeek commented Nov 13, 2025

Test TC run with ARM64=0; FIPS=0.02

@golgeek golgeek marked this pull request as ready for review November 13, 2025 16:28
@golgeek golgeek requested a review from a team as a code owner November 13, 2025 16:28
@golgeek
Copy link
Copy Markdown
Contributor Author

golgeek commented Nov 13, 2025

TFTR!

bors r=herkolategan

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Nov 13, 2025

@craig craig bot merged commit 5805dd4 into cockroachdb:master Nov 13, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants