Skip to content

Conversation

@adamruzicka
Copy link
Contributor

No description provided.

Copy link
Member

@ofedoren ofedoren left a comment

Choose a reason for hiding this comment

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

Seems like it's ready, is there something missing to undraft it?

Also, given that there doesn't seem to be an easy way to migrate to a different mode, this will probably require a big red release note.

@adamruzicka adamruzicka marked this pull request as ready for review November 27, 2025 14:19
@adamruzicka
Copy link
Contributor Author

is there something missing to undraft it?

Not anymore, I wanted to get the whole set of PRs ready first

@adamruzicka
Copy link
Contributor Author

this will probably require a big red release note.

And also a big version bump, maybe even up to 1.0.0

Copy link
Member

@ofedoren ofedoren left a comment

Choose a reason for hiding this comment

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

Ah, I guess a RM issue linked would be nice to have though. Otherwise ACK.

@adamruzicka adamruzicka changed the title Drop async-ssh Fixes #38927 - Drop async-ssh Nov 27, 2025
@adamruzicka
Copy link
Contributor Author

🍏

Copy link
Member

@ofedoren ofedoren left a comment

Choose a reason for hiding this comment

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

Thanks, @adamruzicka, let's get this in.

@ofedoren ofedoren merged commit d8d1811 into theforeman:master Nov 28, 2025
11 checks passed
@adamruzicka adamruzicka deleted the no-async-ssh branch November 28, 2025 11:58
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