Skip to content
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.

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

back out "Update Rust crate tokio to 1.39.2 (#6249)" #6356

Conversation

sunshowers
Copy link
Contributor

I'm having second thoughts about upgrading Tokio so close to release 10, so I
want to suggest backing it out for now and landing it early in the release 11
cycle. Putting up this PR so we have a place to discuss this.

This backs out commit d7d4bea.

Created using spr 1.3.6-beta.1
@sunshowers sunshowers marked this pull request as draft August 16, 2024 04:56
@sunshowers sunshowers marked this pull request as ready for review August 20, 2024 17:18
@sunshowers sunshowers changed the title (RFC) Back out "Update Rust crate tokio to 1.39.2 (#6249)" back out "Update Rust crate tokio to 1.39.2 (#6249)" Aug 20, 2024
@sunshowers
Copy link
Contributor Author

In the control plane meeting today we decided to roll back Tokio 1.39 for r10, with the goal to land it early in r11.

@sunshowers sunshowers merged commit 2e0025c into main Aug 20, 2024
24 checks passed
@sunshowers sunshowers deleted the sunshowers/spr/rfc-back-out-update-rust-crate-tokio-to-1392-6249 branch August 20, 2024 17:26
sunshowers added a commit that referenced this pull request Aug 27, 2024
Tokio 1.39/mio 1.0 switches out the illumos impl to being eventfd based. For release 10 we decided that that was too risky, so we switched back to Tokio 1.38.

Now that the r10 branch has been cut, we can go back and update Tokio to 1.39.3. We'd like to land this early in the cycle to get as much soak time as possible.

See:

* #6356
* #6249
* oxidecomputer/helios#169
* oxidecomputer/helios#171
* #6391
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