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

Code was not updating the RequestedSLO as expected. #1

Open
jcbendernh opened this issue Mar 9, 2022 · 2 comments
Open

Code was not updating the RequestedSLO as expected. #1

jcbendernh opened this issue Mar 9, 2022 · 2 comments

Comments

@jcbendernh
Copy link

jcbendernh commented Mar 9, 2022

We had to change line 200 of the AutoScaler.cs file from ...

var index = availableSlos.IndexOf(currentSLO.ToString());

to...

var index = availableSlos.IndexOf(currentSLO.ToString().ToLower());

Once we did that, the RequestedSLO was showing correctly on the [dbo].[AutoscalerMonitor] table and also the ALTER DATABASE script was executing as expected to scale up/scale down the server.

Neither were happening with the old line 200.

I am also a Microsoft employee, you can reach me at [email protected].

@yorek
Copy link
Contributor

yorek commented Mar 10, 2022

Yeah, that make sense, not sure why it has been working before :). Would be great if you can create a PR so that you can also get the proper credit for fixing this :)

@stanislavrupec
Copy link

stanislavrupec commented Mar 1, 2023

Hello @yorek, first of all, big thanks to you for this solution!!! I want to just propose that it will be good to get this code updated. I just spent some time to figure out why the function is not scaling as I didn't notice that there is opened issue with this. I basically changed line number 21:
From: return $"{Name}gen{Generation}{Cores}".ToUpper();
To: return $"{Name}gen{Generation}{Cores}".ToLower();

Without that, case sensitivity blocked IF conditions in "public static HyperScaleTier GetServiceObjective(HyperScaleTier currentSLO, SearchDirection direction)" Screen from my testing output:
image

Thank you!
BR
Stanislav

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

No branches or pull requests

3 participants