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

CoreOS AMI query is broken #838

Closed
beyondbill opened this issue Sep 25, 2020 · 2 comments · Fixed by #839
Closed

CoreOS AMI query is broken #838

beyondbill opened this issue Sep 25, 2020 · 2 comments · Fixed by #839

Comments

@beyondbill
Copy link
Contributor

Description

CoreOS AMI query returns no result.

Steps to Reproduce

Try creating a k8s cluster in AWS with Typhoon. It throws the following errors

Error: Your query returned no results. Please change your search criteria and try again.

  on .terraform/modules/green.typhoon/aws/container-linux/kubernetes/ami.tf line 11, in data "aws_ami" "coreos":
  11: data "aws_ami" "coreos" {


Error: Your query returned no results. Please change your search criteria and try again.

  on .terraform/modules/green.typhoon/aws/container-linux/kubernetes/workers/ami.tf line 11, in data "aws_ami" "coreos":
  11: data "aws_ami" "coreos" {

Environment

  • Platform: aws
  • OS: container-linux
  • Release: Typhoon v1.18.2
  • Terraform: v0.12.29

Possible Solution

A quick search of CoreOS-stable in all public AMIs shows that most of the results owned by 679593333241. None of the AMIs are owned by the owner hardcoded in Typhoon today. It probably recently deleted all CoreOS-stable AMIs. Updating owners filter to ["679593333241"] will fix the issue. But I won't open a PR until the legitimacy of this owner is verified.

@dghubble
Copy link
Member

dghubble commented Sep 25, 2020

Yes, CoreOS was deprecated in favor of Fedora CoreOS or Flatcar Linux in v1.18.3 (May). The CoreOS-specific code was left in order to give people as much time as possible to move.

But now that CoreOS has followed up on its planned AMI deletion, the various references can be removed. Since they'll not longer be of use to anyone.

Those using Flatcar Linux with AWS would see this error (due to the way the AWS provider unconditionally requests) and need to update the reference #839. Other platforms won't be affected by CoreOS deleting their images.

@dghubble
Copy link
Member

Thanks for the report.

For Flatcar Linx + AWS, you could use 343db5b578b2712753e706509dfd119e786fa018 or later from #839, other platforms shouldn't need a change.

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

Successfully merging a pull request may close this issue.

2 participants