Refactor Reconciliation Flow (open-telemetry#1995)

* save

* bump

* 🤷

* should b good

* change delete logic

* change error comparison

* gomod

* Working status

* remove cruft

* fix selector check for deepequals

* add controller references

* Fix lint and test

* facepalm

* Record field

* Fix diffing bug

* Add lots of tests

* Remove old logic entirely

* dont forget the kube builder

* update comment, reusable builder

* respond to feedback

* update based on feedback

* update comments, remove method that does nothing

* naming

.chloggen/reconcile-change.yaml

@@ -0,0 +1,16 @@
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: enhancement
+
+# The name of the component, or a single word describing the area of concern, (e.g. operator, target allocator, github action)
+component: operator
+
+# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Introduces a new method of reconciliation to reduce duplication and complexity
+
+# One or more tracking issues related to the change
+issues: [1959]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:

bundle/manifests/opentelemetry-operator.clusterserviceversion.yaml

@@ -110,6 +110,8 @@ spec:
           - ""
           resources:
           - configmaps
+          - serviceaccounts
+          - services
           verbs:
           - create
           - delete
@@ -132,46 +134,12 @@ spec:
           verbs:
           - list
           - watch
-        - apiGroups:
-          - ""
-          resources:
-          - serviceaccounts
-          verbs:
-          - create
-          - delete
-          - get
-          - list
-          - patch
-          - update
-          - watch
-        - apiGroups:
-          - ""
-          resources:
-          - services
-          verbs:
-          - create
-          - delete
-          - get
-          - list
-          - patch
-          - update
-          - watch
         - apiGroups:
           - apps
           resources:
           - daemonsets
-          verbs:
-          - create
-          - delete
-          - get
-          - list
-          - patch
-          - update
-          - watch
-        - apiGroups:
-          - apps
-          resources:
           - deployments
+          - statefulsets
           verbs:
           - create
           - delete
@@ -188,18 +156,6 @@ spec:
           - get
           - list
           - watch
-        - apiGroups:
-          - apps
-          resources:
-          - statefulsets
-          verbs:
-          - create
-          - delete
-          - get
-          - list
-          - patch
-          - update
-          - watch
         - apiGroups:
           - autoscaling
           resources:

config/rbac/role.yaml

@@ -8,6 +8,8 @@ rules:
   - ""
   resources:
   - configmaps
+  - serviceaccounts
+  - services
   verbs:
   - create
   - delete
@@ -30,46 +32,12 @@ rules:
   verbs:
   - list
   - watch
-- apiGroups:
-  - ""
-  resources:
-  - serviceaccounts
-  verbs:
-  - create
-  - delete
-  - get
-  - list
-  - patch
-  - update
-  - watch
-- apiGroups:
-  - ""
-  resources:
-  - services
-  verbs:
-  - create
-  - delete
-  - get
-  - list
-  - patch
-  - update
-  - watch
 - apiGroups:
   - apps
   resources:
   - daemonsets
-  verbs:
-  - create
-  - delete
-  - get
-  - list
-  - patch
-  - update
-  - watch
-- apiGroups:
-  - apps
-  resources:
   - deployments
+  - statefulsets
   verbs:
   - create
   - delete
@@ -86,18 +54,6 @@ rules:
   - get
   - list
   - watch
-- apiGroups:
-  - apps
-  resources:
-  - statefulsets
-  verbs:
-  - create
-  - delete
-  - get
-  - list
-  - patch
-  - update
-  - watch
 - apiGroups:
   - autoscaling
   resources: