Skip to content

chore: remove unused Elastic search from CI#3755

Merged
arslanashraf7 merged 1 commit intomasterfrom
arslan/remove-elastic-search
Jan 20, 2026
Merged

chore: remove unused Elastic search from CI#3755
arslanashraf7 merged 1 commit intomasterfrom
arslan/remove-elastic-search

Conversation

@arslanashraf7
Copy link
Contributor

What are the relevant tickets?

None, Initiated from #3750 (comment)

Description (What does it do?)

  • In xPRO CI, we have been maintaining the ElasticSearch step for a long time. The application itself doesn't use Elastic Search. The image and the step are stale and we should now remove them.

Screenshots (if appropriate):

  • Desktop screenshots
  • Mobile width screenshots

How can this be tested?

  • The application should work fine without any issues
  • The CI should pass

Additional Context

@arslanashraf7 arslanashraf7 marked this pull request as ready for review January 16, 2026 14:04
@arslanashraf7 arslanashraf7 changed the title chore: remove elastic search from CI chore: remove unused Elastic search from CI Jan 16, 2026
@asadali145 asadali145 self-assigned this Jan 16, 2026
Copy link
Contributor

@asadali145 asadali145 left a comment

Choose a reason for hiding this comment

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

LGTM! I see poetry.lock has elasticsearch. Do we need to remove it?

@arslanashraf7
Copy link
Contributor Author

LGTM! I see poetry.lock has elasticsearch. Do we need to remove it?

I don't think we need to remove it, that is part of the existing packages.

@arslanashraf7 arslanashraf7 force-pushed the arslan/remove-elastic-search branch from 10086b8 to dfac744 Compare January 20, 2026 12:25
@arslanashraf7 arslanashraf7 merged commit 2e80dba into master Jan 20, 2026
8 checks passed
@arslanashraf7 arslanashraf7 deleted the arslan/remove-elastic-search branch January 20, 2026 12:55
@odlbot odlbot mentioned this pull request Jan 20, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants