🏃Remove most uses of submakes in Makefile

[mdbooth]

What this PR does / why we need it:
This commit series systematically removes most uses of submakes in the top level Makefile. It was fairly easy to write and I personally think it results in a more maintainable Makefile.

HOWEVER, I'm not aware of any bugs it addresses (yet), there's obviously the possibility that it has introduced bugs, and we don't have great test coverage of the Makefile, so... is it worth it?

/hold

[netlify]

✔️ Deploy Preview for kubernetes-sigs-cluster-api-openstack ready!

🔨 Explore the source changes: 0b2df35

🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-sigs-cluster-api-openstack/deploys/61851e580d631300082e1b17

😎 Browse the preview: https://deploy-preview-1040--kubernetes-sigs-cluster-api-openstack.netlify.app

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mdbooth
To complete the pull request process, please assign hidekazuna after the PR has been reviewed.
You can assign the PR to them by writing /assign @hidekazuna in a comment when ready.

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

Needs approval from an approver in each of these files:

• OWNERS

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

[jichenjc]

/test pull-cluster-api-provider-openstack-e2e-test

[jichenjc]

does this conflict to #1039 ?

[tobiasgiese]

does this conflict to #1039 ?

This is just a WIP PR and should not be merged, see #1039 (comment)

@mdbooth you can add the WIP label in such cases

/title WIP Remove .EXPORT_ALL_VARIABLES

[tobiasgiese]

/retitle WIP Remove .EXPORT_ALL_VARIABLES

[iamemilio]

/lgtm

This is more readable too IMO

[jichenjc]

I think it's a great PR
the question I am having is as commit msg said, how to test those ..
if we are sure current test is good enough and overtime we can add more test to cover those newly added rules, I can live with that, just need some agreement .. thanks

[hidekazuna]

/lgtm

[mdbooth]

Seems like there's some agreement to merge this.

/hold cancel

[hidekazuna]

I look back again this PR due to time has passed. I take a look at CAPA and found our current Makefile is like CAPA. I am not familiar with Makefile so that I want to continue to refer to Makefile in CAPA as of now.

/lgtm cancel

[mdbooth]

I look back again this PR due to time has passed. I take a look at CAPA and found our current Makefile is like CAPA. I am not familiar with Makefile so that I want to continue to refer to Makefile in CAPA as of now.

/lgtm cancel

That's entirely reasonable, and I also didn't feel confident pushing it due to the lack of test coverage. I'll drop this.