Vector: Upgrade `hyper` for `futures` 0.3

Created on 23 Mar 2020  ยท  5Comments  ยท  Source: timberio/vector

As part of the https://github.com/timberio/vector/issues/1142, upgrade hyper to v0.13+.


Here's a list of dependencies that are themselves blocked on tokio 0.1:

Crates that are ready for the upgrade:

  • hyper
  • hyper-tls
  • hyper-openssl
  • reqwest
  • rusoto_*
  • kube
  • hyperlocal
  • warp

Overall, not that bad.

task tech debt

All 5 comments

$ cargo tree -i -p hyper:0.12.35
hyper v0.12.35
โ”œโ”€โ”€ hyper-openssl v0.7.1
โ”‚   โ””โ”€โ”€ shiplift v0.6.0
โ”‚       โ””โ”€โ”€ vector v0.9.0 (.../vector)
โ”œโ”€โ”€ hyper-tls v0.3.2
โ”‚   โ”œโ”€โ”€ reqwest v0.9.24
โ”‚   โ”‚   โ”œโ”€โ”€ goauth v0.6.1
โ”‚   โ”‚   โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚   โ”‚   โ””โ”€โ”€ kube v0.16.1
โ”‚   โ”‚       [dev-dependencies]
โ”‚   โ”‚       โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚   โ””โ”€โ”€ rusoto_core v0.41.0
โ”‚       โ”œโ”€โ”€ rusoto_cloudwatch v0.41.0
โ”‚       โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚       โ”œโ”€โ”€ rusoto_firehose v0.41.0
โ”‚       โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚       โ”œโ”€โ”€ rusoto_kinesis v0.41.0
โ”‚       โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚       โ”œโ”€โ”€ rusoto_logs v0.41.0
โ”‚       โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚       โ”œโ”€โ”€ rusoto_s3 v0.41.0
โ”‚       โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚       โ”œโ”€โ”€ rusoto_sts v0.41.0
โ”‚       โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”‚       โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”œโ”€โ”€ hyperlocal v0.6.0
โ”‚   โ””โ”€โ”€ shiplift v0.6.0 (*)
โ”œโ”€โ”€ reqwest v0.9.24 (*)
โ”œโ”€โ”€ rusoto_core v0.41.0 (*)
โ”œโ”€โ”€ rusoto_credential v0.41.1
โ”‚   โ”œโ”€โ”€ rusoto_core v0.41.0 (*)
โ”‚   โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
โ”œโ”€โ”€ shiplift v0.6.0 (*)
โ””โ”€โ”€ warp v0.1.21 (git+https://github.com/timberio/warp?branch=0.1.x#017965c627047bf0e53062940e7def127cb21649)
    โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)
[dev-dependencies]
โ””โ”€โ”€ tracing-metrics v0.1.0 (.../vector/lib/tracing-metrics)
    โ””โ”€โ”€ vector v0.9.0 (.../vector) (*)

The sink side of this has been done and you can find the implementation in http2. Source and Prometheus sinks that use wrap will still need an upgrade.

@ktff this is part of our Tech-Debt Payment #1: Move to Tokio 0.2/Futures 0.3 Project. I'm assigning _just_ this issue to you. I've assigned #2118 and #1142 to @fanatid. I'm assuming this work won't conflict. If it does, let me know and we can try to reassign issues in a way that doesn't.

@fanatid can you prioritize updating src/sinks/aws_cloudwatch_logs/mod.rs to use
src/sinks/util/rusoto2.rs instead of src/sinks/util/rusoto.rs. With that and #2808 being merged, we'll be able to finish the upgrade of hyper and hyper-openssl.

Sure, it was done in #2777 but need move to new PR, should be ready soon.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

leebenson picture leebenson  ยท  3Comments

LucioFranco picture LucioFranco  ยท  3Comments

raghu999 picture raghu999  ยท  3Comments

binarylogic picture binarylogic  ยท  3Comments

LucioFranco picture LucioFranco  ยท  3Comments