Revert "Add cpp stack traces to our own reruns (#119408)"

This reverts commit 47182a8f4b5e36e280ca3595ba134f53499d2dc9.

Reverted https://github.com/pytorch/pytorch/pull/119408 on behalf of https://github.com/clee2000 due to iirc the default setting of env to None causes it to inherit the env of the calling process, I'll make a PR that makes it so that the old env vars don't disappear, and then re merge this on top of it.  Reverting this because I think some important env vars are disappearing (specifically CI) ([comment](https://github.com/pytorch/pytorch/pull/119408#issuecomment-1955128676))
This commit is contained in:
PyTorch MergeBot
2024-02-20 21:28:13 +00:00
parent 2d6c0cc81b
commit dfb83df889

View File

@ -26,7 +26,6 @@ from torch.testing._internal.common_utils import (
FILE_SCHEMA,
get_report_path,
IS_CI,
IS_MACOS,
parser as common_parser,
retry_shell,
set_cwd,
@ -617,16 +616,6 @@ def run_test_retries(
else:
sc_command = f"--sc={stepcurrent_key}"
print_to_file("Retrying...")
# Print full c++ stack traces during retries
# Don't do it for macos inductor tests as it makes them
# segfault for some reason
if not (
IS_MACOS
and len(command) >= 2
and command[2].startswith(INDUCTOR_TEST_PREFIX)
):
env = env or {}
env["TORCH_SHOW_CPP_STACKTRACES"] = "1"
print_items = [] # do not continue printing them, massive waste of space
consistent_failures = [x[1:-1] for x in num_failures.keys() if num_failures[x] >= 3]