Skip to content

chore: search-index-consumer - handle version conflict race condition [CHI-3702]#996

Merged
gpaoloni merged 2 commits intomasterfrom
gian_CHI-3702
Feb 17, 2026
Merged

chore: search-index-consumer - handle version conflict race condition [CHI-3702]#996
gpaoloni merged 2 commits intomasterfrom
gian_CHI-3702

Conversation

@gpaoloni
Copy link
Collaborator

@gpaoloni gpaoloni commented Feb 10, 2026

Description

This PR

  • Refactors ElasticSearch client: index and update operations return a result type wrapping the actual response, consolidating with the changes introduced in chore: improve error handling around ES deletes [CHI-3516] #984.
  • Refactors search index consumer lambda: Introduces retries for index and update operations for the cases where there's a race condition, as described in the linked ticket below.

Checklist

Verification steps

AFTER YOU MERGE

  1. Cut a release tag using the GitHub workflow. Wait for it to complete and notify in the #aselo-deploys Slack channel.
  2. Comment on the ticket with the release tag version AND any additional instructions required to configure an environment to test the changes.
  3. Only then move the ticket into the QA column in JIRA

You are responsible for ensuring the above steps are completed. If you move a ticket into QA without advising what version to test, the QA team will assume the latest tag has the changes. If it does not, the following confusion is on you! :-P

@gpaoloni gpaoloni requested a review from stephenhand February 10, 2026 19:37
Copy link
Collaborator

@stephenhand stephenhand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gpaoloni gpaoloni merged commit 58b8755 into master Feb 17, 2026
18 checks passed
@gpaoloni gpaoloni deleted the gian_CHI-3702 branch February 17, 2026 23:11
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