nv-ingest icon indicating copy to clipboard operation
nv-ingest copied to clipboard

add IngestService to context manager

Open khangich opened this issue 8 months ago • 2 comments

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.

khangich avatar Aug 13 '25 18:08 khangich

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.

copy-pr-bot[bot] avatar Aug 13 '25 18:08 copy-pr-bot[bot]

@khangich Thanks for the PR!

drobison00 avatar Aug 18 '25 18:08 drobison00