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.

Sign up for GitHub

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

Jump to bottom

AHB enabled cores incorrect when using SQL Elastic Pools #685

Open
pvansteenkiste opened this issue Apr 10, 2024 · 6 comments
Open

AHB enabled cores incorrect when using SQL Elastic Pools #685

pvansteenkiste opened this issue Apr 10, 2024 · 6 comments
Assignees
@arthurclares
Labels
Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity Solution: Workbooks Azure Monitor workbooks Status: 📦 Pending release Issue will be included in the next release Type: Bug 🐛 Something isn't working

Comments

@pvansteenkiste
Copy link

🐛 Problem

When I check the "SQL DB AHB Consumed Cores per VM" it seems like it adds up all databases on a SQL Elastic Pool ? For instance: we have 23 databases on a SQL Elastic pool with 6vcores. Then 23x6 is calculated as SQL DB AHB Consumed Cores ? (while of course the AHB is enabled on the Elastic Pool level). This means the overview is unusable to correctly reflect the REAL number of vcores consumed with AHB

👣 Repro steps

  1. Go to Rate Optimization - Azure Hybrid Benefit - SQL DB

🤔 Expected

I expect the value to be correct.

📷 Screenshots

image

image

🙋‍♀️ Ask for the community

We could use your help:

  1. Please vote this issue up (👍) to prioritize it.
  2. Leave comments to help us solidify the vision.
@pvansteenkiste pvansteenkiste added Needs: Triage 🔍 Untriaged issue needs to be reviewed Type: Bug 🐛 Something isn't working labels Apr 10, 2024
@arthurclares
Copy link
Contributor

Hello @pvansteenkiste
Acknowledged. Thank you for submitting this.
I will work on this and revert back with the fix.

@arthurclares arthurclares added Solution: Workbooks Azure Monitor workbooks Status: 🕵️‍♀️ Investigating Issue is actively being investigated and removed Needs: Triage 🔍 Untriaged issue needs to be reviewed labels Apr 10, 2024
@arthurclares
Copy link
Contributor

Hello @pvansteenkiste
There was an error on the queries used to generate these pie charts. I've changed it and it will be published to Advisor at the end of this month.

@arthurclares
Copy link
Contributor

Let me know if you can't wait until the end of the month to have this fixed and I can make it available on the Toolkit (Which has a faster publishing process).

@arthurclares arthurclares added Status: 📦 Pending release Issue will be included in the next release and removed Status: 🕵️‍♀️ Investigating Issue is actively being investigated labels Apr 17, 2024
@pvansteenkiste
Copy link
Author

pvansteenkiste commented Apr 17, 2024 via email

@pvansteenkiste
Copy link
Author

pvansteenkiste commented May 7, 2024 via email

@arthurclares
Copy link
Contributor

@pvansteenkiste The portal should be updated. Can you confirm if you see the SQL Elastic Pool tab under SQL Databases?
image

@arthurclares arthurclares added the Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arthurclares arthurclares

Labels
Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity Solution: Workbooks Azure Monitor workbooks Status: 📦 Pending release Issue will be included in the next release Type: Bug 🐛 Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@flanakin @pvansteenkiste @arthurclares