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

kraft cloud deploy: Memory parameter fails parsing size suffixes, like M, G, ... #1513

Open
skuenzer opened this issue Apr 4, 2024 · 0 comments · May be fixed by #1687
Open

kraft cloud deploy: Memory parameter fails parsing size suffixes, like M, G, ... #1513

skuenzer opened this issue Apr 4, 2024 · 0 comments · May be fixed by #1687
Assignees
@craciunoiuc
Labels
kind/bug Something isn't working priority/low Nice to have issues thatare not urgent, but can be nagging.

Comments

@skuenzer
Copy link
Member

skuenzer commented Apr 4, 2024

Describe the bug

kraft run seems to support size units, I think for better user experience, the same should be the case for kraft cloud commands.

Steps to reproduce

No response

Expected behavior

No response

Which architectures were you using or does this bug affect?

No response

Which operating system were you using or does this bug affect?

No response

Relevant log output

E  invalid argument "1024M" for "-M, --memory" flag: strconv.ParseInt: parsing "1024M": invalid syntax
@skuenzer skuenzer added the kind/bug Something isn't working label Apr 4, 2024
@craciunoiuc craciunoiuc added the priority/low Nice to have issues thatare not urgent, but can be nagging. label Apr 8, 2024
@craciunoiuc craciunoiuc self-assigned this Apr 8, 2024
@craciunoiuc craciunoiuc linked a pull request May 31, 2024 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@craciunoiuc craciunoiuc

Labels
kind/bug Something isn't working priority/low Nice to have issues thatare not urgent, but can be nagging.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(internal): Allow for specifying optional measurement units craciunoiuc/kraftkit
2 participants
@skuenzer @craciunoiuc