*: remove support for container linux

This drops a number of configuration options that are specific to
Container Linux. This also changes the default region to us-east-1
throughout the tests.

Author: crawford

config.tf

@@ -59,18 +59,17 @@ variable "tectonic_container_images" {
   default = {
     addon_resizer                        = "gcr.io/google_containers/addon-resizer:2.1"
     bootkube                             = "quay.io/coreos/bootkube:v0.10.0"
-    tnc_operator                         = "quay.io/coreos/tectonic-node-controller-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
+    tnc_operator                         = "quay.io/coreos/tectonic-node-controller-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
     etcd_cert_signer                     = "quay.io/coreos/kube-etcd-signer-server:678cc8e6841e2121ebfdb6e2db568fce290b67d6"
     etcd                                 = "quay.io/coreos/etcd:v3.2.14"
-    hyperkube                            = "openshift/origin-node:latest"
-    kube_core_renderer                   = "quay.io/coreos/kube-core-renderer-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
-    kube_core_operator                   = "quay.io/coreos/kube-core-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
-    tectonic_channel_operator            = "quay.io/coreos/tectonic-channel-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
-    kube_addon_operator                  = "quay.io/coreos/kube-addon-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
+    kube_core_renderer                   = "quay.io/coreos/kube-core-renderer-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
+    kube_core_operator                   = "quay.io/coreos/kube-core-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
+    tectonic_channel_operator            = "quay.io/coreos/tectonic-channel-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
+    kube_addon_operator                  = "quay.io/coreos/kube-addon-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
     tectonic_alm_operator                = "quay.io/coreos/tectonic-alm-operator:v0.3.1"
-    tectonic_ingress_controller_operator = "quay.io/coreos/tectonic-ingress-controller-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
-    tectonic_utility_operator            = "quay.io/coreos/tectonic-utility-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
-    tectonic_network_operator            = "quay.io/coreos/tectonic-network-operator-dev:436b1b4395ae54d866edc88864c9b01797cebac1"
+    tectonic_ingress_controller_operator = "quay.io/coreos/tectonic-ingress-controller-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
+    tectonic_utility_operator            = "quay.io/coreos/tectonic-utility-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
+    tectonic_network_operator            = "quay.io/coreos/tectonic-network-operator-dev:0a24db2288f00b10ced358d9643debd601ffd0f1"
   }
 }
 
@@ -192,26 +191,6 @@ You can download the pull secret from your Account overview page at [3].
 EOF
 }
 
-variable "tectonic_container_linux_channel" {
-  type = "string"
-
-  description = <<EOF
-The Container Linux update channel.
-
-Examples: `stable`, `beta`, `alpha`
-EOF
-}
-
-variable "tectonic_container_linux_version" {
-  type = "string"
-
-  description = <<EOF
-The Container Linux version to use. Set to `latest` to select the latest available version for the selected update channel.
-
-Examples: `latest`, `1465.6.0`
-EOF
-}
-
 variable "tectonic_update_server" {
   type        = "string"
   default     = "https://tectonic.update.core-os.net"

examples/tectonic.aws.yaml

@@ -3,11 +3,6 @@ admin:
   password: "verysecure"
   sshKey: "ssh-ed25519 AAAA..."
 aws:
-  # (optional) Unique name under which the Amazon S3 bucket will be created. Bucket name must start with a lower case name and is limited to 63 characters.
-  # The Tectonic Installer uses the bucket to store tectonic assets and kubeconfig.
-  # If name is not provided the installer will construct the name using "name", current AWS region and "baseDomain"
-  # assetsS3BucketName:
-
   # (optional) AMI override for all nodes. Example: `ami-foobar123`.
   # ec2AMIOverride:
 
@@ -95,7 +90,7 @@ aws:
   # publicEndpoints: true
 
   # The target AWS region for the cluster.
-  region: eu-west-1
+  region: us-east-1
 
   # Block of IP addresses used by the VPC.
   # This should not overlap with any other networks, such as a private datacenter connected via Direct Connect.
@@ -164,20 +159,6 @@ ca:
   # This field is mandatory if `ca_cert` is set.
   # keyAlg: RSA
 
-containerLinux:
-  # (optional) The Container Linux update channel.
-  #
-  # Examples: `stable`, `beta`, `alpha`
-  # channel: stable
-
-  # The Container Linux version to use. Set to `latest` to select the latest available version for the selected update channel.
-  #
-  # Examples: `latest`, `1465.6.0`
-  version: latest
-
-  # (optional) A list of PEM encoded CA files that will be installed in /etc/ssl/certs on etcd, master, and worker nodes.
-  # customCAPEMList:
-
 iscsi:
   # (optional) Start iscsid.service to enable iscsi volume attachment.
   # enabled: false

examples/tectonic.libvirt.yaml

@@ -32,20 +32,6 @@ ca:
   # This field is mandatory if `ca_cert` is set.
   # keyAlg: RSA
 
-containerLinux:
-  # (optional) The Container Linux update channel.
-  #
-  # Examples: `stable`, `beta`, `alpha`
-  channel: beta
-
-  # The Container Linux version to use. Set to `latest` to select the latest available version for the selected update channel.
-  #
-  # Examples: `latest`, `1465.6.0`
-  version: latest
-
-  # (optional) A list of PEM encoded CA files that will be installed in /etc/ssl/certs on master and worker nodes.
-  # customCAPEMList:
-
 iscsi:
   # (optional) Start iscsid.service to enable iscsi volume attachment.
   # enabled: false

installer/pkg/config-generator/fixtures/kube-system.yaml

@@ -40,7 +40,7 @@ data:
       type: canal
     platform:
       aws:
-        region: eu-west-1
+        region: us-east-1
         vpcCIDRBlock: 10.0.0.0/16
         vpcID: ""
     pullSecret: '{"auths": {}}'

installer/pkg/config-generator/fixtures/test-aws.yaml

@@ -13,14 +13,11 @@ worker:
   nodePools:
     - worker
 pullSecret: '{"auths": {}}'
-containerLinux:
-  channel: stable
-  version: latest
 admin:
   email: test@coreos.com
   password: asd123
 aws:
-  region: eu-west-1
+  region: us-east-1
   sshKey: tectonic
   vpcCIDRBlock: 10.0.0.0/16
   master:

installer/pkg/config-generator/generator_test.go

@@ -14,7 +14,7 @@ import (
 func initConfig(t *testing.T, file string) ConfigGenerator {
 	cluster, err := config.ParseConfigFile("./fixtures/" + file)
 	if err != nil {
-		t.Errorf("Test case TestUrlFunctions: failed to parse test config, %s", err)
+		t.Fatalf("Test case TestUrlFunctions: failed to parse test config, %s", err)
 	}
 
 	return ConfigGenerator{

installer/pkg/config/BUILD.bazel

@@ -14,6 +14,7 @@ go_library(
         "//installer/pkg/config/aws:go_default_library",
         "//installer/pkg/config/libvirt:go_default_library",
         "//installer/pkg/validate:go_default_library",
+        "//pkg/rhcos:go_default_library",
         "//vendor/github.com/Sirupsen/logrus:go_default_library",
         "//vendor/github.com/coreos/ignition/config/v2_2:go_default_library",
         "//vendor/github.com/coreos/tectonic-config/config/tectonic-network:go_default_library",

installer/pkg/config/aws/aws.go

@@ -15,7 +15,7 @@ const (
 	// DefaultProfile is the default AWS credentials profile to use.
 	DefaultProfile = "default"
 	// DefaultRegion is the default AWS region for the cluster.
-	DefaultRegion = "eu-west-1"
+	DefaultRegion = "us-east-1"
 )
 
 // AWS converts AWS related config.

installer/pkg/config/cluster.go

@@ -21,6 +21,8 @@ const (
 	PlatformAWS Platform = "aws"
 	// PlatformLibvirt is the platform for a cluster launched on libvirt.
 	PlatformLibvirt Platform = "libvirt"
+	// DefaultChannel is the default RHCOS channel for the cluster.
+	DefaultChannel = "tested"
 )
 
 // Platform indicates the target platform of the cluster.
@@ -54,10 +56,6 @@ var defaultCluster = Cluster{
 	CA: CA{
 		RootCAKeyAlg: "RSA",
 	},
-	ContainerLinux: ContainerLinux{
-		Channel: ContainerLinuxChannelStable,
-		Version: ContainerLinuxVersionLatest,
-	},
 	Libvirt: libvirt.Libvirt{
 		Network: libvirt.Network{
 			IfName: libvirt.DefaultIfName,
@@ -77,7 +75,6 @@ type Cluster struct {
 	aws.AWS         `json:",inline" yaml:"aws,omitempty"`
 	BaseDomain      string `json:"tectonic_base_domain,omitempty" yaml:"baseDomain,omitempty"`
 	CA              `json:",inline" yaml:"CA,omitempty"`
-	ContainerLinux  `json:",inline" yaml:"containerLinux,omitempty"`
 	IgnitionMasters []string `json:"tectonic_ignition_masters,omitempty" yaml:"-"`
 	IgnitionWorker  string   `json:"tectonic_ignition_worker,omitempty" yaml:"-"`
 	Internal        `json:",inline" yaml:"-"`

installer/pkg/config/parser.go

@@ -2,9 +2,12 @@ package config
 
 import (
 	"errors"
+	"fmt"
 	"io/ioutil"
 
 	"gopkg.in/yaml.v2"
+
+	"github.com/openshift/installer/pkg/rhcos"
 )
 
 // ParseConfig parses a yaml string and returns, if successful, a Cluster.
@@ -28,6 +31,14 @@ func ParseConfig(data []byte) (*Cluster, error) {
 		cluster.PullSecret = string(data)
 	}
 
+	if cluster.EC2AMIOverride == "" {
+		ami, err := rhcos.AMI(DefaultChannel, cluster.AWS.Region)
+		if err != nil {
+			return nil, fmt.Errorf("Failed to determine default AMI: %v", err)
+		}
+		cluster.EC2AMIOverride = ami
+	}
+
 	return &cluster, nil
 }