Skip to content

Add new key to control reattach disk to vm#97

Merged
vishesh92 merged 1 commit into
apache:mainfrom
shapeblue:reattach-disk-on-change
Mar 11, 2024
Merged

Add new key to control reattach disk to vm#97
vishesh92 merged 1 commit into
apache:mainfrom
shapeblue:reattach-disk-on-change

Conversation

@vishesh92

Copy link
Copy Markdown
Member

Fixes #72

@vishesh92 vishesh92 force-pushed the reattach-disk-on-change branch from ab646e4 to 63d12d5 Compare March 7, 2024 10:40
@vishesh92 vishesh92 added this to the v0.5.0 milestone Mar 7, 2024
@vishesh92 vishesh92 requested a review from kiranchavala March 7, 2024 12:02
@vishesh92 vishesh92 force-pushed the reattach-disk-on-change branch from 63d12d5 to 2860a96 Compare March 7, 2024 12:02
@vishesh92 vishesh92 changed the title Add new key to control reatch disk to vm Add new key to control reattach disk to vm Mar 7, 2024

@yadvr yadvr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM didn't test it

@kiranchavala kiranchavala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, tested it manually and, the resize operation happened without any detach of the disk

  # cloudstack_disk.test-disk will be updated in-place
  ~ resource "cloudstack_disk" "test-disk" {
        id                 = "9d87be3a-1199-4cee-b491-3ef4edbec428"
        name               = "test-disk"
      ~ size               = 15 -> 20
        tags               = {}
        # (7 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Screenshot 2024-03-11 at 1 42 50 PM

@vishesh92 vishesh92 merged commit 9b1501e into apache:main Mar 11, 2024
@vishesh92 vishesh92 deleted the reattach-disk-on-change branch March 11, 2024 08:23
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.

Support hot resize of disk

3 participants