-
Notifications
You must be signed in to change notification settings - Fork 15
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
Pulumi: Add smoke tests #1614
Pulumi: Add smoke tests #1614
Conversation
…it is possible to run
63c5261
to
b039cd7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
The Azure API doesn't have a more direct way to write a file to a disk mounted to a VM?
I suppose we use the API here as the machines will not generally be accessible by SSH?
No, weirdly not. Maybe they didn't want to write something that would work cross-OS?
That's right - these VMs have a very limited allowed set of connections in and out which only cover internal DSH connections and platform level stuff (like this) which bypasses the NSG rules. |
53e53ab
into
alan-turing-institute:python-migration
✅ Checklist
Enable foobar integration
rather than515 foobar
).develop
.'[WIP]'
to the title if needed (if you're not yet ready to merge)../tests/AutoFormat_Powershell.ps1 -TargetPath <path to file or directory>
for Powershell).bats
available in the Ubuntu repos which has fewer features)FileUpload
provider to allow files to be uploaded to VMs that cannot access e.g.raw.githubusercontent.com
🌂 Related issues
Closes #1603
🔬 Tests
Tested on a local deployment