Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent custom storage volumes of type block to be used more than once #12463

Conversation

monstermunchkin
Copy link
Contributor

Fixes #12254

@github-actions github-actions bot added Documentation Documentation needs updating API Changes to the REST API labels Oct 25, 2023
@monstermunchkin monstermunchkin force-pushed the issues/12254-block-storage-volumes branch from b93770f to f681555 Compare October 26, 2023 07:14
@monstermunchkin monstermunchkin marked this pull request as ready for review October 26, 2023 20:23
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
The `Update` function returns `map[string]Device` values despite there
being a `Devices` type which is the same thing.

This commit changes the return types to be `Devices` instead.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
This adds a new function `AllowedInstanceDevices`. It ensures that
custom block volumes are attached to a single instance only if
`security.shared` is set to `false` on the custom volume.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
This commit adds checks for ensuring custom block volumes are attached
to a single instance at once unless `security.shared` is set to `true`
on the block volume.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
This adds a check for ensuring that custom block volumes are attached to
a single instance at once unless `security.shared` is set to `true` on
the storage volume.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
@monstermunchkin monstermunchkin force-pushed the issues/12254-block-storage-volumes branch from f681555 to 9a950d4 Compare November 17, 2023 15:06
@tomponline
Copy link
Member

@MusicDin want to take this one over?

@MusicDin
Copy link
Member

MusicDin commented Dec 6, 2023

Sorry, I've somehow missed the notification for this one. Will finish it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Changes to the REST API Documentation Documentation needs updating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prevent custom storage volumes to be used more than once
3 participants