Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

[WIP] Update tokio ecosystem #1880

Closed
wants to merge 3 commits into from
Closed

Conversation

caass
Copy link
Contributor

@caass caass commented Apr 27, 2021

We need to upgrade to tokio 1.0 to get unblocked on using the latest version of cloudflare-rs.

This PR updates tokio and related libs to 1.0 so we can continue using cloudflare-rs. This also unblocks tail v2.

related: #1881

@caass caass added the dependencies Pull requests that update a dependency file label Apr 27, 2021
@caass caass requested a review from a team as a code owner April 27, 2021 16:41
Cargo.toml Outdated Show resolved Hide resolved
Copy link
Contributor

@nataliescottdavidson nataliescottdavidson left a comment

Choose a reason for hiding this comment

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

publish and dev are working, buttail hangs endlessly on This may take a few seconds.... I can investigate a bit as I was just working on tail. Thanks for this @caass , getting us out of dependency hell!

Cargo.toml Outdated Show resolved Hide resolved
@caass
Copy link
Contributor Author

caass commented Apr 27, 2021

publish and dev are working, buttail hangs endlessly on This may take a few seconds.... I can investigate a bit as I was just working on tail. Thanks for this @caass , getting us out of dependency hell!

For anyone else reading (i talked with @nataliescottdavidson offline), I'm pretty sure the issue has to do with message passing between different tasks, that'll be the next commit, i think.

@caass caass mentioned this pull request Apr 28, 2021
@caass
Copy link
Contributor Author

caass commented Apr 28, 2021

Closing in favor of #1886

@caass caass closed this Apr 28, 2021
kflict referenced this pull request in CollectiveSynthesis2021/ORGIN2 Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants