Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Delete bug #489

Merged
merged 24 commits into from
Dec 18, 2024
Merged

fix: Delete bug #489

merged 24 commits into from
Dec 18, 2024

Conversation

wphyojpl
Copy link
Collaborator

@wphyojpl wphyojpl commented Dec 17, 2024

Granule_to_es is not checking the event type, and always ingesting records to UDS-ES.

So, when we delete the first time, the "delete" event from Cumulus is sent to that lambda which creates a new record just after UDS delete endpoint is deleting the record from ES.

When we run it again, there is nothing in Cumulus. So, there is no event, and record from ES is finally deleted.

@wphyojpl wphyojpl requested a review from ngachung December 18, 2024 13:58
@ngachung ngachung merged commit 40b9b97 into develop Dec 18, 2024
1 check passed
@ngachung ngachung mentioned this pull request Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants