Skip to content

[20.10 backport] printServerWarningsLegacy: silence "No kernel memory limit support"; silence "No oom kill disable support" on cgroup v2#3099

Merged
silvin-lubecki merged 2 commits intodocker:20.10from
thaJeztah:20.10_backport_silence_unhandleable_deprecated_warnings
May 19, 2021
Merged

Conversation

@thaJeztah
Copy link
Member

backport of #3077

- What I did
Removed unhandleable warnings from printServerWarningsLegacy().

- How I did it

These warnings have been already silenced on the daemon side.

- How to verify it
Before:

$ docker info
...
Cgroup Version: 2
...

WARNING: No kernel memory limit support
WARNING: No oom kill disable support

After: (no warning)

The kernel memory limit is deprecated in Docker 20.10.0,
and its support was removed in runc v1.0.0-rc94.
So, this warning can be safely removed.

Relevant: moby/moby@b8ca7de

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
(cherry picked from commit 731f52c)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
…group v2

The warning should be ignored on cgroup v2 hosts.

Relevant: moby/moby@8086443

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
(cherry picked from commit 05ec018)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

@AkihiroSuda @silvin-lubecki ptal

@codecov-commenter
Copy link

Codecov Report

Merging #3099 (12e2f94) into 20.10 (9780f41) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            20.10    #3099      +/-   ##
==========================================
- Coverage   57.02%   57.01%   -0.01%     
==========================================
  Files         297      297              
  Lines       18679    18677       -2     
==========================================
- Hits        10651    10649       -2     
  Misses       7163     7163              
  Partials      865      865              

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

@silvin-lubecki silvin-lubecki merged commit 54b529f into docker:20.10 May 19, 2021
@thaJeztah thaJeztah deleted the 20.10_backport_silence_unhandleable_deprecated_warnings branch May 19, 2021 09:44
Sign up for free to 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