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

DEV: bump timeout for non-scheduled builds #924

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Conversation

fitzy101
Copy link
Contributor

The most recent build timed out at 30 minutes. Bump timeout temporarily to get a gauge on how long self-hosted builds take (push to Dockerhub looks to be slower than Github-hosted runners).

@fitzy101 fitzy101 requested a review from tgxworld January 23, 2025 23:57
@fitzy101
Copy link
Contributor Author

@fitzy101 fitzy101 requested review from CvX and davidtaylorhq January 23, 2025 23:58
The most recent build timed out at 30 minutes. Bump timeout temporarily
to get a gauge on how long self-hosted builds take (push to Dockerhub
looks to be slower than Github-hosted runners).
@fitzy101
Copy link
Contributor Author

The successful amd64 build here took 30min 2s 😮‍💨

https://github.com/discourse/discourse_docker/actions/runs/12940557907/job/36095025663

So it looks like we just exceeded the 30min timeout

@tgxworld tgxworld merged commit 8057b69 into main Jan 24, 2025
5 checks passed
@tgxworld tgxworld deleted the fitzy/bump-timeout branch January 24, 2025 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants