Scott Euser commented on merge request !14 at project / ai_vdb_provider_milvus

$this->getLogger('ai_vdb_provider_milvus')->warning($this->t('Error: @code, Message: @message', [
'@code' => $data['code'] ?? 'N/A',
'@message' => $data['message'] ?? 'Unknown error'
]));

Maybe something like this, downgrading to warning so we aren't introducing errors we currently don't have BUT:

I think the N/A unknown need to be translatable somehow too

Probably we only want to log something that has an error code rather than everything. E.g. not 200 codes? Haven't checked what the default codes are, needs checking milvus docs maybe