-
Notifications
You must be signed in to change notification settings - Fork 522
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: PatrickLang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold |
fe5e4fa
to
f5e4770
Compare
Codecov Report
@@ Coverage Diff @@
## master #1651 +/- ##
==========================================
- Coverage 76.48% 76.44% -0.05%
==========================================
Files 129 129
Lines 19033 19004 -29
==========================================
- Hits 14558 14527 -31
- Misses 3682 3690 +8
+ Partials 793 787 -6 |
This is ready for review. I will squash and update commit message to feat: after that. I also need to do a few adhoc perf tests so I can clarify what the improvement is in deployment time. |
Manual tests looking good. Big improvement in 1st launch time of iis Clean deployment, nothing running
OS disks were all left at 30Gb which is default for that image 2 Standard_D2s_v3 with ephemeral 2 Standard_D2s_v3 without ephemeral |
Nice, container local storage improved from ~100 IOPs to ~3300 IOPs Results at https://github.com/PatrickLang/diskspd-containers |
7d1c2db
to
205feab
Compare
/remove hold |
/hold cancel |
lgtm, curious if we should merge this with more documentation language |
alright, will get more docs |
example warning:
|
Although many operations are faster, the E2E test pass duration impact is still within stddev
This PR leaves ephemeral on for the Windows test passes so we can measure if it breaks. If it's too brittle, then we can disable it. |
/remove hold |
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.
/lgsm
and |
/lgtm |
@PatrickLang do we have a sample model or doc for this? Wanted to take it for a spin |
Reason for Change:
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/ephemeral-os-disks make deployment and runtime faster.
This is still a WIP for review & testing. Will squash and make commit messages conventional soon.
Issue Fixed:
Fixes #1287
Requirements:
Notes: