Skip to content

Fix Error in daemon_unix.go and docker_cli_run_unit_test.go#41857

Merged
cpuguy83 merged 1 commit intomoby:masterfrom
gunadhya:Fix_Error_in_daemon_unix
Jan 5, 2021
Merged

Fix Error in daemon_unix.go and docker_cli_run_unit_test.go#41857
cpuguy83 merged 1 commit intomoby:masterfrom
gunadhya:Fix_Error_in_daemon_unix

Conversation

@gunadhya
Copy link
Contributor

@gunadhya gunadhya commented Jan 5, 2021

Signed-off-by: gunadhya 6939749+gunadhya@users.noreply.github.com
fixes #41852

- What I did

expected = "Minimum memory reservation allowed is 4MB"

return warnings, fmt.Errorf("Minimum memory reservation allowed is 4MB")

Fixed the error message as requested #41852 (comment)

- How I did it
Changed the error message from 4MB to 6MB

- How to verify it
gunadhya@64465f3

- Description for the changelog

error message stating the minimum memory reservation allowed is changed from 4MB to 6MB

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

image

Signed-off-by: gunadhya <6939749+gunadhya@users.noreply.github.com>
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.

LGTM, thanks!

@thaJeztah thaJeztah added this to the 20.10.3 milestone Jan 5, 2021
Copy link
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Invalid error message when providing 4MB memory reservations through docker-compose

3 participants