-
Notifications
You must be signed in to change notification settings - Fork 69
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
Apply isFailureFromPrevMergeCommit logic to all similar failures #5323
Conversation
@huydhn is attempting to deploy a commit to the Meta Open Source Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Spot check some more reverted PRs ⏳ 20 Pending, 2 Unrelated FailuresAs of commit 0b36b0301cc77abf86ab9ad0764f945bc1046f95 with merge base f681e3689b857b8811f19d60d439bfb3fb2dd2d3 (): FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
|
@@ -52,6 +52,7 @@ describe("Test various utils used by Dr.CI", () => { | |||
await hasSimilarFailures( | |||
job, | |||
emptyBaseCommitDate, | |||
[], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry is there any testing for this? am i just looking in the wrong place
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me add a test for this, moving mergeCommits
is actually a boon here because I can now easily mock it
Fixes #5188
This is to address the mismatch reported in #5188 where some legit failures from the previous merge commit are marked as flaky. We already have the logic to gate against this in
isFailureFromPrevMergeCommit
. The bug here is that if a match from a subsequent broken trunk commit was found beforeisFailureFromPrevMergeCommit
was applied, the match was returned right away.There are 2 fixes here:
isFailureFromPrevMergeCommit
to all potential matches. Even if one failure shows up from the previous merge commit, the failure will be treated as a new failure.getPRMergeCommits
function call out ofhasSimilarFailures
because this is the same for all failures in the PR, so we don't need to call it multiple times for each failures.Testing
pytorch/pytorch#124045
BEFORE
❌ 9 New Failures, 3 Unrelated Failures
As of commit 0b8801e63882c96a3d8650d3076ffbc96cad37c6 with merge base c59a2369bec99dbbc088ac5477efe1ab6adc7ec8 ():
NEW FAILURES - The following jobs have failed:
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
inductor/test_max_autotune.py::TestMaxAutotune::test_max_autotune_remote_caching_dynamic_False
inductor/test_torchinductor.py::CpuTests::test_AllenaiLongformerBase_repro_cpu
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
AFTER
❌ 12 New Failures
As of commit 0b8801e63882c96a3d8650d3076ffbc96cad37c6 with merge base c59a2369bec99dbbc088ac5477efe1ab6adc7ec8 ():
NEW FAILURES - The following jobs have failed:
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_max_autotune.py::TestMaxAutotune::test_max_autotune_remote_caching_dynamic_False
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
inductor/test_torchinductor.py::CpuTests::test_AllenaiLongformerBase_repro_cpu
inductor/test_codecache.py::TestUtils::test_fresh_inductor_cache
pytorch/pytorch#128464
BEFORE
❌ 1 New Failure, 2 Unrelated Failures
As of commit b245737edbb1a5372b3b63df870aed9746e94958 with merge base 5d8c7f39d46699d8f8e92512309ea3499a29c08a ():
NEW FAILURE - The following job has failed:
>>> Lint for torch/csrc/autograd/profiler_kineto.cpp:
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
test_all_reduce_no_op_with_one_replica
Process completed with exit code 1.
AFTER
❌ 1 New Failure, 2 Unrelated Failures
As of commit b245737edbb1a5372b3b63df870aed9746e94958 with merge base 5d8c7f39d46699d8f8e92512309ea3499a29c08a ():
NEW FAILURE - The following job has failed:
>>> Lint for torch/csrc/autograd/profiler_kineto.cpp:
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
test_all_reduce_no_op_with_one_replica
Process completed with exit code 1.