AWS: Support default for ExternalHost on AWS

justinsb · justinsb · commit 2c7771e90c8d · 2016-10-31T12:01:12.000-04:00
Fix #33563
diff --git a/pkg/genericapiserver/config.go b/pkg/genericapiserver/config.go
@@ -503,14 +503,14 @@ func DefaultAndValidateRunOptions(options *options.ServerRunOptions) {
 	// Set default value for ExternalHost if not specified.
 	if len(options.ExternalHost) == 0 {
 		// TODO: extend for other providers
-		if options.CloudProvider == "gce" {
+		if options.CloudProvider == "gce" || options.CloudProvider == "aws" {
 			cloud, err := cloudprovider.InitCloudProvider(options.CloudProvider, options.CloudConfigFile)
 			if err != nil {
 				glog.Fatalf("Cloud provider could not be initialized: %v", err)
 			}
 			instances, supported := cloud.Instances()
 			if !supported {
-				glog.Fatalf("GCE cloud provider has no instances.  this shouldn't happen. exiting.")
+				glog.Fatalf("%q cloud provider has no instances.  this shouldn't happen. exiting.", options.CloudProvider)
 			}
 			hostname, err := os.Hostname()
 			if err != nil {
