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

azurerm_virtual_machine does not update state file if disks are renumbered #20

Closed
hashibot opened this issue Jun 13, 2017 · 2 comments · Fixed by #426
Closed

azurerm_virtual_machine does not update state file if disks are renumbered #20

hashibot opened this issue Jun 13, 2017 · 2 comments · Fixed by #426

Comments

@hashibot
Copy link

This issue was originally opened by @cchildress as hashicorp/terraform#9437. It was migrated here as part of the provider split. The original body of the issue is below.


Terraform Version

0.7.3 (waiting for 0.7.6 to come out due to [GH-9122]

Affected Resource(s)

azurerm_virtual_machine

Terraform Configuration Files

Before:

  storage_data_disk {
    name = "${var.node_name}_data_disk_premium_01"
    vhd_uri = "${var.node_data_storage_account_premium_blob_endpoint}${azurerm_storage_container.node_container_data_premium.name}/${var.node_name}_data_premium_01.vhd"
    create_option = "Empty"
    disk_size_gb = 1023
    lun = 0
  }
  delete_data_disks_on_termination = true

After:

  storage_data_disk {
    name = "${var.node_name}_data_disk_standard_01"
    vhd_uri = "${var.node_data_storage_account_standard_blob_endpoint}${azurerm_storage_container.node_container_data_standard.name}/${var.node_name}_data_standard_01.vhd"
    create_option = "Empty"
    disk_size_gb = 1023
    lun = 0
  }
  storage_data_disk {
    name = "${var.node_name}_data_disk_premium_01"
    vhd_uri = "${var.node_data_storage_account_premium_blob_endpoint}${azurerm_storage_container.node_container_data_premium.name}/${var.node_name}_data_premium_01.vhd"
    create_option = "Empty"
    disk_size_gb = 1023
    lun = 32
  }
  delete_data_disks_on_termination = true

Debug Output

~ module.<some_server>.azurerm_virtual_machine.node
    storage_data_disk.0.lun:     "32" => "0"
    storage_data_disk.0.name:    "<some_server>_data_disk_premium_01" => "<some_server>_data_disk_standard_01"
    storage_data_disk.0.vhd_uri: "<foo>" => "<bar>"
    storage_data_disk.1.lun:     "0" => "32"
    storage_data_disk.1.name:    "<some_server>_data_disk_standard_01" => "<some_server>_data_disk_premium_01"
    storage_data_disk.1.vhd_uri: "<bar>" => "<foo>"

Expected Behavior

This change is successfully applied to Azure, but does not appear to be recored properly in the state file.

Actual Behavior

This shows up as a change to be made in every following terraform plan

Steps to Reproduce

  1. Set up a VM using the before layout.
  2. Change the Terraform config to match the after.
  3. Run a terraform apply and confirm you see the changes in the Azure portal
  4. Run terraform plan and the changes should be listed as pending still.

Important Factoids

I tried just editing the state file to see if it was something that could be manually corrected. Maybe I missed a spot, but I could never get it to see the change as completed.

@hashibot hashibot added the bug label Jun 13, 2017
apparentlymart pushed a commit that referenced this issue Jul 22, 2017
…ion (#14341)

* initial commit - 101-vm-from-user-image

* changed branch name

* not deploying - storage problems

* provisions vm but image not properly prepared

* storage not correct

* provisions properly

* changed main.tf to azuredeploy.tf

* added tfvars and info for README

* tfvars ignored and corrected file ext

* added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing

* deploy.sh to be executable

* executable deploy files

* added CI files; changed vars

* prep for PR

* removal of old folder

* prep for PR

* wrong args for travis

* more PR prep

* updated README

* commented out variables in terraform.tfvars

* Topic 101 vm from user image (#2)

* initial commit - 101-vm-from-user-image
* added tfvars and info for README
* added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing
* prep for PR

* added new template

* oops, left off master

* prep for PR

* correct repository for destination

* renamed scripts to be more intuitive; added check for docker

* merge vm simple; vm from image

* initial commit

* deploys locally

* updated deploy

* consolidated deploy and after_deploy into a single script; simplified ci process; added os_profile_linux_config

* added terraform show

* initial commit

* changed to allow http & https (like ARM tmplt)

* changed host_name & host_name variable desc

* added az cli check

* on this branch, only build test_dir; master will aggregate all the examples

* deploys locally; need to fix deploy scripts

* chmod for deploy.ci.sh

* corrected deploy.ci.sh

* added resource group var to travis.yml

* merge master

* edited travis.yml

* fixed error on travis.yml

* added new constructs/naming for deploy scripts, etc.

* took rg variable out; added destroy resources to travis.yml

* cleaned up commented out lines in deploy.ci.sh

* suppress az login output

* suppress az login output

* forgot about line breaks

* breaking build as an example

* fixing broken build example

* merge of CI config

* prep for PR

* cleanup for PR

* fixed grammar in readme

* prep for PR

* took out armviz button and minor README changes

* changed host_name

* fixed merge conflicts

* changed host_name variable

* updating Hashicorp's changes to merged simple linux branch

* updating files to merge w/master and prep for Hashicorp pr

* Revert "updating files to merge w/master and prep for Hashicorp pr"

This reverts commit b850cd5d2a858eff073fc5a1097a6813d0f8b362.

* Revert "updating Hashicorp's changes to merged simple linux branch"

This reverts commit dbaf8d14a9cdfcef0281919671357f6171ebd4e6.

* removing vm from user image example from this branch

* removed old branch

* added depends_on resource group for workaround to bug

* resolving conflicts

* restoring previous examples/resolving merge conflict

* resolving conflict

* updated .travis.yml

* azure-2-vms-loadbalancer-lbrules (#13)

* initial commit

* need to change lb_rule & nic

* deploys locally

* updated README

* updated travis and deploy scripts for Hari's repo

* renamed deploy script

* clean up

* prep for PR

* updated readme

* fixing conflict in .travis.yml

* add CI build tag

* updated descriptions

* azure-traffic-manager-vm (#17)

adding example of azure traffic manager

* azure-vnet-to-vnet-peering (#20)

adding vnet-to-vnet-peering example

* returning .travis.yml to correct state

* azure-search-create (#21)

adding azure-search-create example

* merging Hashicorp master into this branch

* added graph to README

* removing unmerged examples
apparentlymart pushed a commit that referenced this issue Jul 22, 2017
* initial commit - 101-vm-from-user-image

* changed branch name

* not deploying - storage problems

* provisions vm but image not properly prepared

* storage not correct

* provisions properly

* changed main.tf to azuredeploy.tf

* added tfvars and info for README

* tfvars ignored and corrected file ext

* added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing

* deploy.sh to be executable

* executable deploy files

* added CI files; changed vars

* prep for PR

* removal of old folder

* prep for PR

* wrong args for travis

* more PR prep

* updated README

* commented out variables in terraform.tfvars

* Topic 101 vm from user image (#2)

* initial commit - 101-vm-from-user-image
* added tfvars and info for README
* added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing
* prep for PR

* added new template

* oops, left off master

* prep for PR

* correct repository for destination

* renamed scripts to be more intuitive; added check for docker

* merge vm simple; vm from image

* initial commit

* deploys locally

* updated deploy

* consolidated deploy and after_deploy into a single script; simplified ci process; added os_profile_linux_config

* added terraform show

* changed to allow http & https (like ARM tmplt)

* changed host_name & host_name variable desc

* added az cli check

* on this branch, only build test_dir; master will aggregate all the examples

* merge master

* added new constructs/naming for deploy scripts, etc.

* suppress az login output

* suppress az login output

* forgot about line breaks

* breaking build as an example

* fixing broken build example

* merge of CI config

* fixed grammar in readme

* prep for PR

* took out armviz button and minor README changes

* changed host_name

* fixed merge conflicts

* changed host_name variable

* updating Hashicorp's changes to merged simple linux branch

* updating files to merge w/master and prep for Hashicorp pr

* Revert "updating files to merge w/master and prep for Hashicorp pr"

This reverts commit b850cd5d2a858eff073fc5a1097a6813d0f8b362.

* Revert "updating Hashicorp's changes to merged simple linux branch"

This reverts commit dbaf8d14a9cdfcef0281919671357f6171ebd4e6.

* removing vm from user image example from this branch

* removed old branch

* initial commit - azure search create

* removing graph from README

* azure-2-vms-loadbalancer-lbrules (#13)

* initial commit

* need to change lb_rule & nic

* deploys locally

* updated README

* updated travis and deploy scripts for Hari's repo

* renamed deploy script

* clean up

* prep for PR

* updated readme

* fixing conflict in .travis.yml

* add CI build tag

* updated .travis.yml to have the correct branch; validated resource with azure cli

* deleted commented out item in deploy script

* azure-traffic-manager-vm (#17)

adding example of azure traffic manager

* azure-vnet-to-vnet-peering (#20)

adding vnet-to-vnet-peering example

* returning .travis.yml to correct state

* correcting .travis.yml

* merging with master

* merging Hashicorp master into this branch

* removing unmerged example

* changed README
@rcarun rcarun added this to the M1 milestone Oct 11, 2017
@tombuildsstuff
Copy link
Contributor

Fixed in #426

@ghost
Copy link

ghost commented Apr 1, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants