Skip to content

Update build.sh#3789

Merged
sekyondaMeta merged 1 commit intomainfrom
aws_distributed_disable
Mar 9, 2026
Merged

Update build.sh#3789
sekyondaMeta merged 1 commit intomainfrom
aws_distributed_disable

Conversation

@sekyondaMeta
Copy link
Contributor

Getting a FileNotFoundError: [Errno 2] No such file or directory: 'beginner_source/aws_distributed_training_tutorial.py' in tutorial builds

File does not exist

Getting a FileNotFoundError: [Errno 2] No such file or directory: 'beginner_source/aws_distributed_training_tutorial.py'

File does not exist
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 9, 2026

🔗 Helpful Links

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

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

⏳ No Failures, 1 Pending

As of commit 06044aa with merge base 01bcdff (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla bot added the cla signed label Mar 9, 2026
@sekyondaMeta
Copy link
Contributor Author

@claude

@claude
Copy link

claude bot commented Mar 9, 2026

Claude finished @sekyondaMeta's task in 49s —— View job


Review of PR #3789

  • Gather context and read changed files
  • Verify the referenced tutorial file doesn't exist
  • Check for other references to the removed tutorial
  • Post review feedback

Summary: This PR comments out a call to remove_runnable_code.py for beginner_source/aws_distributed_training_tutorial.py in .jenkins/build.sh:51, because that tutorial file no longer exists and was causing a FileNotFoundError during builds.

Verdict: The change is correct and appropriate. The file beginner_source/aws_distributed_training_tutorial.py does not exist in the repository, confirming the build error. Commenting the line out is the right fix since the || true at the end of the original line should have suppressed the error at the shell level, but the Python script itself is raising the FileNotFoundError before the shell can catch it.

One additional note: There is another reference to this tutorial at .jenkins/build.sh:150:

rsync -av docs_with_plot/docs/ docs --exclude='**aws_distributed_training_tutorial*'

This rsync --exclude is harmless (excluding a non-existent file is a no-op), but it's dead code that could also be cleaned up for clarity. Not a blocker for this PR — just a minor housekeeping observation.

Otherwise, LGTM. The fix is minimal and targeted.


@svekars svekars added the build issue Issues relating to the tutorials build label Mar 9, 2026
@sekyondaMeta sekyondaMeta merged commit 9e4d9c6 into main Mar 9, 2026
40 checks passed
@sekyondaMeta sekyondaMeta deleted the aws_distributed_disable branch March 9, 2026 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build issue Issues relating to the tutorials build cla signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants