Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

socket: return from loop after EOF #4805

Merged
merged 1 commit into from
Jan 20, 2024

Conversation

tonistiigi
Copy link
Member

@tonistiigi tonistiigi commented Jan 20, 2024

After a plugin receives the termination request via socket and closes the context, it atm goes into a tight loop that never returns.

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Jan 20, 2024

Codecov Report

Merging #4805 (8cd3b00) into master (e2519ae) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4805   +/-   ##
=======================================
  Coverage   59.63%   59.63%           
=======================================
  Files         287      287           
  Lines       24777    24777           
=======================================
  Hits        14775    14775           
  Misses       9114     9114           
  Partials      888      888           

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oof! Nice find, thanks!

LGTM

(cc @laurazard @neersighted)

@thaJeztah thaJeztah merged commit f18a476 into docker:master Jan 20, 2024
77 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants