Skip to content

Conversation

@SeanTAllen
Copy link
Contributor

This is the first iteration of policy enforcement at unmount. There is an additional set
of functionality that will come as part of a larger change in the near future.

With this commit, we record that a device has been unmounted such that it isn't eligible to
be used in any overlay after unmounting.

In a future commit, I will be adding disallowing unmounting a device that is being used by a
running container.

Signed-off-by: Sean T. Allen [email protected]

@SeanTAllen SeanTAllen requested a review from a team as a code owner September 15, 2021 16:49
This is the first iteration of policy enforcement at unmount. There is an additional set
of functionality that will come as part of a larger change in the near future.

With this commit, we record that a device has been unmounted such that it isn't eligible to
be used in any overlay after unmounting.

In a future commit, I will be adding disallowing unmounting a device that is being used by a
running container.

Signed-off-by: Sean T. Allen <[email protected]>
@SeanTAllen
Copy link
Contributor Author

@dcantah PTAL

BobsBurgers_620_Ain_tMissDebatin_27_S04_1_hires2

Copy link
Contributor

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

Thanks for the Bob's Burgers pic 😎

@anmaxvl anmaxvl merged commit 403164d into microsoft:master Sep 16, 2021
@SeanTAllen SeanTAllen deleted the policy-at-device-unmount branch September 17, 2021 11:54
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
This is the first iteration of policy enforcement at unmount. There is an additional set
of functionality that will come as part of a larger change in the near future.

With this commit, we record that a device has been unmounted such that it isn't eligible to
be used in any overlay after unmounting.

In a future commit, I will be adding disallowing unmounting a device that is being used by a
running container.

Signed-off-by: Sean T. Allen <[email protected]>
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