-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
ASM Module fails #609
Comments
@bharathkkb Any ideas? |
@chanuka-techo the example has
|
Thanks, tried that and got the following error.
I will install
Apparently
At this point the problem is probably not with the ASM module. But I just documented the errors hoping it'll help some unfortunate soul who takes the same route. |
@chanuka-techo thanks for the update. I am fixing the kustomize and beta (anthoscli) dependency in #602. However kpt should have been automatically installed. If you get a chance, could you give it a shot with #602? module "acm-example" {
source = "github.com/terraform-google-modules/terraform-google-kubernetes-engine//modules/acm?ref=fix-gcloud-install"
.
.
.
} Ideally, all of these should be automatically installed by the ASM module :) |
I have shot myself in the foot somewhere back in time and So the trick was |
@bharathkkb I see your change refers to a module named 'gcloud_wait_for_cluster'. This is an aside but reminded me how cluster creation always failed for me citing the subnet being not valid. I had to plan/apply again to start going. Should we need changes for kubernetes-engine modules as well? |
I am not sure if I understood this correctly, but if you were asking whether to change the |
Sorry, I misinterpreted the situation. This is a completely different problem. Maybe I'll report in appropriate place if needed. Terraform reports that the subnet creation succeeds.
Still, I get below error every time I start from scratch. Gotta plan/apply again to continue.
Thanks for the prompt support! |
@bharathkkb As you requested, I'm trying out #602 Just noticed you mentioned acm in your reply, not asm. Either it's a mistake or I really don't follow what's going on. Anyhow, trying out #602 for ASM. |
@chanuka-techo my apologies, I meant asm |
I have tried with both master and v10.0.0.
Since my attempt was failing, I just tried the code from the example but still the result is the same.
Please find the full apply output (personal info redacted) on this gist
The text was updated successfully, but these errors were encountered: