Revert "Fix goroutine leak when reconciling"

[turkenf]

Description of your changes

This reverts commit 0927b1f.

With the above-reverted change, we encountered issues creating and deleting some resources.

• examples/cloudplatform/v1beta1/projectservice.yaml

    message: 'async create failed: failed to create the resource: [{0 Error when reading
      or editing Project Service : Parent context of request project/official-provider-testing/services
      canceled  []}]'

• examples/cloudplatform/v1beta1/projectiammember.yaml

    message: 'create failed: async create failed: failed to create the resource: [{0
      Parent context of request iam-project-official-provider-testing modifyIamPolicy
      canceled  []}]'

• examples/container/v1beta1/cluster.yaml

    message: 'create failed: async create failed: cannot get connection details: cannot
      get additional connection details: endpoint: not a string'

    message: 'async delete failed: failed to delete the resource: [{0 Error waiting
      for deleting GKE cluster: error while retrieving operation: unable to finish
      polling, context has been cancelled  []}]'

Fixes #611

I have:

• Read and followed Crossplane's contribution process.
• Run make reviewable to ensure this PR is ready for review.
• Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

The above failed tests have been tested manually and with uptest:

• examples/cloudplatform/v1beta1/projectservice.yaml: https://github.com/crossplane-contrib/provider-upjet-gcp/actions/runs/10771811848

NAME                                                          SYNCED   READY   EXTERNAL-NAME                                          AGE
projectservice.cloudplatform.gcp.upbound.io/project-service   True     True    official-provider-testing/clouddeploy.googleapis.com   2m8s

• examples/cloudplatform/v1beta1/projectiammember.yaml: https://github.com/crossplane-contrib/provider-upjet-gcp/actions/runs/10771726984

NAME                                                                SYNCED   READY   EXTERNAL-NAME
                                                    AGE
projectiammember.cloudplatform.gcp.upbound.io/project-iam-member2   True     True    official-provider-testing/roles/viewer/serviceAccount:official-provider-testing-noop@
official-provider-testing.iam.gserviceaccount.com   2m25s

• examples/container/v1beta1/cluster.yaml: https://github.com/crossplane-contrib/provider-upjet-gcp/actions/runs/10771724541

NAME                                       SYNCED   READY   EXTERNAL-NAME   AGE
cluster.container.gcp.upbound.io/primary   True     True    primary         8m30s

[turkenf]

/test-examples="examples/container/v1beta1/cluster.yaml"

[turkenf]

/test-examples="examples/cloudplatform/v1beta1/projectiammember.yaml"

[turkenf]

/test-examples="examples/cloudplatform/v1beta1/projectservice.yaml"

[mergenci]

Thanks @turkenf. LGTM.

[github-actions]

Successfully created backport PR #615 for release-1.8.