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

Fix nightly build. They've been failing due to missing secrets #1325

Merged
merged 3 commits into from
Sep 26, 2024

Conversation

andrewkho
Copy link
Contributor

@andrewkho andrewkho commented Sep 26, 2024

Unsure if CI will succeed without this, I don't see it referenced in many other pytorch repos though, and none of our tests which build seem to include it either.

From this workflow:
https://github.com/pytorch/data/actions/workflows/nightly_release.yml

Invalid workflow file: .github/workflows/nightly_release.yml#L18
The workflow is not valid. .github/workflows/nightly_release.yml (Line: 18, Col: 41): Invalid secret, PYTORCH_BINARY_AWS_ACCESS_KEY_ID is not defined in the referenced workflow. .github/workflows/nightly_release.yml (Line: 19, Col: 45): Invalid secret, PYTORCH_BINARY_AWS_SECRET_ACCESS_KEY is not defined in the referenced workflow.

Update digging through blame, I can see that test_release and pull_release had these keys dropped as part of this PR, probably just missed the nightly_release update: de5791b

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 26, 2024
Copy link

pytorch-bot bot commented Sep 26, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit b318ac6 with merge base ff01a06 (image):
💚 Looks good so far! There are no failures yet. 💚

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

Copy link
Contributor

@divyanshk divyanshk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@andrewkho andrewkho merged commit 1a98f21 into main Sep 26, 2024
47 checks passed
@andrewkho andrewkho deleted the andrewkh/fix-nightly branch September 26, 2024 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants