Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[16.0] Add a proper error message when user tries to share a file-based volume between apps#5394

Merged
rene merged 2 commits into
lf-edge:16.0from
europaul:backport/16.0/share-volume-error-msg
Nov 12, 2025
Merged

[16.0] Add a proper error message when user tries to share a file-based volume between apps#5394
rene merged 2 commits into
lf-edge:16.0from
europaul:backport/16.0/share-volume-error-msg

Conversation

@europaul

Copy link
Copy Markdown
Contributor

Description

Backport of #5350

PR dependencies

None

How to test and validate this PR

See the original PR.

Changelog notes

Added a proper error message when user tries to share a file-based volume between apps.

Checklist

  • I've added a reference link to the original PR
  • PR's title follows the template

Since we have both `VolumeStatus` and `VolumeRefStatus` types, we should
name variables `vs` and `vrs` respectively to avoid confusion.

Signed-off-by: Paul Gaiduk <[email protected]>
(cherry picked from commit 82fba0c)
When multiple app instances try to use the same file-based volume,
it results in a race condition and error. This is now properly logged
with a descriptive error message before the app instances start, so
there are no cryptic QEMU errors.

Signed-off-by: Paul Gaiduk <[email protected]>
(cherry picked from commit 9001b52)

@eriknordmark eriknordmark left a comment

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.

LGTM

@rene rene merged commit 25c6efd into lf-edge:16.0 Nov 12, 2025
44 of 45 checks passed
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.

3 participants