Support for deployment in Azure Government Cloud (Leonardo)

[bennettn4]

Part of large effort to add support for services to run in Azure Government cloud.

Two parts:

1. Support for government cloud curls

• Added environment variable for AZURE_ENVIRONMENT
• Updated azure-vm-init script to support gov suffix

2. Fixes for running Leo in Azure. The user pet token is unavailable in Azure so calls which used it for Leo->WSM were replaced with the Leo service account token, and appropriate permissions to the workspace were given to the Leo service account.

https://broadworkbench.atlassian.net/browse/TOAZ-372
See related pull requests here:
workbench-libs sam bpm wsm cromwell terra-helmfile

[codecov]

Codecov Report

Attention: Patch coverage is 70.27027% with 22 lines in your changes missing coverage. Please review.

Project coverage is 74.56%. Comparing base (f0dd5f9) to head (1f5480c).

Files with missing lines	Patch %	Lines
...sde/workbench/leonardo/monitor/MonitorAtBoot.scala	50.00%	8 Missing ⚠️
...bench/leonardo/config/AzureHostingModeConfig.scala	80.00%	3 Missing ⚠️
...workbench/leonardo/monitor/LeoMetricsMonitor.scala	50.00%	3 Missing ⚠️
...de/workbench/leonardo/app/CromwellAppInstall.scala	50.00%	2 Missing ⚠️
...kbench/leonardo/app/CromwellRunnerAppInstall.scala	60.00%	2 Missing ⚠️
...te/dsde/workbench/leonardo/app/WdsAppInstall.scala	60.00%	2 Missing ⚠️
...e/workbench/leonardo/app/WorkflowsAppInstall.scala	50.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #4813      +/-   ##
===========================================
- Coverage    74.59%   74.56%   -0.04%     
===========================================
  Files          166      166              
  Lines        14609    14630      +21     
  Branches      1101     1218     +117     
===========================================
+ Hits         10898    10909      +11     
- Misses        3711     3721      +10     

Files with missing lines	Coverage Δ
...e/workbench/leonardo/app/HailBatchAppInstall.scala	62.50% <ø> (ø)
...rkbench/leonardo/http/AppDependenciesBuilder.scala	97.90% <100.00%> (+0.02%)	⬆️
...ch/leonardo/http/service/LeoAppServiceInterp.scala	87.41% <100.00%> (+0.07%)	⬆️
.../dsde/workbench/leonardo/util/AKSInterpreter.scala	88.66% <100.00%> (-0.13%)	⬇️
...e/workbench/leonardo/util/AzurePubsubHandler.scala	84.56% <100.00%> (-0.11%)	⬇️
...workbench/leonardo/util/BuildHelmChartValues.scala	97.18% <ø> (ø)
...de/workbench/leonardo/app/CromwellAppInstall.scala	67.74% <50.00%> (-1.23%)	⬇️
...kbench/leonardo/app/CromwellRunnerAppInstall.scala	84.09% <60.00%> (-1.63%)	⬇️
...te/dsde/workbench/leonardo/app/WdsAppInstall.scala	78.94% <60.00%> (-3.41%)	⬇️
...e/workbench/leonardo/app/WorkflowsAppInstall.scala	66.66% <50.00%> (-1.34%)	⬇️
... and 3 more

---

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 f0dd5f9...1f5480c. Read the comment docs.

[LizBaldo]

Were you able to test this? The Azure integration tests have been removed a few months ago unfortunately 🤔

This looks good, but I have a few comments around preserving the pre-existing behavior for GCP as much as possible.

[jsaun]

Were you able to test this? The Azure integration tests have been removed a few months ago unfortunately 🤔

Yeah, we've been running this branch in our dev environment for a while, it will need some final validation as I don't think we've tested the most recent commits though.

This looks good, but I have a few comments around preserving the pre-existing behavior for GCP as much as possible.

Yep, makes sense. I was trying to avoid the branching logic if possible to keep it consistent/simpler but I will try to update shortly for those cases you pointed out.

[LizBaldo]

This should work for now, feel free to merge, but a lot of this functionality might soon be removed or significantly paired down as we discussed.