-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Plan always shows an update with security_group_ids #33
Comments
@freimer Hey! Thanks for reporting. I've heard this before elsewhere, but was unable to reproduce it myself in recent versions of the provider. So - would you mind sharing me your version of the provider? If it's old, I'd appreciate if you could test out more recent version(s) of it. |
My lock file says this:
|
Same over here. version: provider "registry.terraform.io/mumoshu/eksctl" {
version = "0.15.2"
constraints = "0.15.2"
hashes = [
"h1:AwrWTLusDsAAbGq9NXabtKRUxpegdv+YEyx6c5tE+58=",
]
} Here's the snippet that's failing for me: resource "eksctl_cluster" "this" {
name = var.cluster_name
region = var.region
version = var.kubernetes_version
api_version = "eksctl.io/v1alpha5"
eksctl_version = "0.41.0"
spec = <<-EOS
vpc:
cidr: "${var.cidr}"
nat:
gateway: Disable
EOS
lifecycle {
ignore_changes = [
kubeconfig_path,
]
}
} Ended up ignoring |
i see it too, on terraform 0.15.2 and mumoshu/eksctl 0.16.2 i recall it was present months ago also |
In my log too, every time, very confusing, in fact, if you try to set it directly as a resource field, it refused to be set.
|
Same for me, I have a cluster that was created with Terraform v0.14.4 @mumoshu any idea what could be causing this?
|
I'm facing the same issue eksctl_cluster.eks will be updated in-place~ resource "eksctl_cluster" "eks" {
Solved kubectl_path issue by adding to lifecycle.ignore_changes |
A plan (or apply) always seems to show a change, when there is no change. Here's the output:
If I dump the state, security_group_ids is not shown. You can't set the security group IDs either, so I don't know the purpose for this.
The text was updated successfully, but these errors were encountered: