Add contactpoint resource

[duncan485]

Any feedback is highly appreciated! Used #1420 as a refrence to create this.

closes #1455

[theSuess]

Thanks for the contribution! Works well and the overall code quality looks good

I have some concerns around UID handling. The current approach does not allow for renaming of resources, as the UID is retrieved by matching the name.

IMHO a better approach would be to use the Kubernetes UID (part of the object metadata). This way we don't have to do any name matching in the operator and have a consistent UID. We do something similar in the folder resource.

[duncan485]

Thanks for the contribution! Works well and the overall code quality looks good

I have some concerns around UID handling. The current approach does not allow for renaming of resources, as the UID is retrieved by matching the name.

IMHO a better approach would be to use the Kubernetes UID (part of the object metadata). This way we don't have to do any name matching in the operator and have a consistent UID. We do something similar in the folder resource.

I agree with you, will change it to use the K8s UID

[theSuess]

LGTM! Please fix the last whitespace issue and we're good to merge from my side.

@pb82 @NissesSenap - can you take a quick look at this as well? I've tested this but can't hurt to have someone else take a glance at this as well

[duncan485]

Thanks @theSuess , I just removed the whitespace so hopefully passes that last test now

[HVBE]

LGTM, looking forward to hearing some reports of how well this works for our users, awesome work! thank you @duncan485