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

cherry pick: "Fix nvidia gpu resource name on AWS" to 1.1 #1130

Merged
merged 1 commit into from
Aug 9, 2018
Merged

cherry pick: "Fix nvidia gpu resource name on AWS" to 1.1 #1130

merged 1 commit into from
Aug 9, 2018

Conversation

typical-whack
Copy link
Contributor

cherry pick of:
#648

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 8, 2018
@rpunetha
Copy link

rpunetha commented Aug 9, 2018

When will this merge be performed? I'm having issue with version 1.1.1 of ASG.

@MaciekPytel
Copy link
Contributor

/lgtm

A word of caution - regardless of this fix CA only works correctly with GPU in GCP/GKE. For other providers it will create too many nodes with GPU when scaling-up due to kubernetes/kubernetes#54959. We still don't have a generic fix for this issue, due to reasons explained in my comment #597 (comment).

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 9, 2018
@MaciekPytel MaciekPytel merged commit b51d2fe into kubernetes:cluster-autoscaler-release-1.1 Aug 9, 2018
@typical-whack typical-whack deleted the cherry-pick-fix-nvidia-gpu-resource-name-1.1 branch August 9, 2018 14:30
yaroslava-serdiuk pushed a commit to yaroslava-serdiuk/autoscaler that referenced this pull request Feb 22, 2024
Bumps [k8s.io/component-base](https://github.com/kubernetes/component-base) from 0.27.5 to 0.27.6.
- [Commits](kubernetes/component-base@v0.27.5...v0.27.6)

---
updated-dependencies:
- dependency-name: k8s.io/component-base
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants