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
Added decimal support to bucket quota selectors #2126
Conversation
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.
Observations:
- We could apply same changes in Create Bucket screen as well.
- With
mc
mc admin bucket quota local/b-bucket --hard 1.7GB
Successfully set bucket quota of 1.6 GiB with hard type on `b-bucket`
mc admin bucket quota local/b-bucket
Bucket `b-bucket` has hard quota of 1.6 GiB
so as per the above mc commands, ~0.1
is lesser capacity is set for quota
but if we update from the UI, it sets the exact value.
e.g 2.6 GB gets updated as 2.6 GB
Otherwise changes look good to me 👍
I think this is related to GB vs GiB conversion, so this should be expected |
Found issues with Add Bucket form validation. Will fix them on this PR as well |
- Fixed an issue with calculateBytes function - Fixed add bucket validation form Signed-off-by: Benjamin Perez <benjamin@bexsoft.net>
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.
LGTM
What does this do?
How does it look?
Signed-off-by: Benjamin Perez benjamin@bexsoft.net