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

Rollback tokio to LTS release v1.14 #25028

Merged
merged 1 commit into from
May 6, 2022

Conversation

CriesofCarrots
Copy link
Contributor

Problem

The RPC stall problem we are debugging appears when upgrading tokio from v1.15 to v1.16. Our master branch is already on tokio v1.17, which means it will suffer the stalls.

Summary of Changes

Rollback tokio, and pin to the long-term support release v1.14, until we can figure out why newer tokio breaks things.

Inspired by #24644

(Last one. Sorry for doing this in the exactly wrong order, but each of these needed a little hand-holding)

@CriesofCarrots CriesofCarrots requested a review from t-nelson May 6, 2022 01:39
@CriesofCarrots CriesofCarrots added the automerge Merge this Pull Request automatically once CI passes label May 6, 2022
@codecov
Copy link

codecov bot commented May 6, 2022

Codecov Report

Merging #25028 (d1cc433) into master (69a0ff9) will increase coverage by 0.0%.
The diff coverage is 88.1%.

@@           Coverage Diff            @@
##           master   #25028    +/-   ##
========================================
  Coverage    82.0%    82.0%            
========================================
  Files         598      598            
  Lines      165882   166085   +203     
========================================
+ Hits       136125   136322   +197     
- Misses      29757    29763     +6     

@mergify mergify bot merged commit c4aa082 into solana-labs:master May 6, 2022
@fabioberger
Copy link

Any update on this @CriesofCarrots? solana-client is conflicting with other dependencies that depend on newer versions of Tokio.

@CriesofCarrots
Copy link
Contributor Author

Any update on this?

No, sorry @fabioberger , I don't have anything new to report at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants