Skip to content

BED-8747: Check-in to BHE during collection#42

Closed
StranDutton wants to merge 9 commits into
mainfrom
feature/BED-8747-health-checkins-during-collections
Closed

BED-8747: Check-in to BHE during collection#42
StranDutton wants to merge 9 commits into
mainfrom
feature/BED-8747-health-checkins-during-collections

Conversation

@StranDutton

@StranDutton StranDutton commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

OpenHound stops making any API calls to BHE while a collection job runs. BHE kills the job after the 10-minute client check-in timeout.

This PR adds a periodic check-in so BHE knows the collector is still alive throughout a collection.

Changes:

  • Hard-coded POLL_INTERVAL = 30 in service.py.
  • Removed interval parameter from Service.init.
  • Updated _poll() to branch based on job status:
  • Idle: Calls check_jobs() to look for new jobs
  • Active: Accesses self.client.jobs_current to perform a check-in
    Configuration & Cleanup:
  • Removed all interval parsing and loading logic from src/scheduler.py.
  • Purged DESTINATION__BLOODHOUNDENTERPRISE__INTERVAL and related interval keys from:
  • deployments/helm/openhound/templates/deployment.yaml
  • deployments/helm/openhound/values.yaml
  • deployments/helm/values.example.yaml
  • All .dlt-example/secrets_*.toml files.
    Tests:
  • Unit tests added
  • Updated test fixtures to remove the interval argument.

/resolves BED-8747

@StranDutton StranDutton self-assigned this Jun 25, 2026
@StranDutton StranDutton force-pushed the feature/BED-8747-health-checkins-during-collections branch from 2f9a6c8 to 01f2804 Compare June 25, 2026 19:36
@StranDutton

Copy link
Copy Markdown
Contributor Author

closing so i can base the work on feature branch

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.

3 participants