Introduce common controller#348
Introduce common controller#348kstrenkova wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kstrenkova The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
08b9788 to
30261b6
Compare
30261b6 to
7c7848c
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/adfde4fcd4304119bd34f1850e6865e3 ❌ openstack-k8s-operators-content-provider FAILURE in 8m 07s |
7c7848c to
ebe6b98
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d88e72ae94b345839b13e9924a10f912 ✔️ openstack-k8s-operators-content-provider SUCCESS in 27m 00s |
ebe6b98 to
43aab5f
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/500729983e5a4460b8eef76457b2d54f ❌ openstack-k8s-operators-content-provider FAILURE in 8m 28s |
43aab5f to
d20663b
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/308dd766cd8e431ab9626de71046829a ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 34m 29s |
0894c5a to
ba993e6
Compare
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.
ba993e6 to
fdb7757
Compare
|
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 🙏 |
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