Skip to content

Adds journal option to container volume create.#1412

Merged
jglogan merged 1 commit intoapple:mainfrom
jglogan:ext4-journal
Apr 22, 2026
Merged

Adds journal option to container volume create.#1412
jglogan merged 1 commit intoapple:mainfrom
jglogan:ext4-journal

Conversation

@jglogan
Copy link
Copy Markdown
Contributor

@jglogan jglogan commented Apr 11, 2026

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Motivation and Context

Should make containers more resilient to ungraceful stops.

Testing

  • Tested locally
  • Added/updated tests
  • Added/updated docs

@jglogan jglogan marked this pull request as draft April 11, 2026 02:56
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 11, 2026
- Closes apple#1411.
- Default behavior doesn't change - with no
  `journal` option, no journal gets created.
@jglogan jglogan marked this pull request as ready for review April 22, 2026 00:02
Comment thread docs/command-reference.md

Driver options are passed with `--opt key=value`. The following options are supported for the default `local` driver:

* `size=<value>`: Volume size with optional unit suffix (K, M, G, T, P). Minimum 1 MiB. Equivalent to `-s`; if `-s` is also specified, `-s` takes precedence.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we have both -s <s> and this size=<value>?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@realrajaryan for any follow-up on this...

I believe it's just because size should be a driver option (not all volumes will allow you to choose a size; see the PR for SMB/NFS volumes, for example), but for the common case of ext4-backed volumes, it's probably the most useful one for the average user so -s/--size is there for convenience.

@jglogan jglogan merged commit 86611f5 into apple:main Apr 22, 2026
3 checks passed
@jglogan jglogan deleted the ext4-journal branch April 22, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request]: Add journal mode option to container volume create.

3 participants