Skip to content

[25.0 backport] run: fix GetList return empty issue for throttledevice - #5341

Merged
vvoland merged 1 commit into
docker:25.0from
thaJeztah:25.0_backport_fix_bps_limit
Aug 12, 2024
Merged

[25.0 backport] run: fix GetList return empty issue for throttledevice#5341
vvoland merged 1 commit into
docker:25.0from
thaJeztah:25.0_backport_fix_bps_limit

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

Test "--device-read-bps" "--device-write-bps" will fail. The root cause is that GetList helper return empty as its local variable initialized to zero size.

This patch fix it by setting the related slice size to non-zero.

(cherry picked from commit 73e78a5)

- What I did

- How I did it

- How to verify it

- Description for the changelog

Fix `--device-read-bps` and `--device-write-bps` options not taking effect.

- A picture of a cute animal (not mandatory but encouraged)

Test "--device-read-bps" "--device-write-bps" will fail. The root
cause is that GetList helper return empty as its local variable
initialized to zero size.
This patch fix it by setting the related slice size to non-zero.
Signed-off-by: Jianyong Wu <wujianyong@hygon.cn>
Fixes: docker#5321
(cherry picked from commit 73e78a5)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 61.36%. Comparing base (6fb9a5b) to head (a73610d).
Report is 3 commits behind head on 25.0.

Additional details and impacted files
@@ Coverage Diff @@## 25.0 #5341 +/- ##
=======================================
Coverage 61.36% 61.36% =======================================
Files 290 290 Lines 20098 20098 =======================================
Hits 12334 12334 Misses 6871 6871 Partials 893 893 

@vvoland
vvoland merged commit 32ac720 into docker:25.0Aug 12, 2024
@thaJeztah
thaJeztah deleted the 25.0_backport_fix_bps_limit branch August 12, 2024 10:27
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@thaJeztah@codecov-commenter@vvoland@jongwu