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

Use opensearchPy instead of direct opensearch API requests #72

Merged
merged 8 commits into from
Sep 8, 2023

Conversation

alexdunnjpl
Copy link
Contributor

🗒️ Summary

Supporting work to facilitate work on #70
Replaces raw-request OpenSearch interaction with opensearch-py package

⚙️ Test Data and/or Report

Tests pass. Manually tested as a sanity check for db reads/writes against I&T docker registry

♻️ Related Issues

related to #70

src/pds/registrysweepers/utils/db/__init__.py Outdated Show resolved Hide resolved
src/pds/registrysweepers/utils/db/__init__.py Outdated Show resolved Hide resolved
@alexdunnjpl alexdunnjpl merged commit 6f7b796 into main Sep 8, 2023
@alexdunnjpl alexdunnjpl deleted the 70pre-opensearch-py-rewrite branch September 8, 2023 20:55
@tloubrieu-jpl tloubrieu-jpl changed the title 70pre opensearch py rewrite Use opensearchPy instead of direct opensearch API requests Oct 6, 2023
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