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.
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
rbd: support size suffixes for size-based options #5481
Conversation
@jdurgin , I have checked strict_sistrtoll function: and it returns only value of option if user will not specify size suffix with the value which will be bytes only as these options default value is bytes so we don't need to handle any specific case for these options. |
"stripe_count": 8, | ||
"stripe_unit": 1048576 | ||
} | ||
$ rbd rm test --no-progress |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you include a test case with non-byte units too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Sage for review , sure I will add it!
This patch will provide support for size suffixes for below given options: --stripe-unit <size in B/K/M> --io-size <size in B/K/M/G/T> --io-total <size in B/K/M/G/T> Signed-off-by: Vikhyat Umrao <vumrao@redhat.com>
c58a5d6
to
cd4ac1c
Compare
Please ignore the bot failure on Ubuntu, it is a false negative that has been fixed. |
@dachary thanks ! |
@jdurgin Thanks Josh ! |
This patch will provide support for size suffixes for
below given options:
--stripe-unit [size in B/K/M]
--io-size [size in B/K/M/G/T]
--io-total [size in B/K/M/G/T]
Signed-off-by: Vikhyat Umrao vumrao@redhat.com