You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@fluo.apache.org by GitBox <gi...@apache.org> on 2021/06/07 15:29:11 UTC

[GitHub] [fluo-muchos] karthick-rn commented on pull request #395: Add validation checks for Azure deployments

karthick-rn commented on pull request #395:
URL: https://github.com/apache/fluo-muchos/pull/395#issuecomment-856036592


   @arvindshmicrosoft Hi, Thanks for taking the time to review this PR and also for helping me out to refine this work further and cover few more checks. Based on our last discussion I have applied the changes in this [1c5c08f](https://github.com/apache/fluo-muchos/pull/395/commits/1c5c08f3d1f2367e094f8e9ec9ea02c759658408). 
   I have tested these changes against the following scenarios successfully
   - Standard Lsv2 NVMe with no data disks
   - Multiple VMSS with HA & without HA
   - ADLS Gen2 
   - With & without Azure proxy host
   
   I have also included a _TODO_ in `azure/test_config.py` for future work on adding test cases. 
   
   
   


-- 
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.

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