Skip to content

qa: update pcp-mpstat CPU utilization output - #2699

Merged
natoscott merged 1 commit into
performancecopilot:mainfrom
orasagar:pcp_tools_bug_fix
Aug 27, 2026
Merged

qa: update pcp-mpstat CPU utilization output#2699
natoscott merged 1 commit into
performancecopilot:mainfrom
orasagar:pcp_tools_bug_fix

Conversation

@orasagar

Copy link
Copy Markdown
Contributor

Refresh QA 883 and 1782 expected output after correcting CPU utilization scaling. Aggregate values are divided by the CPU count, while per-CPU values retain their full utilization.

Refresh QA 883 and 1782 expected output after correcting CPU
utilization scaling. Aggregate values are divided by the CPU count,
while per-CPU values retain their full utilization.
@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • qa/1782.out is excluded by !**/*.out
  • qa/883.out is excluded by !**/*.out

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 2ba2ac57-92a7-4d1e-b01e-837166e91346

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@natoscott
natoscott merged commit a9b7fe5 into performancecopilot:main Aug 27, 2026
17 checks passed
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.

2 participants