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

Standardize our Load Balancers to be Standard Class, as opposed to Basic #376

Closed
thanasisk opened this issue Sep 7, 2018 · 6 comments · Fixed by #848
Closed

Standardize our Load Balancers to be Standard Class, as opposed to Basic #376

thanasisk opened this issue Sep 7, 2018 · 6 comments · Fixed by #848
Assignees
@mjudeikis

Comments

@thanasisk
Copy link
Contributor

https://azure.microsoft.com/en-us/support/legal/sla/summary/

Basic LBs have NO SLA
In order to maintain consistency in our SLO/SLA calculations, we should switch to Standard Load Balancers that come with an SLA guarantee
@0xmichalis
Copy link
Contributor

https://azure.microsoft.com/en-us/support/legal/sla/load-balancer/v1_0/

good catch @thanasisk ! Mind opening a PR?

cc @pweil- @jim-minter

@0xmichalis 0xmichalis added priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. private-preview labels Sep 7, 2018
@jim-minter
Copy link
Contributor

standard load balancer support begins in k8s 1.11 kubernetes/kubernetes#62707

@jim-minter jim-minter removed priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. private-preview labels Sep 7, 2018
@jim-minter
Copy link
Contributor

it does not appear to be in openshift 3.10.

@jim-minter
Copy link
Contributor

Closed for AZURE-136

@0xmichalis 0xmichalis mentioned this issue Sep 26, 2018
Closed
@mjudeikis mjudeikis mentioned this issue Dec 5, 2018
Merged
@jim-minter jim-minter reopened this Dec 14, 2018
@jim-minter
Copy link
Contributor

@mjudeikis don't think this can be considered closed until loadBalancerSku: standard appears in our cloud provider conf? ;-)

@mjudeikis mjudeikis mentioned this issue Dec 17, 2018
Closed
@mjudeikis
Copy link
Contributor

Done. Was not fully aware of that option :)

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

@mjudeikis mjudeikis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

arm move to standard lb mjudeikis/openshift-azure
add missing config mjudeikis/openshift-azure
4 participants
@jim-minter @thanasisk @mjudeikis @0xmichalis