API should validate uploaded asset metadata about mlx

[ckadner]

I will need to take a look at this. In a broader sense, the API should validate the uploaded template and reject malformed YAML or invalid metadata formats.

[ckadner]

Thanks to @drewbutlerbb4 for fixing part of the issue in PR #104

I still need to add some code in the MLX API to validate the YAML structure and return a meaningful error message if something is not right.

Currently we show an error like this, if the YAML is missing required elements:

Upload Error: Please Retry - "TypeError: 'NoneType' object is not iterable"

[ckadner]

@romeokienzler -- the component you attached here does not cause any errors for me (anymore?). Can you try again on the latest code base?

[drewbutlerbb4]

@ckadner #104 fixed the error that @romeokienzler was seeing for the UI.

When uploading a (potentially defunct) component.yaml breaks UI.

I'm not sure if this component.yaml is invalid? If it is, it passes through the API instead of returning an error to the UI. I think that was @romeokienzler initial point.

[ckadner]

I did not see any errors in the API when I tried it. I had manufactured an error by altering the "inputs" field, to catch/fix the Upload Error: 'NoneType' object is not iterable e4145b2

[ckadner]

@ckadner #104 fixed the error that @romeokienzler was seeing for the UI.

When uploading a (potentially defunct) component.yaml breaks UI.

@drewbutlerbb4 -- what error did you see there then?

[romeokienzler]

out.mp4

I get the same behavior (see above) on version/commit e4145b2

[drewbutlerbb4]

@romeokienzler what is your setup here? Docker-compose? I can't reproduce this using the nightly build image with docker-compose setup

[ckadner]

I get the same behavior (see above) on version/commit e4145b2

@romeokienzler -- in your video, I see the MLX UI still says:

"Enter a name and upload a .tgz or .tar.gz file to create a new component".

That was before Andrew's fix in PR #104 (bb69575). Now it should read:

"Enter a name and upload a .tgz, .tar.gz, .yaml or .yml file to register a new component".

Could you do a docker compose pull to get the latest nightly images to be pulled into your Docker Compose stack?

[romeokienzler]

@ckadner @drewbutlerbb4 sorry for the hassle - I did a "git pull" but no "docker-compose pull" - everything is working fine now, thanks a lot!