Arpit Jasapara
Arpit Jasapara
@nkvuong Ideally we want to set some TF attribute like `computed` or `suppress_diff` so that it wouldn't delete existing tags.
@nkvuong one of the other issues with MLflow models is that there are many system-generated tags. This makes me think we should add a `tf:"computed"` attribute to its definition, but...
@nkvuong your branch looks fine but one thing I noticed was shouldn't we merge in the new tags as well (and `dedup` as necessary)?
@nkvuong oh I see the issue. I don't think that API call accepts tags in the request: https://mlflow.org/docs/latest/rest-api.html#update-registeredmodel. AFAIK this is the only way to set a model tag: https://mlflow.org/docs/latest/rest-api.html#set-registered-model-tag
@nkvuong also even if the update API could receive tags, I just realized that users basically would not be able to delete tags from their Terraform configuration right? Since we...
@nfx yeah currently updating tags via Terraform is broken. Model tags can only be created upon creation, but cannot be updated.
Create a `mlflow_model_serving` resource with expected behavior as follows: Input Vars - `registered_model_name` or `experiment_id` Behavior - Resource creation enables serving on the model via POST request at `/mlflow/endpoints/enable` -...
@nfx yeah the resource still fails without admin permissions
Hi @sbarzowski is there any update on this?
> > Also, it's not clear how to handle options inside the `config.served_entities.external_model.config` - technically we should have one entry per different model type, similar to the task types in...