Introduce common controller

[kstrenkova]

To remove duplicate code across the test-operator controllers, this patch introduces a common controller. This change should make fixing controller bugs easier and prepare the test-operator for new resource addition, if it is needed in the future.

Assisted-by: Claude AI

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/adfde4fcd4304119bd34f1850e6865e3

❌ openstack-k8s-operators-content-provider FAILURE in 8m 07s
⚠️ podified-multinode-edpm-deployment-crc-test-operator SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d88e72ae94b345839b13e9924a10f912

✔️ openstack-k8s-operators-content-provider SUCCESS in 27m 00s
❌ podified-multinode-edpm-deployment-crc-test-operator RETRY_LIMIT in 11m 45s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/500729983e5a4460b8eef76457b2d54f

❌ openstack-k8s-operators-content-provider FAILURE in 8m 28s
⚠️ podified-multinode-edpm-deployment-crc-test-operator SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/308dd766cd8e431ab9626de71046829a

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 34m 29s
❌ podified-multinode-edpm-deployment-crc-test-operator TIMED_OUT in 3h 18m 25s

[kstrenkova]

Hi @stuggi 👋 I have this big change to create a common controller structure to reduce duplication in the controllers for different resource types. I wanted to ask if you could take a look at it logic-wise or if you know someone that could help me with reviewing this. I plan to do more local testing for different parts of the code, but as the check job passes I think the main logic should work, so I wanted to get some eyes on it now. Thank you in advance 🙏

[kstrenkova]

The change is ready for review. I will keep the hold label until enough people review the code, as it is a big change.

@adrianfusco @belolipa @sauragar @imatza-rh Please take a look when you have time 🙏

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrianfusco, kstrenkova

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [kstrenkova]

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