Skip to content
New issue

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

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

Already on GitHub? # to your account

Virtual Machine should allow configuring boot diagnostics with "managed" storage account #1107

Closed
khaliddermoumi opened this issue Mar 8, 2022 · 2 comments · Fixed by #1365
Assignees
Labels
[cat] modules category: modules enhancement New feature or request good first issue Good for newcomers

Comments

@khaliddermoumi
Copy link
Contributor

Description

A VM can be created with boot diagnostics enabled. Diagnostics data is written to a storage account.
There is a feature to not specify a custom storage account, and instead use a storage account managed by Azure. This is probably the better option than using your own storage accounts because of the limitations (your storage account needs an open firewall, and many options for it are not supported).
The current module does not allow this configuration, instead you must always specify a custom storage account.
It would make much sense to add this feature, as it is probably the best way to activate boot diagnostics.
Inside an ARM template, you would specify the feature like this:

"diagnosticsProfile": {
   "bootDiagnostics": {
      "enabled": true
   }
}

So you actually just need to enable the feature, and not specify an existing storage account.

@khaliddermoumi khaliddermoumi added the enhancement New feature or request label Mar 8, 2022
@AlexanderSehr AlexanderSehr added [cat] modules category: modules good first issue Good for newcomers labels Mar 8, 2022
@AlexanderSehr
Copy link
Contributor

@khaliddermoumi good idea & good catch. We'll move it into today. If you happen to have implemented that yourself and would like to share, Pull Requests are always welcome :)

@khaliddermoumi
Copy link
Contributor Author

Thanks for asking! I haven't implemented this yet, as I am currently extremely busy with work etc. But I may be able to manage to do this in 1-2 weeks.
So if you need this fix right now, I can't help, but maybe later?

@lsnoddy lsnoddy self-assigned this May 4, 2022
@rahalan rahalan moved this to Done in Backlog Dec 11, 2022
@rahalan rahalan added this to Backlog Dec 11, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
[cat] modules category: modules enhancement New feature or request good first issue Good for newcomers
Projects
Status: Done
3 participants