-
Notifications
You must be signed in to change notification settings - Fork 97
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
Add possibility to select osDisk SKU #231
Comments
Thanks, I have put this in backlog. by jieshe |
Any updates? Any help needed? (I don't know the codebase, but I could try if it's similar to eg. osDiskSize) by stefanscherer |
Stefan Scherer Feel free to create a PR. Some useful pages : https://docs.microsoft.com/en-us/azure/templates/microsoft.compute/2019-03-01/virtualmachines#OSDisk and https://docs.microsoft.com/en-us/azure/templates/microsoft.compute/2019-03-01/virtualmachines#ManagedDiskParameters. by jieshe |
Thanks, I searched through the codebase and I think this variable "storageAccountType": "Standard_LRS", in https://github.com/jenkinsci/azure-vm-agents-plugin/blob/dev/src/main/resources/customImageTemplateWithManagedDisk.json#L15 doesn't get changed. by stefanscherer |
I've also created |
The "storageAccountType" is changed dynamically at Line 517 in 4a0960e
|
Any updates on this, not really sure this is tied to storage account type. I believe storage account type is related to the disk storage type? On Azure if I navigate to the disk resource the storage types available are: on the storage account type dropdown I can only see Standard_LRS and Premium LRS Thanks in advance, great plugin overall by one_random_dev |
This appears to already be available in the 'Storage Account Type' field |
Ah right looking at the last comment 'StandardSSD_LRS' is missing, ultra disk requires data disk which this plugin doesn't support |
When using templates and managed disks, in Azure it is possible to define the osDisk type during creation (HDD, Standard_SSD, Premium_SSD).
It would great if one could define the type for each Azure Virtual Machine Template.
[JENKINS-55943] created by claasd
The text was updated successfully, but these errors were encountered: