Skip to content

Add 60 minute timeout to CI jobs#922

Open
benthecarman wants to merge 2 commits into
lightningdevkit:mainfrom
benthecarman:ci-timeouts
Open

Add 60 minute timeout to CI jobs#922
benthecarman wants to merge 2 commits into
lightningdevkit:mainfrom
benthecarman:ci-timeouts

Conversation

@benthecarman
Copy link
Copy Markdown
Contributor

We ran out of our CI limit largely from ldk-node. We had a few jobs this week run for multiple hours because of a hanging test. Add 60 minute timeout to all our jobs to prevent this in the future.

We ran out of our CI limit largely from ldk-node. We had a few jobs this
week run for multiple hours because of a hanging test. Add 60 minute
timeout to all our jobs to prevent this in the future.
@benthecarman benthecarman requested a review from tnull June 5, 2026 15:57
@ldk-reviews-bot
Copy link
Copy Markdown

ldk-reviews-bot commented Jun 5, 2026

👋 Thanks for assigning @tnull as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

Copy link
Copy Markdown
Collaborator

@tnull tnull left a comment

Choose a reason for hiding this comment

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

We ran out of our CI limit largely from ldk-node.

Not opposed to this change, but note that it's mainly ldk-node as rust-lightning has switched to a hosted runner for a while. The more important fix here is to also switch LDK Node over for all jobs where we can. I need to finally do that.

@benthecarman
Copy link
Copy Markdown
Contributor Author

We ran out of our CI limit largely from ldk-node.

Not opposed to this change, but note that it's mainly ldk-node as rust-lightning has switched to a hosted runner for a while. The more important fix here is to also switch LDK Node over for all jobs where we can. I need to finally do that.

Added a commit for that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants