Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Best Practice Summary may incorrectly state Power Plan issue. #145

Open
gambit9009 opened this issue Mar 17, 2022 · 0 comments
Open

Best Practice Summary may incorrectly state Power Plan issue. #145

gambit9009 opened this issue Mar 17, 2022 · 0 comments
Assignees
Labels

Comments

@gambit9009
Copy link
Contributor

gambit9009 commented Mar 17, 2022

The perfstatsanalysis script creates an sp called "proc_PowerPlan".

if not exists (select * from tbl_PowerPlan where ActivePlanName like '%High Performance%')

So if the power plan name is not fully spelled out like that, it may flag "Power Plan is not set to high performance". But, at least in later Windows, you can rename your power plan. PC manufactures may have their own names for plans, etc. Someone could use the High Perf plan and change settings to it, without renaming it. The actual settings under the power plan, like CPU throttling, are really what we are concerned with, and not the name of the plan.

  1. Need to consider for SQL Nexus, at least changing the message that server "might not" have proper power plan...and then be helpful and actually concatenate the power plan name we do see in the message. Might help stop having to look up false positives about the server's power plan.
  2. PSSDiag captures __SQL_Base_Power_Plan_Startup.txt and that appears to be what gets added to tbl_PowerPlan. But also _SQL2019_MiscPssdiagInfo_Startup.OUT captures powerplan in --ServerProperty-- which gets added into tbl_ServerProperties by Nexus. "proc_PowerPlan" consider updating to check both places if one doesn't have it.
  3. See also DiagManager issue 148 as we may decide to change what we collect. Power plan capture may be misleading. DiagManager#148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant