Skip to content

refactor(homes): persist appvm homes by default#1560

Merged
brianmcgillion merged 1 commit intotiiuae:mainfrom
mbssrc:home-persist
Nov 12, 2025
Merged

refactor(homes): persist appvm homes by default#1560
brianmcgillion merged 1 commit intotiiuae:mainfrom
mbssrc:home-persist

Conversation

@mbssrc
Copy link
Copy Markdown
Collaborator

@mbssrc mbssrc commented Nov 12, 2025

Description of Changes

Persist appvms 'appuser' home by default.

If appusers' home directories or files are explicitly defined in an appvm instance via the storagevm options, this setting will be ineffective - only the defined files and directories are then persisted. This is generally useful as apps often create app-specific folders.

Also adds shared folder for flatpack-vm.

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other: both work, in case of nixos-rebuild the old file structures are still present

Test Steps To Verify:

  1. ...

Persist appvms 'appuser' home by default.

If appusers' home directories or files are explicitly defined in an
appvm instance via the storagevm options, this setting will be ineffective
- only the defined files and directories are then persisted.

Signed-off-by: Manuel Bluhm <manuel@ssrc.tii.ae>
@brianmcgillion
Copy link
Copy Markdown
Collaborator

nice. verified

@brianmcgillion brianmcgillion merged commit 214f5db into tiiuae:main Nov 12, 2025
27 of 28 checks passed
@milva-unikie
Copy link
Copy Markdown

Thank you!

@milva-unikie milva-unikie added the Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon label Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants