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

Remove Azure support #229

Merged
merged 1 commit into from
Aug 18, 2017
Merged

Remove Azure support #229

merged 1 commit into from
Aug 18, 2017

Conversation

mwielgus
Copy link
Contributor

@mwielgus mwielgus commented Aug 17, 2017

With deep regret I have to remove the Azure-related code from Cluster Autoscaler. There are reports that it doesn't work and doesn't use the correct azure resources. Moreover the code doesn't compile with the newest Azure sdk and we couldn't find anyone to actually review and fix the existing code (#133 kubernetes/kubernetes#47511).

@mwielgus mwielgus requested a review from MaciekPytel August 17, 2017 20:33
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Aug 17, 2017
@MaciekPytel
Copy link
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 18, 2017
@mwielgus mwielgus merged commit caed0ee into kubernetes:master Aug 18, 2017
@pzerger
Copy link

pzerger commented Oct 3, 2017

@mwielgus, I am working on a project where we need to implement autoscale at the cluster level. Would be happy to review with my team to see if we can leverage this and clean up something broken for the project.

@feiskyer feiskyer mentioned this pull request Nov 6, 2017
8 tasks
yaroslava-serdiuk pushed a commit to yaroslava-serdiuk/autoscaler that referenced this pull request Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cluster-autoscaler cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants