[GitHub] [fluo-muchos] arvindshmicrosoft commented on a change in pull request #415: Added hadoop 2.10.1 support and azure bug fix

[GitBox <gi...@apache.org>]

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: notifications-unsubscribe@fluo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org