Upgrade CAPO version to v0.12.4 (v2)

[Dmitriy Rabotyagov (noonedeadpunk)]

In CAPO version v0.11.2 there is a severe bug allowing to accomplish
Denial of Service by any tenant.

Manual removal of VM by tenant which is managed by CAPO results
in a pod crash in a loop. This has been fixed with [1] and is part
of the 0.12.2 release.

This is alternative patch to #152 due to messing up with introduction
of DCO requirement.

[1] kubernetes-sigs/cluster-api-provider-openstack#2477

[Mohammed Naser (mnaser)]

The approach I rather take here is a new role specific for the k-orc since it's just a dependency, so I'm going to add a new role to this and add it as a dependency instead.

I'll see if we can use Kustomize as well.

[Mohammed Naser (mnaser)]

alright, it's working great now, I just have to add the nodeSelector patch and we should be able to land this.

Jonathan Rosser (@jrosser) Dmitriy Rabotyagov (@noonedeadpunk) lmk if this is looking good, i dont think it should break you and keeps things nice and organized.

[Mohammed Naser (mnaser)]

alright, this should sort out all of the issues surrounding pre-commit and linters..

[Oleksandr K. (okozachenko1203)]

Dmitriy Rabotyagov (@noonedeadpunk) Mohammed Naser (@mnaser) i upgraded the version to 0.12.4.
We can eliminate this patch and use upstream image https://github.com/vexxhost/atmosphere/blob/e129ccb98b2666e10aa2dcc48d2521ec7bc81181/patches/kubernetes-sigs/cluster-api-provider-openstack/allow-switching-filter-name-to-id.patch

[Oleksandr K. (okozachenko1203)]

recheck

[Mohammed Naser (mnaser)]

Oleksandr K. (@okozachenko1203) looks like its gonna pass, please push in your approval when you can :)