Skip to content

Increase timeout for check_nodes_in_sync #20172

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 22, 2025
Merged

Increase timeout for check_nodes_in_sync #20172

merged 1 commit into from
Oct 22, 2025

Conversation

@emlowe
Copy link
Contributor

@emlowe emlowe commented Oct 21, 2025 •
edited
Loading

macOS Intel often hits this 300s timeout - try increasing to 600s

Testing passed 40/40 with this change

@aqk aqk added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Oct 21, 2025
@emlowe emlowe marked this pull request as ready for review October 21, 2025 18:06
@emlowe emlowe requested a review from a team as a code owner October 21, 2025 18:06
@emlowe emlowe added CI CI changes Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Oct 21, 2025
@pmaslana pmaslana merged commit 30c828b into main Oct 22, 2025
894 of 905 checks passed
@pmaslana pmaslana deleted the EL-test_long_reorgs_nodes-timeout branch October 22, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aqk aqk aqk approved these changes

@arvidn arvidn arvidn approved these changes

Assignees

No one assigned

Labels

Changed Required label for PR that categorizes merge commit message as "Changed" for changelog CI CI changes Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@emlowe @aqk @arvidn @pmaslana