MGDAPI-5901 - feat: addon instance uninstallation flow

[KevFan]

Issue link

https://issues.redhat.com/browse/MGDAPI-5901

What

• Acccount for addon uninstallation flow by addon instance
  • Implementation is based on the changes provided by Feat: Support new deletion flow openshift/addon-operator#358
  • Note: status condition type used to the implementation above was incorrect and will be fixed by fix: use addonsv1alpha1.AddonInstanceConditionReadyToBeDeleted.String() openshift/addon-operator#375 which will also provide a client function also that we can later use once released

Verification steps

• Checkout this branch
• Install RHOAM

export LOCAL=false
make cluster/prepare/local
make code/run

• Verify installation completes successfully
• Mark addon instance spec for addon deletion

oc patch addoninstance addon-instance -p $'spec:\n markedForDeletion: true' --type merge -n redhat-rhoam-operator

• Verify RHMI CR is marked for deletion
• Verify uninstallation completed successfully
• Verify a new status condition is added

watch "oc get addoninstance addon-instance -n redhat-rhoam-operator -o json | jq '.status'"

• The following should be added after a successfull uninstall to signal that the addon can be removed in ocm

    {
      "lastTransitionTime": "2023-07-06T13:07:43Z",
      "message": "Addon ready to be deleted",
      "observedGeneration": 2,
      "reason": "AddonReadyToBeDeleted",
      "status": "True",
      "type": "addons.managed.openshift.io/ReadyToBeDeleted"
    }

[codecov]

Codecov Report

Merging #3303 (1440432) into master (7d22568) will decrease coverage by 0.65%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3303      +/-   ##
==========================================
- Coverage   72.95%   72.30%   -0.65%     
==========================================
  Files         107      107              
  Lines       29864    30194     +330     
==========================================
+ Hits        21786    21832      +46     
- Misses       7284     7566     +282     
- Partials      794      796       +2     

Impacted Files	Coverage Δ
apis/v1alpha1/rhmi_conditions.go	100.00% <100.00%> (ø)
controllers/status/status_controller.go	77.50% <100.00%> (+0.87%)	⬆️

... and 1 file with indirect coverage changes

[Patryk-Stefanski]

The verification steps were successful and the code looks good.
/lgtm

[MStokluska]

/approve

[austincunningham]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: austincunningham, MStokluska

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 [MStokluska,austincunningham]

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

[KevFan]

/override codecov/project

[openshift-ci]

@KevFan: Overrode contexts on behalf of KevFan: codecov/project

In response to this:

/override codecov/project

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.