Add azure machine learning module

gruntwork-io / terratest

Terratest is a Go library that makes it easier to write automated tests for your infrastructure code.

https://terratest.gruntwork.io/

Apache License 2.0

7.47k stars 1.32k forks source link

Add azure machine learning module #1372

Open hattan opened 10 months ago

hattan commented 10 months ago

Description

Added support for Azure Machine Learning Workspace and Azure Machine Learning Workspace Compute. Fixes #1371.

TODOs

Read the Gruntwork contribution guidelines.

[x] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[X] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes

Added support for Azure Machine Learning Workspace and Azure Machine Learning Workspace Compute.

HadwaAbdelhalem commented 10 months ago

[Microsoft CI Bot] TL;DR; failure :facepalm:

You can check the status of the CI Pipeline logs here ; https://github.com/HadwaAbdelhalem/terratest/actions/runs/6791271804

hattan commented 10 months ago

[Microsoft CI Bot] TL;DR; failure 🤦

You can check the status of the CI Pipeline logs here ; https://github.com/HadwaAbdelhalem/terratest/actions/runs/6791271804

The failing tests are expected and due to quota limits on the Azure account used for testing. However, the new module introduced here (Azure Machine Learning) is passing.

HadwaAbdelhalem commented 10 months ago

LGTM! confirmed the failed tests are due to quota restrictions on the azure test subscription. @denis256 can you please review it.

denis256 commented 10 months ago

@HadwaAbdelhalem Hi, looks like imports should be fixed:

goimports................................................................Failed
- hook id: goimports
- files were modified by this hook

modules/azure/machinelearning.go