Skip to content

chore: Clean opensearch integration (M2-10948)#2085

Merged
sricharan-varanasi merged 3 commits into
developfrom
clean-opensearch-integration
Jul 6, 2026
Merged

chore: Clean opensearch integration (M2-10948)#2085
sricharan-varanasi merged 3 commits into
developfrom
clean-opensearch-integration

Conversation

@sricharan-varanasi

@sricharan-varanasi sricharan-varanasi commented Jul 2, 2026

Copy link
Copy Markdown
Contributor
  • Related documentation has been added / updated

📝 Description

Audit-log storage was migrated from OpenSearch to Postgres; this removes the leftover OpenSearch code, which is no longer referenced anywhere at runtime.

🔗 Jira Ticket M2-10948

Changes include:

  • Deleted the unused OpenSearchClient, OpenSearchSettings, AUDIT_LOG_MAPPING, and their tests
  • Removed the settings.opensearch field and the OPENSEARCH__* vars from .env.default
  • Dropped the opensearch-py dependency and regenerated uv.lock
  • Cleaned up stale OpenSearch references in audit docstrings and docs/audit-pipeline.md

✏️ Notes

No behavior change — the audit pipeline (RabbitMQ → Postgres) is untouched. grep -rni opensearch over the repo returns zero hits; audit and infrastructure test suites pass.

@sricharan-varanasi sricharan-varanasi added the Do not merge Pull request cannot be merged for some reason label Jul 2, 2026
@sricharan-varanasi sricharan-varanasi changed the title Clean opensearch integration chore: Clean opensearch integration (M2-10948) Jul 2, 2026
@sricharan-varanasi sricharan-varanasi removed the Do not merge Pull request cannot be merged for some reason label Jul 2, 2026

@divbzero divbzero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All changes look clean to me. Thank you @sricharan-varanasi!

@sricharan-varanasi sricharan-varanasi merged commit c1043ac into develop Jul 6, 2026
5 checks passed
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