alan-turing-institute / data-safe-haven

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

Unchangable Pulumi workspace configuration #2237

Closed JimMadge closed 1 month ago

JimMadge commented 1 month ago

:white_check_mark: Checklist

:vertical_traffic_light: Depends on

2241

:arrow_heading_up: Summary

Protects against changing values which we have declared shouldn't be updated in the Pulumi workspace.

:closed_umbrella: Related issues

closes #2199

:microscope: Tests

❯ hatch run dsh sre deploy hojo
You are logged into the Azure CLI as:
        user: Jim Madge (…)
        tenant: turing.ac.uk (…)
Are these details correct? [y/n] (y): y
You are logged into the Microsoft Graph API as:
        user: aad.admin.jim.madge@green.develop.turingsafehaven.ac.uk
(…)
        tenant: green.develop.turingsafehaven.ac.uk (…)
Are these details correct? [y/n] (y): y
SRE will be deployed to subscription 'Data Safe Haven Development 2'
('…')
SRE will be registered in SHM 'daimyo.develop.turingsafehaven.ac.uk'
SHM subscription '…'
Loaded stack shm-daimyo-sre-hojo.
Unchangeable configuration variable 'azure-native:subscriptionId' not consistent, your configuration: '…', Pulumi workspace: '…'.
Applying Pulumi configuration options failed..
Pulumi deployment failed.
Could not deploy Secure Research Environment 'hojo'.
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
Project Total  

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