Remove conflicting & inaccurate Docker Desktop memory warning #9144
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I propose we remove this warning for a couple of reasons:
The 3GB recommendation here conflicts with the 2.25 GB recommendation we make 18 lines above it.
This puts us in an awkward position of providing conflicting advice: we first ask the user to increase to 2.25GB, then we warn that they haven't increased it to 3GB
I have observed no performance degradation with 2GB of memory. Intuitively, with the exception of OOM pressure, the amount of memory has little measurable effect on Kubernetes performance. I would be willing to accept measurements that disprove this assertion, however.
If there was a warning to be made here, there is nothing that should make it a Docker Desktop specific warning
For people who intentionally allocate only 2GB of memory, the warning is noisy and there is no way to turn it off.
Fixes #8892