alan-turing-institute / data-safe-haven

https://data-safe-haven.readthedocs.io
BSD 3-Clause "New" or "Revised" License
57 stars 15 forks source link

Improve VM SKU documentation and errors #2111

Closed jemrobinson closed 1 month ago

jemrobinson commented 1 month ago

:white_check_mark: Checklist

:vertical_traffic_light: Depends on

n/a

:arrow_heading_up: Summary

IMPORTANT: I am confident that merging this will not affect the findings of the ongoing penetration test

 stderr:
Pulumi error: +  azure-native:compute:VirtualMachine sre_workspaces_vm_workspace_02 creating (3s)  Code="InvalidParameter" Message="Requested operation cannot be performed because the VM size Standard_D8_v5 does not support the storage account
type Premium_LRS of disk 'shm-pink-sre-coral-vm-workspace-02-osdisk'. Consider updating the VM to a size that supports Premium storage." Target="osDisk.managedDisk.storageAccountType"
Pulumi error: +  azure-native:compute:VirtualMachine sre_workspaces_vm_workspace_02 **creating failed**  Code="InvalidParameter" Message="Requested operation cannot be performed because the VM size Standard_D8_v5 does not support the storage
account type Premium_LRS of disk 'shm-pink-sre-coral-vm-workspace-02-osdisk'. Consider updating the VM to a size that supports Premium storage." Target="osDisk.managedDisk.storageAccountType"
Pulumi error:    pulumi:pulumi:Stack data-safe-haven-shm-pink-sre-coral running  update failed
Pulumi error:  Code="InvalidParameter" Message="Requested operation cannot be performed because the VM size Standard_D8_v5 does not support the storage account type Premium_LRS of disk 'shm-pink-sre-coral-vm-workspace-02-osdisk'. Consider
updating the VM to a size that supports Premium storage." Target="osDisk.managedDisk.storageAccountType"
Pulumi error:  update failed
Pulumi update failed.
Pulumi deployment failed.
Could not deploy Secure Research Environment 'coral'.

:closed_umbrella: Related issues

Closes #2067

:microscope: Tests

Tested by trying to add an incompatible VM to an existing SRE

github-actions[bot] commented 1 month ago

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  data_safe_haven/infrastructure
  project_manager.py 277
Project Total  

This report was generated by python-coverage-comment-action