DARPA-ASKEM / terarium

https://app.terarium.ai

Apache License 2.0

12 stars 2 forks source link

improved mira controller error messages #3930

Closed dgauldie closed 3 weeks ago

dgauldie commented 3 weeks ago

Description

Include a summary of the changes and the related issue.
Include relevant motivation and context.

Resolves #(issue)

dgauldie commented 3 weeks ago

A few log.errors were downgraded to log.warn, but still throw an exception and are a failure case. We may want these to be errors? but over all good.

I was kinda going with 4xx and 503 errors are warnings... as they are still failures of the controllers but associated with problems of the client and of other services. 500 errors are errors.