arvindshmicrosoft commented on a change in pull request #415:
URL: https://github.com/apache/fluo-muchos/pull/415#discussion_r761225336



##########
File path: lib/requirements.txt
##########
@@ -2,3 +2,4 @@ flake8==3.8.0
 ansible-core==2.11.3
 ansible-lint==5.1.2
 nose==1.3.7
+azure==4.0.0

Review comment:
       @zachradtka, it would also be good to know a bit more about the 
environment where the above issue was detected - the GitHub Actions runner 
seemed to not require this installed, but that may very well be because other 
installations (like the Azure CLI) might have provided that package.
   
   Going out on a proverbial limb - originally, we used to install the 
`ansible[azure]` package, which used to install the required parts of the 
`azure` package implicitly. Earlier this year when we switched to more current 
Ansible (core) a possible gap might have been created, which (I'm hypothesizing 
here) may not come up in the CI (or in local environments) where AZ CLI was 
also installed. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to