Configurable instance types on AWS?

[wking]

This could be me just not understanding how this operator is wired up, but it looks like you currently hard-code m4.large here. I'd have expected that information to come from the installer via the cluster config. Am I missing something? Or is this just future work that hasn't happened yet?

[enxebre]

thanks @wking yes this is a todo

[enxebre]

Closing this as machines definition lives on the installer now
For reference:
openshift/installer#468
https://github.com/openshift/installer/tree/master/pkg/asset/machines