Skip to content

Issue #17487: Extend exceeding CI job#18577

Merged
romani merged 1 commit into
checkstyle:masterfrom
Pankraz76:fix-rw-timeout
Jan 10, 2026
Merged

Issue #17487: Extend exceeding CI job#18577
romani merged 1 commit into
checkstyle:masterfrom
Pankraz76:fix-rw-timeout

Conversation

@Pankraz76

@Pankraz76 Pankraz76 commented Jan 9, 2026

Copy link
Copy Markdown

Issue #17487: Extend exceeding CI job

@Pankraz76

Copy link
Copy Markdown
Author

Too long with no output (exceeded 10m0s): context deadline exceeded

@Pankraz76 Pankraz76 marked this pull request as ready for review January 9, 2026 14:41
Comment thread .circleci/config.yml
command:
description: "command to run"
type: string
default: ""

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

this is the default actually, no need for this.

@romani romani 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.

It is not a minor.

Update commit to reference pull number as prefix.
In PR description put explanation on problem, with link to failures.

Max timeout should be 15 minutes.
If we ever go above such limit, we will need to split execution in smaller parts

@Pankraz76

Copy link
Copy Markdown
Author

Max timeout should be 15 minutes.

yes only need 1-2 min, actually.

@Pankraz76 Pankraz76 changed the title minor: Extend exceeding rewrite job Issue #17487: Extend exceeding CI job Jan 9, 2026

@romani romani 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.

15 minutes should be our max for job, to void problems with CIs

@romani romani merged commit d7d1e49 into checkstyle:master Jan 10, 2026
119 checks passed
@Pankraz76

Copy link
Copy Markdown
Author

ty

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