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

Disable retry build #4124

Closed
wants to merge 1 commit into from
Closed

Conversation

gtwang01
Copy link
Contributor

@gtwang01 gtwang01 commented Jan 10, 2025

Summary: gh run seems to time out every now and then, crashing the retry build. Changing to use gh view may resolve this issue.

Differential Revision: D68032708

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@gtwang01 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

1 similar comment
@facebook-github-bot
Copy link
Contributor

@gtwang01 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Summary:
```gh run``` seems to time out every now and then, crashing the retry build. I suspect this because whenever the retry build fails, it lacks any logging that would happen if ```gh run``` finishes.

Changing to use ```gh view``` and sleeping for 10 minutes before viewing again may resolve this issue, as it isn't continuously watching.


Test Plan:
https://github.com/gtwang01/faiss/actions/runs/12718383559/job/35456710369

{F1974306037} 
{F1974306103} 


Also, monitor nightlies to ensure it works properly

Reviewed By: mengdilin

Differential Revision: D68032708

Pulled By: gtwang01
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@gtwang01 merged this pull request in b9fe1dc.

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.

2 participants