Skip to content

fix(processor): Make monitor agent polling cancellable - #902

Merged
gfieni merged 2 commits into
masterfrom
refactor/processor-monitor-agent-lifecycle
Aug 5, 2026
Merged

fix(processor): Make monitor agent polling cancellable#902
gfieni merged 2 commits into
masterfrom
refactor/processor-monitor-agent-lifecycle

Conversation

@gfieni

@gfieni gfieni commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This PR improves shutdown handling for the Kubernetes and OpenStack monitoring agents.
It replaces blocking sleep calls with interruptible event waits, allowing the agents to react promptly to shutdown signals during polling delays.

@gfieni gfieni self-assigned this Aug 5, 2026
@gfieni
gfieni requested a review from a team as a code owner August 5, 2026 13:11
@gfieni gfieni added the bug Something isn't working label Aug 5, 2026
@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.22%. Comparing base (6e927b7) to head (64d0e49).

Files with missing lines Patch % Lines
.../powerapi/processor/pre/openstack/monitor_agent.py 0.00% 4 Missing ⚠️
src/powerapi/processor/pre/k8s/monitor_agent.py 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
- Coverage   77.29%   77.22%   -0.07%     
==========================================
  Files          83       83              
  Lines        3144     3144              
==========================================
- Hits         2430     2428       -2     
- Misses        714      716       +2     
Files with missing lines Coverage Δ
src/powerapi/processor/pre/k8s/monitor_agent.py 42.30% <0.00%> (-0.97%) ⬇️
.../powerapi/processor/pre/openstack/monitor_agent.py 73.07% <0.00%> (-1.93%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gfieni
gfieni merged commit 5c1ff8a into master Aug 5, 2026
11 checks passed
@gfieni
gfieni deleted the refactor/processor-monitor-agent-lifecycle branch August 5, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant