nv-ingest
nv-ingest copied to clipboard
add IngestService to context manager
Description
Make RedisIngestService lifecycle management more explicit. It can help with redis warm up connections and gracefully shutdown.
Feature request: https://github.com/NVIDIA/nv-ingest/issues/969
Checklist
- [ ] I am familiar with the Contributing Guidelines.
- [ ] New or existing tests cover these changes.
- [ ] The documentation is up to date with these changes.
- [ ] If adjusting docker-compose.yaml environment variables have you ensured those are mimicked in the Helm values.yaml file.
This pull request requires additional validation before any workflows can run on NVIDIA's runners.
Pull request vetters can view their responsibilities here.
Contributors can view more details about this message here.
@khangich Thanks for the PR!