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

Support GPUs in scale from 0 #401

Merged
merged 1 commit into from
Oct 18, 2017
Merged

Support GPUs in scale from 0 #401

merged 1 commit into from
Oct 18, 2017

Conversation

bskiba
Copy link
Member

@bskiba bskiba commented Oct 18, 2017

When scaling up from 0, read information about GPUs from instance template. This allows us to support scaling up a relevant NodeGroup from 0 when we have a pending pod that needs GPUs.

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 18, 2017
@bskiba
Copy link
Member Author

bskiba commented Oct 18, 2017

Fixes #392

@mwielgus
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 Oct 18, 2017
@mwielgus mwielgus merged commit 2418dba into kubernetes:master Oct 18, 2017
@mwielgus
Copy link
Contributor

cc: @vishh

Copy link

@vishh vishh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep in mind starting from v1.8, the resource name for GPUs is nvidia.com/gpu and it's not included in core k8s API. So this PR IIUC won't autoscaling clusters>1.8.1 since it's using the old resource name alpha.kubernetes.io/nvidia-gpu (apiv1.ResourceNvidiaGPU)

@bskiba bskiba mentioned this pull request Oct 19, 2017
@@ -182,6 +190,7 @@ func TestBuildAllocatableFromKubeEnv(t *testing.T) {
capacityMemory string
expectedCpu string
expectedMemory string
gcuCount int64
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: This should be gpuCount.

yaroslava-serdiuk pushed a commit to yaroslava-serdiuk/autoscaler that referenced this pull request Feb 22, 2024
Add enhancements template and script for table of contents
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants