Skip to content

Conversation

@Horiodino
Copy link
Contributor

Description:
added validation to ensure hostPath is part of a configured mount. if the path is not mounted, an error is returned.

Testing:
added unit tests for isMounted

@Horiodino Horiodino force-pushed the hostPath_mount_validation branch from 75f0a14 to 84ff74e Compare December 1, 2025 10:01
@Horiodino Horiodino force-pushed the hostPath_mount_validation branch from 84ff74e to c44939c Compare December 1, 2025 12:25
@Horiodino Horiodino requested a review from AkihiroSuda December 2, 2025 13:43
@Horiodino Horiodino force-pushed the hostPath_mount_validation branch from 57736fb to 6f59ab8 Compare December 3, 2025 12:31
Copy link
Member

Choose a reason for hiding this comment

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

Tests are failing on Windows

Also please squash the commits
https://lima-vm.io/docs/dev/git/#squashing-commits

Copy link
Member

Choose a reason for hiding this comment

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

ping @Horiodino

Copy link
Contributor Author

Choose a reason for hiding this comment

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

got caught up with my end-semester exams, but back now

@AkihiroSuda AkihiroSuda added this to the v2.1.0 (?) milestone Dec 8, 2025
@Horiodino Horiodino force-pushed the hostPath_mount_validation branch from 6f59ab8 to 202d7c9 Compare December 8, 2025 11:25
Signed-off-by: Praful Khanduri <[email protected]>

added tests for hostPath validation

Signed-off-by: Praful Khanduri <[email protected]>

return path translation logs to agent

Signed-off-by: Praful Khanduri <[email protected]>

refactor : use constant for io.lima-vm/warnings

Signed-off-by: Praful <[email protected]>

use path.Join for cross-platform abs paths

Signed-off-by: Praful <[email protected]>
@Horiodino Horiodino force-pushed the hostPath_mount_validation branch from 202d7c9 to 97d457a Compare December 17, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants