Skip to content

chore(ci): set a 30 minute timeout for 'functional' tests - #3092

Merged
nejch merged 1 commit into
mainfrom
jlvillal/artifacts
Jan 23, 2025
Merged

nejch merged 1 commit into
mainfrom
jlvillal/artifacts

Conversation

@JohnVillalovos

@JohnVillalovos JohnVillalovos commented Jan 21, 2025 •

Copy link
Copy Markdown
Member

Currently the functional API test takes around 17 minutes to run. And
the functional CLI test takes around 12 minutes to run.

Occasionally a job gets stuck and will sit until the default 360
minutes job timeout occurs.

Now have a 30 minute timeout for the 'functional' tests.

@JohnVillalovos
JohnVillalovos force-pushed the jlvillal/artifacts branch 2 times, most recently from d292a78 to c40228f Compare January 21, 2025 19:24
@JohnVillalovos
JohnVillalovos force-pushed the jlvillal/artifacts branch 8 times, most recently from 96d546c to 43b0aa8 Compare January 22, 2025 16:57
@JohnVillalovos JohnVillalovos changed the title WIP: debugging hanging test chore(ci): set a 30 minute timeout for 'functional' tests Jan 22, 2025
@JohnVillalovos
JohnVillalovos requested a review from nejch January 22, 2025 16:58
@JohnVillalovos
JohnVillalovos force-pushed the jlvillal/artifacts branch 2 times, most recently from a2bee45 to c8d627a Compare January 22, 2025 18:01

@nejch nejch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

17 minutes 😭 I remember when it was around 6 minutes haha

@nejch
nejch enabled auto-merge (rebase) January 22, 2025 21:09
Currently the functional API test takes around 17 minutes to run. And
the functional CLI test takes around 12 minutes to run.

Occasionally a job gets stuck and will sit until the default 360
minutes job timeout occurs.

Now have a 30 minute timeout for the 'functional' tests.
@nejch
nejch merged commit e8d6953 into main Jan 23, 2025
@nejch
nejch deleted the jlvillal/artifacts branch January 23, 2025 00:33
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.

2 participants