[AN-385] new sam disk permissions

[LizBaldo]

Jira ticket: https://broadworkbench.atlassian.net/browse/AN-385

Summary of changes

What

• This PR contains the changes from [CORE-211] Convert disk operations to new Sam permissions model #4822 as well as the necessary clean up to remove the includeDeleted query parameter now that our get disk endpoint won't be able to return deleted disks

Why

• Since there have been partial reverts previously, it was easier to build a new PR on top of the branch from [AN-385] Move listRuntimes over to new Sam permissions model - take 2 #4827. Note that the base for this PR is NOT develop, but revert-4823-mtalbott-revert-listRuntimes, so we should merge 4827 first.

Testing these changes

What to test

Who tested and where

• This change is covered by automated tests
  • NB: Rerun automation tests on this PR by commenting jenkins retest or jenkins multi-test.
• I validated this change
• Primary reviewer validated this change
• I validated this change in the dev environment

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.57%. Comparing base (7e8b90d) to head (7c58ad6).
Report is 1 commits behind head on revert-4823-mtalbott-revert-listRuntimes.

Additional details and impacted files

@@                             Coverage Diff                              @@
##           revert-4823-mtalbott-revert-listRuntimes    #4828      +/-   ##
============================================================================
- Coverage                                     74.62%   74.57%   -0.05%     
============================================================================
  Files                                           166      166              
  Lines                                         14685    14609      -76     
  Branches                                       1164     1101      -63     
============================================================================
- Hits                                          10959    10895      -64     
+ Misses                                         3726     3714      -12     

Files with missing lines	Coverage Δ
...ute/dsde/workbench/leonardo/dao/sam/SamUtils.scala	100.00% <100.00%> (ø)
...e/workbench/leonardo/db/DiskServiceDbQueries.scala	100.00% <100.00%> (ø)
...rkbench/leonardo/http/AppDependenciesBuilder.scala	97.87% <100.00%> (ø)
...bench/leonardo/http/AzureDependenciesBuilder.scala	97.14% <ø> (-0.16%)	⬇️
...rkbench/leonardo/http/GcpDependenciesBuilder.scala	73.14% <ø> (-0.31%)	⬇️
...ench/leonardo/http/service/DiskServiceInterp.scala	95.27% <100.00%> (+3.51%)	⬆️
...ch/leonardo/http/service/DiskV2ServiceInterp.scala	97.22% <100.00%> (+4.76%)	⬆️
...ch/leonardo/http/service/LeoAppServiceInterp.scala	87.34% <ø> (-0.04%)	⬇️
...workbench/leonardo/http/service/ProxyService.scala	73.55% <100.00%> (+0.12%)	⬆️
...rkbench/leonardo/http/service/RuntimeService.scala	100.00% <ø> (ø)
... and 2 more

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e8b90d...7c58ad6. Read the comment docs.

[lucymcnatt]

LGTM!