Remove scheduler::Error

[nightkr]

#826 neuters scheduler::Error into a void type (since DelayQueue is now infallible). This followup PR removes it completely, as well as any usages of it.

[codecov-commenter]

Codecov Report

Merging #827 (45d87b0) into master (077e0f8) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #827      +/-   ##
==========================================
+ Coverage   69.92%   70.00%   +0.08%     
==========================================
  Files          58       58              
  Lines        3983     3977       -6     
==========================================
- Hits         2785     2784       -1     
+ Misses       1198     1193       -5     

Impacted Files	Coverage Δ
kube-runtime/src/controller/mod.rs	0.00% <ø> (ø)
kube-runtime/src/controller/runner.rs	95.16% <100.00%> (+1.41%)	⬆️
kube-runtime/src/scheduler.rs	96.87% <100.00%> (+0.57%)	⬆️
kube-runtime/src/wait.rs	70.00% <0.00%> (+2.00%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 077e0f8...45d87b0. Read the comment docs.

[nightkr]

@clux Is it fine to ignore the cargo-deny failures for now, or do we want to wait for the upstreams to release their own upgrades?

[clux]

LGTM. duplicate dep from h2 via hyper tho.

[clux]

hm, to me, this one is probably OK to merge - i don't think we expose anything from tokio_util that will be needed by other users to pass to their tokio_util.

it would be nice to know when we can remove ignore directives though somehow :(

[nightkr]

Cargo deny will start yelling once there are no more dependencies that match the skip clause. The downside is that this would probably only pop up in some completely unrelated PR...

[clux]

I think it's also set to be a warning - so we would have to just catch it in the output atm.

[clux]

let's put a pin in this until after a 0.69 release so we can merge all the ones that require cargo-deny overrides after that.

holding on

[nightkr]

@clux Are we ready to merge this now that 0.69 is out?

[clux]

ah, yes, let's carry on :-)