search

zenml-io / mlstacks

A series of Terraform based recipes to provision popular MLOps stacks on the cloud.
https://mlstacks.zenml.io/
Apache License 2.0
247 stars 32 forks source link

[BUGFIX] Fix mlflow reference when disabled in Vertex recipe #43

Closed wjayesh closed 1 year ago

wjayesh commented 1 year ago

Fixes #41

htahir1 commented 1 year ago

Can we test these sorts of things?

wjayesh commented 1 year ago

@htahir1 this could've been caught by our existing tests if mlflow was disabled in the variables file. Testing it for all individual components, I imagine, would be tricky going forward because we need a way to run our tests for both enable/disable and for all the variables. I'll create a ticket