Fix CRDUtils::apply by replacing serverSideApply() with get(), update() and create()

[Donnerbart]

Fixes #659

[csviri]

LGTM , see my comment below

[csviri]

To be honest not sure if is this absolutely correct to have this based on ssaBasedCreateUpdateMatchForDependentResources since that is for completely other think.

IMO the apply of CRD could be done just by simple create / update. Since those are anwywas not managed by controllers.

@metacosm what do you think?

[Donnerbart]

Yeah, I'm also not sure if that's really a good configuration to use. My first approach was just using get(), update() and create(). Just tell me what you think, I can quickly adapt the code if needed.

[csviri]

I think the CRD apply should just use get/create/update, the SSA should be removed from here.

[metacosm]

Yes, I think we can remove SSA altogether from this, this was an overzealous change on my part.

[metacosm]

Please remove SSA altogether and change the commit message to follow the conventional commit format.

[Donnerbart]

The code is updated. I hope I've anticipated the commit message format correctly.

[csviri]

will try to fix the workflows before merge

[csviri]

Thank you @Donnerbart