Skip to content

Add TB to the FIFO quota units and make it the default (#253) - #257

Merged
AnthonyCvn merged 3 commits into
mainfrom
253-fifo-quota-tb
Nov 3, 2026
Merged

AnthonyCvn merged 3 commits into
mainfrom
253-fifo-quota-tb

Conversation

@AnthonyCvn

Copy link
Copy Markdown
Member

Closes #253

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features) (not needed: no docs cover the quota unit selector)
  • CHANGELOG.md has been updated (for bug fixes / features / docs)

What kind of change does this PR introduce?

Feature (UI): improved bucket quota settings form.

What was changed?

The quota size selector now offers TB in addition to B, KB, MB, and GB, and new quotas default to 1 TB. Quota size is hidden when the quota type is NONE and shown only for FIFO or HARD, with a default of 1 TB pre-filled when the type is first switched to an active state. Quota size is validated on submit and non-integer input shows "Quota size must be a whole number". The settings modal now discards unsaved edits on close, and the Create/Update button shows a loading spinner while the request is in flight.

Related issues

No

Does this PR introduce a breaking change?

No. Stored quota values and Max Block Size behave as before.

Other information:

@AnthonyCvn AnthonyCvn self-assigned this Nov 2, 2026
@AnthonyCvn
AnthonyCvn requested a review from atimin November 2, 2026 18:17
@AnthonyCvn
AnthonyCvn enabled auto-merge (squash) November 2, 2026 18:23
ghulamali9020 and others added 3 commits November 3, 2026 19:12
The quota size selector stopped at GB and defaulted to MB, so a value
typed as TB could set a much smaller FIFO quota. Add a decimal TB factor,
offer it for the quota only, and select it for a new quota. Existing
quotas keep the unit picked by calcBestFactor, and Max Block Size keeps
its B to GB units with the MB default.
@AnthonyCvn
AnthonyCvn merged commit f803db6 into main Nov 3, 2026
2 checks passed
@AnthonyCvn
AnthonyCvn deleted the 253-fifo-quota-tb branch November 3, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default FIFO quota size unit to TB

3 participants