follow up PR 421 to make ubuntu and centos work

[jichenjc]

What this PR does / why we need it:

The PR #421 has some missing code change for ubuntu and centos,
I created ubuntu correctly but centos has some issue (#391), so modify the code according to
ubuntu changes only

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

1. Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jichenjc

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 [jichenjc]

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

[jichenjc]

@sbueringer PTAL, thanks

[sbueringer]

/lgtm

[sbueringer]

But if this PR here works for you it means we don't need the cacert for APIServer, right? :)

Which makes sense to me. Because I don't think APIServer calls OpenStack

[jichenjc]

@sbueringer
https://github.com/kubernetes-sigs/cluster-api-provider-openstack/blob/master/pkg/cloud/openstack/services/userdata/kubeadm.go#L71 already has it

[sbueringer]

@sbueringer
https://github.com/kubernetes-sigs/cluster-api-provider-openstack/blob/master/pkg/cloud/openstack/services/userdata/kubeadm.go#L71 already has it

Yup. But only for the controller-manager. The apiserver has the same cloudprovider openstack flags but no cacert

[jichenjc]

ok, maybe I missed something, anyway, let's be it and if we find some issue later, we can do modifications