Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

debos: Warn if parsed memory/scratchsize may be too small #510

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

debos: Warn if parsed memory/scratchsize may be too small #510

wants to merge 1 commit into from

Commits on Jul 1, 2024

  1. debos: Warn if parsed memory/scratchsize may be too small 

    Since the --memory and --scratchsize arguments are parsed with human-readable
suffixes into bytes (e.g 2048MB or 2GB), if a user does not put a suffix
the arguments are parsed as bytes which can cause issues later when a user
sets the memory to be 2048 assuming it will be parsed as MB.

Change the documentation to match reality and also add a warning if the user
has set either parameter less than the recommended default.

Closes: #509
Signed-off-by: Christopher Obbard <[email protected]>
    @obbardc
    obbardc committed Jul 1, 2024
    Configuration menu
    Copy the full SHA
    13f6822 View commit details
    Browse the repository at this point in the history