Skip to content

Conversation

@cben
Copy link
Collaborator

@cben cben commented Mar 5, 2022

stealing @kjetijor's commit 0781a9d from #537 (again, this time on master branch)
I'm still not sure where net/smtp actually gets require-d, but pretty certain it's not in kubeclient itself so a dev dependency sounds right.

Fixes #543 🎉

Kjetil Joergensen and others added 2 commits March 5, 2022 23:14
@cben
Copy link
Collaborator Author

cben commented Mar 5, 2022

2.5 run was just stuck, which I'm sometimes seeing recently. minitest starts, and then... nothing for 20min? (because I cancelled manually. saw 5hours in another run!)

Run options: --seed 40851

# Running:

Error: The operation was canceled.

re-running...

@cben
Copy link
Collaborator Author

cben commented Mar 5, 2022

2.5 stuck again. But OK, 3.1 and ruby-head (3.2-to-be) passed, merging.

I'll add test timeouts in another PR.

@cben cben merged commit 0f35317 into ManageIQ:master Mar 5, 2022
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.

Ruby 3.1 support

1 participant