Value Statement

ACM has introduced a Custom Resource called InternalHubComponent ([ACM-12784|https://github.com/stolostron/multiclusterhub-operator/pull/1690)] which allows deployed components to signal to the multiclusterhub-operator that there is still cleanup left to do before MCH will attempt to delete the component operator.

By putting a finalizer on the Submariner instance of the InternalHubComponent and then removing it after the Submariner operator is finished with cleanup, we eliminate race conditions and orphaned components that can cause uninstalls to require manual intervention by the customer.

Definition of Done for Engineering Story Owner (Checklist)

• [ ] The submariner operator ensures that there is a finalizer on the Submariner InternalHubComponent instance
• [ ] When a deletionTimestamp is detected on the InternalHubComponent, Submariner operator should understand that it is being deleted and begin the cleanup process
• [ ] When the cleanup is done, removing the finalizer from the InternalHubComponent will signal to MCH that it can safely delete the Submariner operator

Development Complete

• The code is complete.
• Functionality is working.
• Any required downstream Docker file changes are made.

Tests Automated

• [ ] Unit/function tests have been automated and incorporated into the
  build.
• [ ] 100% automated unit/function test coverage for new or changed APIs.

Secure Design

• [ ] Security has been assessed and incorporated into your threat model.