search

integr8ly / application-monitoring-operator

Operator for installing the Application Monitoring Stack on OpenShift (Prometheus, AlertManager, Grafana)
Apache License 2.0
30 stars 44 forks source link

remove subresources from crd #150

Closed pb82 closed 3 years ago

pb82 commented 3 years ago

Remove the subresources from the AMO CRD because we have not updated the controllers to use .Status().Update(). Without this change, the operator will get stuck because it can't update its phase.

The RHMI operator has addressed this in their copies of the CRDs: https://github.com/integr8ly/integreatly-operator/blob/master/manifests/integreatly-monitoring/1.2.1/applicationmonitoring.integreatly.org_applicationmonitorings_crd.yaml

openshift-ci-robot commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: david-martin

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/integr8ly/application-monitoring-operator/blob/master/OWNERS)~~ [david-martin] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment