Merge pull request #927 from wking/aws-public-worker-ips

pkg/asset/machines/aws/machinesets: Give workers public IPs (for now)

Author: openshift-merge-robot

pkg/asset/machines/aws/machinesets.go

@@ -6,6 +6,8 @@ import (
 
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
+	"k8s.io/utils/pointer"
+	awsprovider "sigs.k8s.io/cluster-api-provider-aws/pkg/apis/awsproviderconfig/v1alpha1"
 	clusterapi "sigs.k8s.io/cluster-api/pkg/apis/cluster/v1alpha1"
 
 	"github.com/openshift/installer/pkg/types"
@@ -88,3 +90,14 @@ func MachineSets(config *types.InstallConfig, pool *types.MachinePool, role, use
 
 	return machinesets, nil
 }
+
+// ConfigWorkers sets the PublicIP flag for the given machine sets.
+//
+// Deprecated: We'll remove this once the e2e tests no longer require
+// worker SSH access.
+func ConfigWorkers(machineSets []clusterapi.MachineSet) {
+	for _, machineSet := range machineSets {
+		providerConfig := machineSet.Spec.Template.Spec.ProviderConfig.Value.Object.(*awsprovider.AWSMachineProviderConfig)
+		providerConfig.PublicIP = pointer.BoolPtr(true)
+	}
+}

pkg/asset/machines/worker.go

@@ -100,6 +100,7 @@ func (w *Worker) Generate(dependencies asset.Parents) error {
 		if err != nil {
 			return errors.Wrap(err, "failed to create worker machine objects")
 		}
+		aws.ConfigWorkers(sets)
 
 		list := listFromMachineSets(sets)
 		raw, err := yaml.Marshal(list)