Skip to content

Improve logging in cpuallocator#5677

Open
zedi-pramodh wants to merge 1 commit intolf-edge:masterfrom
zedi-pramodh:improve-logging-cpuallocator
Open

Improve logging in cpuallocator#5677
zedi-pramodh wants to merge 1 commit intolf-edge:masterfrom
zedi-pramodh:improve-logging-cpuallocator

Conversation

@zedi-pramodh
Copy link

Add little more info to error message and also print actual error in domainmgr.

How to test and validate this PR

We can see these error messages if number of CPUs are exhausted in the device for the CPU pinning VMs.

Changelog notes

None.

PR Backports

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've written the test verification instructions

And the last but not least:

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

Please, check the boxes above after submitting the PR in interactive mode.

Add little more info to error message and also print actual error in domainmgr. Signed-off-by: Pramodh Pallapothu <pramodh@zededa.com>
@zedi-pramodh zedi-pramodh force-pushed the improve-logging-cpuallocator branch from e8a5ccc to 760f645 Compare March 18, 2026 20:08
@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.49%. Comparing base (2281599) to head (760f645).
⚠️ Report is 346 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@ ## master #5677 +/- ## ========================================== + Coverage 19.52% 29.49% +9.96%  ========================================== Files 19 18 -1 Lines 3021 2417 -604 ========================================== + Hits 590 713 +123  + Misses 2310 1552 -758  - Partials 121 152 +31 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Copy link
Contributor

@rene rene left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants