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

Unable to remove IG with LauchTemplate #6945

Closed
pkutishch opened this issue May 13, 2019 · 2 comments
Closed

Unable to remove IG with LauchTemplate #6945

pkutishch opened this issue May 13, 2019 · 2 comments

Comments

@pkutishch
Copy link
Contributor

1. What kops version are you running? The command kops version, will display
this information.

Version 1.12.0-beta.2 (git-d1453d22a)

2. What Kubernetes version are you running? kubectl version will print the
version if a cluster is running or provide the Kubernetes version specified as
a kops flag.

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.5", GitCommit:"32ac1c9073b132b8ba18aa830f46b77dcceb0723", GitTreeState:"clean", BuildDate:"2018-06-21T11:46:00Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.11", GitCommit:"637c7e288581ee40ab4ca210618a89a555b6e7e9", GitTreeState:"clean", BuildDate:"2018-11-26T14:25:46Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

**3. AWS

**4. kops delete ig test

**5.
Deleting instance group configuration failed with error: I0513 10:07:01.462950 2349 delete.go:51] Deleting "test"

error deleting cloud resources for InstanceGroup: error deleting autoscaling launch configuration "": InvalidParameter: 1 validation error(s) found.

  • minimum field size of 1, DeleteLaunchConfigurationInput.LaunchConfigurationName.

**6.
I expect IG to be removed.

9. Anything else do we need to know?
https://github.com/kubernetes/kops/blob/release-1.12/upup/pkg/fi/cloudup/awsup/aws_cloud.go#L367

Here is checks for LauchConfiguration but if you're using ASG with LauchTemplate it does not reference launch config and as the fact it returns empty string.

@pkutishch
Copy link
Contributor Author

#6951 - Fix in this PR

k8s-ci-robot added a commit that referenced this issue May 14, 2019
@pkutishch
Copy link
Contributor Author

PR merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant