Skip to content

refactor(BA-7761): switch runtime_variant_preset update DTO to Unset - #14419

Open
wngh4628 wants to merge 2 commits into
mainfrom
feature/BA-7761-runtime-variant-preset-unset
Open

refactor(BA-7761): switch runtime_variant_preset update DTO to Unset#14419
wngh4628 wants to merge 2 commits into
mainfrom
feature/BA-7761-runtime-variant-preset-unset

Conversation

@wngh4628

@wngh4628 wngh4628 commented Sep 8, 2026

Copy link
Copy Markdown

Switch the runtime_variant_preset update DTO from the legacy Sentinel marker to Unset (omitted = no change, null = clear).

  • common/dto/manager/v2/runtime_variant_preset/request.py: UpdateRuntimeVariantPresetInput.description, default_value, category, display_name, and ui_option change from X | Sentinel | None = SENTINEL to X | None | Unset = UNSET; each field description now reads "Omit to leave unchanged; null clears."; the default_value validator skips Unset instead of SENTINEL.
  • manager/api/adapters/runtime_variant_preset/adapter.py: the five SENTINEL branches are replaced with TriState.from_unset(...).

🤖 Generated with Claude Code

https://claude.ai/code/session_017idp6rKVV33XrzJG1yWYXo


📚 Documentation preview 📚: https://sorna--14419.org.readthedocs.build/en/14419/


📚 Documentation preview 📚: https://sorna-ko--14419.org.readthedocs.build/ko/14419/

@wngh4628
wngh4628 requested a review from a team as a code owner September 8, 2026 06:57
@github-actions github-actions Bot added the size:M 30~100 LoC label Sep 8, 2026
@github-actions github-actions Bot added comp:manager Related to Manager component comp:common Related to Common component area:docs Documentations labels Sep 8, 2026
@wngh4628
wngh4628 force-pushed the feature/BA-7761-runtime-variant-preset-unset branch from b903f0e to 9918d43 Compare September 8, 2026 07:26
Co-authored-by: octodog <mu001@lablup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Documentations comp:common Related to Common component comp:manager Related to Manager component size:M 30~100 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant