🌱 Add items to cache immediately after apply

[lentzi90]

What this PR does / why we need it:

Add items to cache immediately after apply to avoid extra no-op patch just for caching.

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 #

Part of #12291

[lentzi90]

Hmm race detected. I wonder if that could be a genuine issue from the code change 🤔
/test pull-cluster-api-test-main

[sbueringer]

Should be unrelated to your PR . I'll take a closer look during review. Also very likely the race is just in test code (fake client)

[sbueringer]

@lentzi90 Thank you very much. Last nit. I also tested it with Tilt and it works perfectly

[lentzi90]

Thanks! Fixed the last nit. Do you want me to squash the commits? I kept the changes to ComputeRequestIdentifier on a separate commit for now since I wasn't sure if we wanted it.

[sbueringer]

Separate commits is fine.

Thank you!

/lgtm
/assign @fabriziopandini

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: 44b6fc3887fa81c6adc276abef888b2632edac3b

[fabriziopandini]

Thanks for this improvement (and thanks @sbueringer for helping in getting this PR is such a good state)!
/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fabriziopandini

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [fabriziopandini]

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