Skip to content
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

Improve VM SKU documentation and errors #2111

Conversation

jemrobinson
Copy link
Member

@jemrobinson jemrobinson commented Aug 8, 2024

✅ Checklist

  • You have given your pull request a meaningful title (e.g. Enable foobar integration rather than 515 foobar).
  • You are targeting the appropriate branch. If you're not certain which one this is, it should be develop.
  • Your branch is up-to-date with the target branch (it probably was when you started, but it may have changed since then).

🚦 Depends on

n/a

⤴️ Summary

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

  • Add note to documentation explaining that premium-SSD-compatible VMs must be used
  • Improve logging of Pulumi errors so that this also appears in the logs:
 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'.

🌂 Related issues

Closes #2067

🔬 Tests

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

@jemrobinson jemrobinson requested review from a team as code owners August 8, 2024 14:57
@jemrobinson jemrobinson requested review from craddm and JimMadge August 8, 2024 14:57
@jemrobinson jemrobinson changed the base branch from develop to release-v5.0.0 August 8, 2024 15:01
Copy link

github-actions bot commented Aug 8, 2024

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

@jemrobinson jemrobinson force-pushed the 2067-better-vm-sku-documentation branch from c4bc1f9 to 794e9b3 Compare August 8, 2024 15:09
@jemrobinson jemrobinson mentioned this pull request Aug 8, 2024
3 tasks
@jemrobinson jemrobinson force-pushed the 2067-better-vm-sku-documentation branch 2 times, most recently from 7b7f491 to ca0222e Compare August 8, 2024 15:47
@jemrobinson jemrobinson force-pushed the 2067-better-vm-sku-documentation branch from ca0222e to 794e9b3 Compare August 9, 2024 09:54
@jemrobinson jemrobinson force-pushed the 2067-better-vm-sku-documentation branch from 794e9b3 to 24e1602 Compare August 9, 2024 09:57
@jemrobinson jemrobinson merged commit 5826b25 into alan-turing-institute:release-v5.0.0 Aug 9, 2024
11 checks passed
@jemrobinson jemrobinson deleted the 2067-better-vm-sku-documentation branch January 30, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Premium disks not supported by all VM SKUs
2 participants