virusseq / portal-ui

Canadian VirusSeq Data Portal
https://virusseq-dataportal.ca/
GNU Affero General Public License v3.0
8 stars 8 forks source link

Maestro fails to reindex the updates/changes in metadata done through Song API's #330

Closed ghost closed 1 year ago

ghost commented 1 year ago

Current Behavior: When metadata is changed for a record using Song API https://song.dev.cancogen.cancercollaboratory.org/swagger-ui.html#/Analysis/updateAnalysisUsingPUT Maestro fails to reindex the changes in Elastic Search.

Additional Information: Manually trying to index ES using Maestro API's also fails although the API sends a 200 response.

Reference dev Data: 2af9f060-9529-4422-b9f0-609529d422cf /AB_239807 58d4541e-e695-428b-9454-1ee695228b63 /AB_240250 f8037b35-a1a8-451b-837b-35a1a8c51b56 /BC_328819

You will see different data in Song Vs ES.

Expected Behavior: When metadata is changed for a record using Song API https://song.dev.cancogen.cancercollaboratory.org/swagger-ui.html#/Analysis/updateAnalysisUsingPUT Maestro should reindex the changes in Elastic Search.

ghost commented 1 year ago

created a ticket in Maestro Repo https://github.com/overture-stack/maestro/issues/252