Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set GTEST_SHARD_STATUS_FILE in test setup #18469

Closed
wants to merge 1 commit into from

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 <fabian@meumertzhe.im>
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