Skip to content

[26.1 backport] run: fix GetList return empty issue for throttledevice - #5340

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

[26.1 backport] run: fix GetList return empty issue for throttledevice#5340
vvoland merged 1 commit into
docker:26.1from
thaJeztah:26.1_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 (a72d7cd) to head (e1246ba).

Additional details and impacted files
@@ Coverage Diff @@## 26.1 #5340 +/- ##
=======================================
Coverage 61.36% 61.36% =======================================
Files 298 298 Lines 20712 20712 =======================================
Hits 12709 12709 Misses 7102 7102 Partials 901 901 

@vvoland
vvoland merged commit dea625c into docker:26.1Aug 12, 2024
@thaJeztah
thaJeztah deleted the 26.1_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