Skip to content

Enable timeout with stack in CI#585

Merged
jameslamb merged 8 commits into
rapidsai:mainfrom
pentschev:timeout-with-stack
Feb 17, 2026
Merged

Enable timeout with stack in CI#585
jameslamb merged 8 commits into
rapidsai:mainfrom
pentschev:timeout-with-stack

Conversation

@pentschev
Copy link
Copy Markdown
Member

Additionally print the value of maximum number of pending connections (somaxconn) to help debug test issues with large number of simultaneous connections.

@pentschev pentschev self-assigned this Feb 6, 2026
@pentschev pentschev requested a review from a team as a code owner February 6, 2026 11:36
@pentschev pentschev requested a review from rockhowse February 6, 2026 11:36
@pentschev pentschev added ci non-breaking Introduces a non-breaking change labels Feb 6, 2026
@pentschev pentschev added the improvement Improves an existing functionality label Feb 6, 2026
@pentschev pentschev requested a review from a team as a code owner February 6, 2026 12:48
@pentschev pentschev requested a review from jameslamb February 6, 2026 12:48
Copy link
Copy Markdown
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

Mostly fine, but please move the gdb dependency to conda-only groups. I don't think you intended to have pip install gdb ever happen.

Comment thread dependencies.yaml Outdated
- output_types: [conda, requirements, pyproject]
packages:
- cloudpickle
- gdb
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.

These should be moved to conda-specific groups. There isn't a gdb project on PyPI.

It looks like the scripts you're modifying only affect tests that use the conda packages, but just in case I'm wrong about that... if you want gdb at test time for wheels, you'll need to add code to install it at test time for the wheel-based jobs with apt / dnf / yum.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Absolutely, this was oversight by my part, should be fixed in 9ba818c. Thanks James for pointing it out.

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.

No problem, thanks for the fix.

@jameslamb jameslamb removed the request for review from rockhowse February 6, 2026 14:49
@pentschev
Copy link
Copy Markdown
Member Author

No rush, but could you also admin merge this next week @jameslamb ? Thank you!

@jameslamb
Copy link
Copy Markdown
Member

Yes absolutely, keep @-ing me until we get check-nightly-ci fixed. Sorry 😫

@jameslamb jameslamb merged commit 5b61b5e into rapidsai:main Feb 17, 2026
160 of 165 checks passed
@pentschev pentschev deleted the timeout-with-stack branch February 28, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants