search

CompositionalIT / farmer

Repeatable Azure deployments with ARM templates - made easy!
https://compositionalit.github.io/farmer
MIT License
527 stars 159 forks source link

Compute.VirtualMachine resource does not respect DiagnosticsStorageAccount ManagedResource/UnmanagedResource #959

Closed ericsoler1 closed 1 year ago

ericsoler1 commented 2 years ago

This will always add depends on clause regardless of if the storage account resource is managed or unmanaged. https://github.com/CompositionalIT/farmer/blob/c0f861b7487d1e4dc3b4bc080edb9f9e050858b5/src/Farmer/Arm/Compute.fs#L96

This needs to be a LinkedResource https://github.com/CompositionalIT/farmer/blob/c0f861b7487d1e4dc3b4bc080edb9f9e050858b5/src/Farmer/Arm/Compute.fs#L72

ninjarobot commented 1 year ago

@ericsoler1 the fix is available in the 1.7.19 release now available on nuget.