simonsobs / simonsobs/sotodlib
Add support for Monitor InfluxDBClient timeout
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 19
- Forks
- 23
- Avg merge
- 1d 5h
- Merged PRs (30d)
- 14
Description
The timeout argument for InfluxDBClient should be added to the Monitor class so that we can handle timeouts to the InfluxDB that might occur during a flow.
This timeout gets used when trying to make the initial connection to InfluxDB and on any request, which is how data gets sent to the database. Right now it'll probably just hang if the request doesn't go through.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in sotodlib/site_pipeline/monitor.py by reading the Monitor class and the existing InfluxDBClient initialization around lines 97-99. Trace how the client is created and how requests send data, then add support for passing the timeout through so both initial connection and requests can time out. Done means Monitor users can configure this timeout instead of requests hanging indefinitely.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- databases
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100