controller: don't set cloud provider for baremetal

[stbenjam]

This causes an error during installation:

Jun 18 16:39:30 master-0 hyperkube[2223]: F0618 16:39:30.475150    2223
server.go:267] failed to run Kubelet: unknown cloud provider "baremetal"

[kikisdeliveryservice]

Can we have a little more information about this? What will you be using as the value for platform? Have you done a fully successful install by removing this?

[stbenjam]

Can we have a little more information about this? What will you be using as the value for platform?

What do you mean by value for platform? Like libvirt, baremetal IPI doesn't have a cloud provider supported by kubelet, so it is set to "". My mistake when I added this in #846

[kikisdeliveryservice]

My mistake when I added this in #846

Ah didn't realize you were the person who added it.

[stbenjam]

@kikisdeliveryservice Do you want me to update the commit msg to match new title?

[kikisdeliveryservice]

this seems fine but will leave it to @cgwalters to lgtm

[cgwalters]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ashcrow, cgwalters, stbenjam

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [ashcrow,cgwalters]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment