Resource processor - Re-instate az login and az acr login commands pr…

…ior to porter explain command to fix microsoft#3950 (microsoft#3951)

* Update commands.py

Resource processor - Re-instate az login and az acr login commands prior to porter explain command to fix microsoft#3950

* commands.py - fix linter whitespace error - W293 blank line contains whitespace

* Update resource processor version and changelog

CHANGELOG.md

@@ -15,6 +15,7 @@ BUG FIXES:
 * Remove TLS1.0/1.1 support from Application Gateway ([#3914](https://github.com/microsoft/AzureTRE/issues/3914))
 * GitHub Actions version updates. ([#3847](https://github.com/microsoft/AzureTRE/issues/3847))
 * Add workaround to avoid name clashes for storage accounts([#3863](https://github.com/microsoft/AzureTRE/pull/3858))
+* Resource processor fails to deploy first workspace on fresh TRE deployment ([#3950](https://github.com/microsoft/AzureTRE/issues/3950))
 * Dependency and Vulnerability updates
 
 COMPONENTS:

resource_processor/_version.py

@@ -1 +1 @@
-__version__ = "0.8.5"
+__version__ = "0.8.6"

resource_processor/resources/commands.py

@@ -104,7 +104,9 @@ async def build_porter_command_for_outputs(msg_body):
 
 
 async def get_porter_parameter_keys(config, msg_body):
-    command = [f"porter explain --reference {config['registry_server']}/{msg_body['name']}:v{msg_body['version']} --output json"]
+    command = [f"{azure_login_command(config)} && \
+        {azure_acr_login_command(config)} && \
+        porter explain --reference {config['registry_server']}/{msg_body['name']}:v{msg_body['version']} --output json"]
 
     proc = await asyncio.create_subprocess_shell(
         ''.join(command),