Skip to content

Conversation

@fmeum
Copy link
Collaborator

@fmeum fmeum commented May 22, 2023

googletest only reads GTEST_SHARD_STATUS_FILE, not TEST_SHARD_STATUS_FILE, so this variable has to be set to keep sharded cc_tests relying on the test framework working with --incompatible_check_sharding_support.

googletest only reads `GTEST_SHARD_STATUS_FILE`, not
`TEST_SHARD_STATUS_FILE`, so this variable has to be set to keep sharded
`cc_test`s relying on the test framework working with
`--incompatible_check_sharding_support`.
@fmeum fmeum requested a review from meteorcloudy May 22, 2023 16:24
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label May 22, 2023
@Pavank1992 Pavank1992 added the team-Configurability platforms, toolchains, cquery, select(), config transitions label May 23, 2023
@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels May 23, 2023
@fmeum
Copy link
Collaborator Author

fmeum commented May 23, 2023

@bazel-io flag

@bazel-io bazel-io added the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label May 23, 2023
@meteorcloudy
Copy link
Member

@bazel-io fork 6.3.0

@bazel-io bazel-io removed the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label May 23, 2023
@Pavank1992 Pavank1992 removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label May 23, 2023
@fmeum fmeum deleted the fix-test-shard-status branch May 23, 2023 15:38
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request May 23, 2023
googletest only reads `GTEST_SHARD_STATUS_FILE`, not `TEST_SHARD_STATUS_FILE`, so this variable has to be set to keep sharded `cc_test`s relying on the test framework working with `--incompatible_check_sharding_support`.

Closes bazelbuild#18469.

PiperOrigin-RevId: 534375567
Change-Id: I0ca909cc7eb0b0f28f756e90e333e6bf39a0954d
iancha1992 added a commit that referenced this pull request May 24, 2023
googletest only reads `GTEST_SHARD_STATUS_FILE`, not `TEST_SHARD_STATUS_FILE`, so this variable has to be set to keep sharded `cc_test`s relying on the test framework working with `--incompatible_check_sharding_support`.

Closes #18469.

PiperOrigin-RevId: 534375567
Change-Id: I0ca909cc7eb0b0f28f756e90e333e6bf39a0954d

Co-authored-by: Fabian Meumertzheim <[email protected]>
fweikert pushed a commit to fweikert/bazel that referenced this pull request May 25, 2023
googletest only reads `GTEST_SHARD_STATUS_FILE`, not `TEST_SHARD_STATUS_FILE`, so this variable has to be set to keep sharded `cc_test`s relying on the test framework working with `--incompatible_check_sharding_support`.

Closes bazelbuild#18469.

PiperOrigin-RevId: 534375567
Change-Id: I0ca909cc7eb0b0f28f756e90e333e6bf39a0954d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Configurability platforms, toolchains, cquery, select(), config transitions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants