Skip to content

Commit

Permalink
turn one_shot off for docker stats (#475)
Browse files Browse the repository at this point in the history
the docker.stats() call fails with stream: false, one_shot: true for
docker 24.0.7 (this is because the response for those options now
returns an object that does not have the name or id fields set, which
bollard considers to be required)

this should be reverted once
fussybeaver/bollard#347 has landed.
  • Loading branch information
pretentious7 committed Nov 7, 2023
1 parent 3e6c729 commit 87da5d2
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion drone/src/agent/engines/docker/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,9 @@ impl DockerInterface {
fn get_stats(&self, backend_id: &BackendId) -> impl Stream<Item = Stats> {
let options = StatsOptions {
stream: false,
one_shot: true,
// one_shot is set to false until https://github.com/fussybeaver/bollard/pull/347
// has landed.
one_shot: false,
};

let ticker = IntervalStream::new({
Expand Down

0 comments on commit 87da5d2

Please sign in to comment.