-
Notifications
You must be signed in to change notification settings - Fork 66
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
Cannot create cluster #65
Comments
This change avoids the error: diff --git a/caliban/platform/gke/util.py b/caliban/platform/gke/util.py
index 9a754ea..0d70c45 100644
--- a/caliban/platform/gke/util.py
+++ b/caliban/platform/gke/util.py
@@ -395,9 +395,10 @@ def resource_limits_from_quotas(
gd = gpu_match.groupdict()
gpu_type = gd['gpu']
- limits.append({
- 'resourceType': 'nvidia-tesla-{}'.format(gpu_type.lower()),
- 'maximum': str(limit)
+ if limit > 0:
+ limits.append({
+ 'resourceType': 'nvidia-tesla-{}'.format(gpu_type.lower()),
+ 'maximum': str(limit)
})
return limits |
Erik, thanks for the bug report, and for the fix. I'll merge that in shortly. |
ajslone
added a commit
that referenced
this issue
Aug 2, 2020
ajslone
added a commit
that referenced
this issue
Aug 2, 2020
This addresses an error where we specify resource maxima for GKE clusters using quota information returned from the GCP compute API. The API can return a quota of zero for a given resource type, but requesting a limit of zero in the cluster creation API causes an error, so this change prevents setting a limit of zero for any resource.
Erik, I have merged this change, please let me know if this does not resolve your issue. Thanks again for the bug report and for using Caliban + GKE. |
This change resolves the issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I want to create a GKE cluster following your instructions. I have set up the cloud tools, authentication, etc. I receive this error message:
When I use
--dry_run
, I see these details:There is indeed a resource request with a maximum of 0.
I am using the current master branch.
The text was updated successfully, but these errors were encountered: