Skip to content

docs(adr): record server boot-mode policy (ADR-0009) - #6

Merged
kallioli merged 1 commit into
mainfrom
docs/adr-server-boot-mode
Apr 23, 2026
Merged

docs(adr): record server boot-mode policy (ADR-0009)#6
kallioli merged 1 commit into
mainfrom
docs/adr-server-boot-mode

Conversation

@kallioli

Copy link
Copy Markdown
Contributor

Context

PR #3 (server create) and PR #4 (server clone) flipped the default boot mode from boot-from-volume to boot-from-image. That's a policy decision worth recording as an ADR — the kind of change a future contributor could silently undo when refactoring if they don't know the why.

Changes

  • New `docs/adr/0009-server-boot-mode-policy.md` documenting:
    • the costs of the old BFV-unconditional default (Cinder quota drain, slower provisioning, dead flavor disk, divergence from Horizon),
    • the resolved-precedence rule (`--boot-from-volume` > `--boot-from-image` > auto on `flavor.disk`),
    • the behaviour of `--disk-size` in both modes,
    • the alternatives considered (unconditional ephemeral, `--ephemeral / --boot-from-volume` naming) and why each was rejected,
    • a pointer to the implementation (`_resolve_boot_mode`) and the test files.
  • Updated `docs/adr/README.md` index.

Docs-only; no code change, no CI impact beyond the markdown hooks (yaml/toml/whitespace).

Documents the non-obvious default shift introduced by PR #3 and PR #4:
`orca server create` and `orca server clone` now boot from image on
the compute's local disk by default, falling back to boot-from-volume
only when the flavor has `disk == 0`.

Records the alternatives considered (unconditional ephemeral, the
`--ephemeral / --boot-from-volume` pairing that was tried first) so a
future contributor reading the diff does not silently undo the choice.
@kallioli
kallioli merged commit edaf76b into main Apr 23, 2026
10 checks passed
@kallioli
kallioli deleted the docs/adr-server-boot-mode branch April 23, 2026 08:03
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.

1 participant