search

mumoshu / terraform-provider-eksctl

Manage AWS EKS clusters using Terraform and eksctl
Apache License 2.0
234 stars 44 forks source link

Bug: Only capture stdout #64

Closed benjiro closed 1 year ago

benjiro commented 2 years ago

https://github.com/mumoshu/terraform-provider-eksctl/issues/62

eksctl writes various information to stderr that don't have anything to do with json output. Remove the capture on stderr to avoid false positives when parsing the output.

JustAnotherDevOps commented 2 years ago

@mumoshu since this resolves #62 could you please review to have this merged in?

Many users are pinned on eksctl v0.84.x which is not compatible with kubernetes v1.22 so are stuck without an upgrade option using this repo.

benjiro commented 2 years ago

@mumoshu any chance of getting this PR looked at? I want to upgrade to the latest EKS version using this provider but it's currently impossible unless I publish my fork.

mumoshu commented 1 year ago

Ah, apologies for missing your comments. Obviously I'm not maintaing this well- please feel free to fork this or ask for adding you folks as additioinoal maintainers if you're still using this.

benjiro commented 1 year ago

@mumoshu Yea I'm currently still using it. I don't mind help with maintaining it