Skip to content

[test] use JK to force graph break on slow aliasing/mutation/dynamic_shape behavior#155257

Closed
bdhirsh wants to merge 1 commit intopytorch:mainfrom
bdhirsh:export-D76051868
Closed

[test] use JK to force graph break on slow aliasing/mutation/dynamic_shape behavior#155257
bdhirsh wants to merge 1 commit intopytorch:mainfrom
bdhirsh:export-D76051868

Conversation

@bdhirsh
Copy link
Collaborator

@bdhirsh bdhirsh commented Jun 5, 2025

Summary: test to unblock shampoo, needs cleanup

Test Plan:
CI

Rollback Plan:
steps:

  • jk.update:
    jk: pytorch/compiler:aliased_inputs_with_mutation_and_dyn_shapes_killswitch
    constant_bool: null
    consistent_pass_rate: null
    fractional_host_rollout: null
    sampling_rate: null
  • manual.note:
    content: Set it to false.

Reviewed By: c00w

Differential Revision: D76051868

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @chauhang @amjames

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 5, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/155257

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 Cancelled Job, 3 Unrelated Failures

As of commit c9e01d1 with merge base fc77269 (image):

CANCELLED JOB - The following job was cancelled. Please retry:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

UNSTABLE - The following jobs are marked as unstable, possibly due to flakiness on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D76051868

@bdhirsh bdhirsh added the topic: not user facing topic category label Jun 5, 2025
@c00w c00w self-requested a review June 5, 2025 22:05
@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Jun 5, 2025
@bdhirsh bdhirsh force-pushed the export-D76051868 branch from a83607a to e13d73b Compare June 5, 2025 23:59
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D76051868

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D76051868

@bdhirsh bdhirsh force-pushed the export-D76051868 branch from 069647b to a778782 Compare June 6, 2025 15:53
…shape behavior (pytorch#155257)

Summary:
Pull Request resolved: pytorch#155257

test to unblock shampoo, needs cleanup

Test Plan:
CI

Rollback Plan:
steps:
  - jk.update:
      jk: pytorch/compiler:aliased_inputs_with_mutation_and_dyn_shapes_killswitch
      constant_bool: null
      consistent_pass_rate: null
      fractional_host_rollout: null
      sampling_rate: null
  - manual.note:
      content: Set it to false.

Reviewed By: c00w

Differential Revision: D76051868
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D76051868

@bdhirsh bdhirsh force-pushed the export-D76051868 branch from a778782 to c9e01d1 Compare June 6, 2025 15:57
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: 1 jobs have failed, first few of them are: trunk / linux-jammy-rocm-py3.10 / test (distributed, 1, 1, linux.rocm.gpu.4)

Details for Dev Infra team Raised by workflow job

@clee2000
Copy link
Contributor

clee2000 commented Jun 9, 2025

@pytorchbot merge -f "merged internally"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

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.

5 participants