Skip to content

Update tfadd for a fix to remove zero/default value attribute only for optional attribute#627

Merged
magodo merged 1 commit intoAzure:mainfrom
magodo:tfadd_remove_optional_zero_attr
May 29, 2025
Merged

Update tfadd for a fix to remove zero/default value attribute only for optional attribute#627
magodo merged 1 commit intoAzure:mainfrom
magodo:tfadd_remove_optional_zero_attr

Conversation

@magodo
Copy link
Copy Markdown
Collaborator

@magodo magodo commented May 29, 2025

Update tfadd to involve the fix: magodo/tfadd@d62ece5

Fix #626

@magodo magodo added dependency bug Something isn't working labels May 29, 2025
@gerrytan gerrytan assigned gerrytan and unassigned gerrytan May 29, 2025
@gerrytan gerrytan self-requested a review May 29, 2025 00:13
@magodo magodo merged commit bc79cc4 into Azure:main May 29, 2025
7 checks passed
@brijendrakainth1
Copy link
Copy Markdown

today i have generated aztfexport code again, i am still getting the same error.

C:\Users\user\OneDrive - \Daikin\AZTFexport-Backup-project\Output files\aztfexport-backup-testrg-20250603\TestBackupRG> terraform plan

│ Error: Missing required argument

│ on main.tf line 32, in resource "azurerm_virtual_machine_data_disk_attachment" "res-1":
│ 32: resource "azurerm_virtual_machine_data_disk_attachment" "res-1" {

│ The argument "lun" is required, but no definition was found.

@brijendrakainth1
Copy link
Copy Markdown

My bad, i was using older version of aztfexport . issue has been fixed after using newer version

Thanks

gerrytan pushed a commit to gerrytan/aztfexport that referenced this pull request Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

azurerm_virtual_machine_data_disk_attachment is not getting required parameter in aztfexport

3 participants