Expose admin resource overrides beside VPS and dataset allocation controls - #503
Merged
Merged
Conversation
Match legacy VPS and dataset forms by placing allocation exceptions beside resource inputs. Allow explicit retry inside VPS confirmation after an allocation failure, and avoid resending unrelated advanced ZFS properties. Keep overrides admin-only and clear them after a successful save.
Use the checkbox test ID after the resource override label was clarified. This restores the overview smoke test and ensures member and read-only permission tests still check the actual control rather than an old label.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
English
Admins cannot readily find the legacy resource-allocation exception when increasing VPS RAM or dataset size: the VPS checkbox is buried in administration and the dataset checkbox inside collapsed advanced ZFS settings.
admin_override/admin_lock_typeAPI options independently of advanced dataset edits, without resending unrelated ZFS properties. Reset these per-save options after success.Contract verified against legacy
webui/pages/page_adminvps.php,webui/forms/dataset.forms.php, and API VPS/Dataset Update authorization. This bypasses the user's available allocation check; it does not bypass other API checks. No backend change or deployment is needed for the parameter.Validation:
Risk: allocation exceptions intentionally let admins exceed a user's allocation. They remain an explicit unchecked choice and API authorization is unchanged. No merge/deploy included.
Česky
Při navýšení RAM nebo datasetu byla výjimka z kontroly volných prostředků obtížně dohledatelná. Podle starého UI je nyní přímo u prostředků a velikosti datasetu; u VPS také v potvrzovacím dialogu, kde lze po chybě zaškrtnout výjimku a znovu uložit zachovaný návrh.
Používá stávající
admin_overrideaadmin_lock_type, jen pro administrátora. Výchozí stav je vypnutý a po úspěchu se volby resetují. U datasetu nevyžaduje otevření pokročilých vlastností a neodesílá vedlejší změny ZFS. Sjednocené české/anglické popisky také u SSD.Ověřeno: typecheck, lint, překlady, build a 28 unit testů. Širší fixture Playwright běh měl 100 úspěchů; všech 6 selhání bylo v testech opraveno a cílený běh následně prošel 12/12 na desktopu/mobilu. Zahrnuje zamítnutou alokaci, výslovné opakování s výjimkou, přesná těla požadavků, reset a oprávnění. Nejde o live VM ověření. Opravené starší očekávání návratu po smazání uživatelského datasetu odpovídá současné navigaci do úložiště VPS.
Ostatní kontroly API zůstávají platné. Bez změny backendu, merge nebo nasazení.
CI follow-up / Doplnění ověření
Updated overview/member/read-only assertions to target the actual override checkbox after its label changed. The CI failure was the obsolete
Admin overridetext; the relevant desktop/mobile specs now pass 8/8 locally. The unrelated OOM history test was flaky in the previous CI run and passed on retry; it is not changed here.Testy přehledu a oprávnění nyní kontrolují skutečný checkbox místo původního textu
Admin override. Příslušné desktopové i mobilní scénáře prošly lokálně 8/8. Nesouvisející test historie OOM byl v předchozím CI běhu nestabilní a prošel při opakování; toto PR jej nemění.